Commit 77f5922a authored by jrummell's avatar jrummell Committed by Commit bot

Relax DCHECK for |legacy_destination_url|

Only license requests should not specify |legacy_destination_url|.

BUG=448242

Review URL: https://codereview.chromium.org/804713003

Cr-Commit-Position: refs/heads/master@{#311550}
parent 39337115
...@@ -718,11 +718,11 @@ void CdmAdapter::OnSessionMessage(const char* session_id, ...@@ -718,11 +718,11 @@ void CdmAdapter::OnSessionMessage(const char* session_id,
uint32_t message_size, uint32_t message_size,
const char* legacy_destination_url, const char* legacy_destination_url,
uint32_t legacy_destination_url_size) { uint32_t legacy_destination_url_size) {
// Only license renewals should specify |legacy_destination_url|. // License requests should not specify |legacy_destination_url|.
// |legacy_destination_url| is not passed to unprefixed EME applications, // |legacy_destination_url| is not passed to unprefixed EME applications,
// so it can be removed when the prefixed API is removed. // so it can be removed when the prefixed API is removed.
PP_DCHECK(legacy_destination_url_size == 0 || PP_DCHECK(legacy_destination_url_size == 0 ||
message_type == cdm::MessageType::kLicenseRenewal); message_type != cdm::MessageType::kLicenseRequest);
PostOnMain(callback_factory_.NewCallback( PostOnMain(callback_factory_.NewCallback(
&CdmAdapter::SendSessionMessageInternal, &CdmAdapter::SendSessionMessageInternal,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment