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

Unified Diff: chromecast/browser/media/cast_browser_cdm_factory.cc

Issue 1407933010: media: Make MediaKeys ref-counted. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase only Created 5 years, 1 month 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: chromecast/browser/media/cast_browser_cdm_factory.cc
diff --git a/chromecast/browser/media/cast_browser_cdm_factory.cc b/chromecast/browser/media/cast_browser_cdm_factory.cc
index 62a1df259652ebcd91b3423b5a0ce9295eb56ea7..e93927b0f488c4e1d8641ac0ad7703ced9dcf571 100644
--- a/chromecast/browser/media/cast_browser_cdm_factory.cc
+++ b/chromecast/browser/media/cast_browser_cdm_factory.cc
@@ -15,7 +15,7 @@
namespace chromecast {
namespace media {
-::media::ScopedBrowserCdmPtr CastBrowserCdmFactory::CreateBrowserCdm(
+scoped_refptr<::media::MediaKeys> CastBrowserCdmFactory::CreateBrowserCdm(
const std::string& key_system_name,
bool use_hw_secure_codecs,
const ::media::SessionMessageCB& session_message_cb,
@@ -28,7 +28,7 @@ namespace media {
CastKeySystem key_system(GetKeySystemByName(key_system_name));
- scoped_ptr<chromecast::media::BrowserCdmCast> browser_cdm;
+ scoped_refptr<chromecast::media::BrowserCdmCast> browser_cdm;
if (key_system == chromecast::media::KEY_SYSTEM_CLEAR_KEY) {
// TODO(gunsch): handle ClearKey decryption. See crbug.com/441957
} else {
@@ -45,17 +45,17 @@ namespace media {
::media::BindToCurrentLoop(legacy_session_error_cb),
::media::BindToCurrentLoop(session_keys_change_cb),
::media::BindToCurrentLoop(session_expiration_update_cb)));
- return ::media::ScopedBrowserCdmPtr(new BrowserCdmCastUi(
- browser_cdm.Pass(), MediaMessageLoop::GetTaskRunner()));
+ return scoped_refptr<::media::MediaKeys>(
+ new BrowserCdmCastUi(browser_cdm, MediaMessageLoop::GetTaskRunner()));
}
LOG(INFO) << "No matching key system found.";
- return ::media::ScopedBrowserCdmPtr();
+ return nullptr;
}
-scoped_ptr<BrowserCdmCast> CastBrowserCdmFactory::CreatePlatformBrowserCdm(
+scoped_refptr<BrowserCdmCast> CastBrowserCdmFactory::CreatePlatformBrowserCdm(
const CastKeySystem& key_system) {
- return scoped_ptr<BrowserCdmCast>();
+ return nullptr;
}
} // namespace media
« no previous file with comments | « chromecast/browser/media/cast_browser_cdm_factory.h ('k') | chromecast/browser/media/cma_message_filter_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698