Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2133)

Unified Diff: content/renderer/media/cdm/ppapi_decryptor.cc

Issue 2255943002: EME: Remove obsolete legacy APIs related to versions of prefixed EME (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix build and add bug reference for obsoletes Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/cdm/ppapi_decryptor.cc
diff --git a/content/renderer/media/cdm/ppapi_decryptor.cc b/content/renderer/media/cdm/ppapi_decryptor.cc
index c077112ee8bdeefd0e72d24f4ca584583b0117de..dd07e01c24ee6fe79bf9d85cbb478d0bc06ecc2c 100644
--- a/content/renderer/media/cdm/ppapi_decryptor.cc
+++ b/content/renderer/media/cdm/ppapi_decryptor.cc
@@ -34,7 +34,6 @@ void PpapiDecryptor::Create(
const CreatePepperCdmCB& create_pepper_cdm_cb,
const media::SessionMessageCB& session_message_cb,
const media::SessionClosedCB& session_closed_cb,
- const media::LegacySessionErrorCB& legacy_session_error_cb,
const media::SessionKeysChangeCB& session_keys_change_cb,
const media::SessionExpirationUpdateCB& session_expiration_update_cb,
const media::CdmCreatedCB& cdm_created_cb) {
@@ -56,10 +55,9 @@ void PpapiDecryptor::Create(
return;
}
- scoped_refptr<PpapiDecryptor> ppapi_decryptor(
- new PpapiDecryptor(std::move(pepper_cdm_wrapper), session_message_cb,
- session_closed_cb, legacy_session_error_cb,
- session_keys_change_cb, session_expiration_update_cb));
+ scoped_refptr<PpapiDecryptor> ppapi_decryptor(new PpapiDecryptor(
+ std::move(pepper_cdm_wrapper), session_message_cb, session_closed_cb,
+ session_keys_change_cb, session_expiration_update_cb));
// |ppapi_decryptor| ownership is passed to the promise.
std::unique_ptr<media::CdmInitializedPromise> promise(
@@ -73,13 +71,11 @@ PpapiDecryptor::PpapiDecryptor(
std::unique_ptr<PepperCdmWrapper> pepper_cdm_wrapper,
const media::SessionMessageCB& session_message_cb,
const media::SessionClosedCB& session_closed_cb,
- const media::LegacySessionErrorCB& legacy_session_error_cb,
const media::SessionKeysChangeCB& session_keys_change_cb,
const media::SessionExpirationUpdateCB& session_expiration_update_cb)
: pepper_cdm_wrapper_(std::move(pepper_cdm_wrapper)),
session_message_cb_(session_message_cb),
session_closed_cb_(session_closed_cb),
- legacy_session_error_cb_(legacy_session_error_cb),
session_keys_change_cb_(session_keys_change_cb),
session_expiration_update_cb_(session_expiration_update_cb),
render_task_runner_(base::ThreadTaskRunnerHandle::Get()),
@@ -87,7 +83,6 @@ PpapiDecryptor::PpapiDecryptor(
DCHECK(pepper_cdm_wrapper_.get());
DCHECK(!session_message_cb_.is_null());
DCHECK(!session_closed_cb_.is_null());
- DCHECK(!legacy_session_error_cb_.is_null());
DCHECK(!session_keys_change_cb.is_null());
DCHECK(!session_expiration_update_cb.is_null());
}
@@ -106,7 +101,6 @@ void PpapiDecryptor::InitializeCdm(
key_system, allow_distinctive_identifier, allow_persistent_state,
base::Bind(&PpapiDecryptor::OnSessionMessage, weak_this),
base::Bind(&PpapiDecryptor::OnSessionClosed, weak_this),
- base::Bind(&PpapiDecryptor::OnLegacySessionError, weak_this),
base::Bind(&PpapiDecryptor::OnSessionKeysChange, weak_this),
base::Bind(&PpapiDecryptor::OnSessionExpirationUpdate, weak_this),
base::Bind(&PpapiDecryptor::OnFatalPluginError, weak_this),
@@ -396,11 +390,9 @@ void PpapiDecryptor::OnDecoderInitialized(StreamType stream_type,
void PpapiDecryptor::OnSessionMessage(const std::string& session_id,
MessageType message_type,
- const std::vector<uint8_t>& message,
- const GURL& legacy_destination_url) {
+ const std::vector<uint8_t>& message) {
DCHECK(render_task_runner_->BelongsToCurrentThread());
- session_message_cb_.Run(session_id, message_type, message,
- legacy_destination_url);
+ session_message_cb_.Run(session_id, message_type, message);
}
void PpapiDecryptor::OnSessionKeysChange(const std::string& session_id,
@@ -430,16 +422,6 @@ void PpapiDecryptor::OnSessionClosed(const std::string& session_id) {
session_closed_cb_.Run(session_id);
}
-void PpapiDecryptor::OnLegacySessionError(
- const std::string& session_id,
- MediaKeys::Exception exception_code,
- uint32_t system_code,
- const std::string& error_description) {
- DCHECK(render_task_runner_->BelongsToCurrentThread());
- legacy_session_error_cb_.Run(session_id, exception_code, system_code,
- error_description);
-}
-
void PpapiDecryptor::AttemptToResumePlayback() {
if (!new_audio_key_cb_.is_null())
new_audio_key_cb_.Run();

Powered by Google App Engine
This is Rietveld 408576698