CryptoSession: Propagate err from GetSystemIdInternal in Open

Merge of http://go/wvgerrit/96509

Bug: 151231204
Test: GtsMediaTestCases
Change-Id: Ib29691fcea1c9d5fcfa981caf3e5bdab00cbb0bf
This commit is contained in:
Robert Shih
2020-03-26 13:54:05 -07:00
parent c113df815d
commit 07f40d1bed

View File

@@ -748,12 +748,13 @@ CdmResponseType CryptoSession::Open(SecurityLevel requested_security_level) {
open_ = true; open_ = true;
// Get System ID and save it. // Get System ID and save it.
if (GetSystemIdInternal(&system_id_) == NO_ERROR) { result = GetSystemIdInternal(&system_id_);
if (result == NO_ERROR) {
metrics_->crypto_session_system_id_.Record(system_id_); metrics_->crypto_session_system_id_.Record(system_id_);
} else { } else {
LOGE("Failed to fetch system ID"); LOGE("Failed to fetch system ID");
metrics_->crypto_session_system_id_.SetError(LOAD_SYSTEM_ID_ERROR); metrics_->crypto_session_system_id_.SetError(result);
return LOAD_SYSTEM_ID_ERROR; return result;
} }
// Set up request ID // Set up request ID