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

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

Issue 1099633003: Bind BrowserCdmManager callbacks to UI thread before passing to CDM. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: header for ~CdmKeyInformation (BindToCurrentLoop call required definition) Created 5 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 ee2107a0799ac4125844377cba009870b5e107d0..10bcfd3247a8d42fe204539d2fd80e166f6c79cc 100644
--- a/chromecast/browser/media/cast_browser_cdm_factory.cc
+++ b/chromecast/browser/media/cast_browser_cdm_factory.cc
@@ -9,6 +9,8 @@
#include "base/message_loop/message_loop_proxy.h"
#include "chromecast/browser/media/cma_message_loop.h"
#include "chromecast/media/cdm/browser_cdm_cast.h"
+#include "media/base/bind_to_current_loop.h"
+#include "media/base/cdm_key_information.h"
namespace chromecast {
namespace media {
@@ -36,9 +38,12 @@ scoped_ptr<::media::BrowserCdm> CastBrowserCdmFactory::CreateBrowserCdm(
CmaMessageLoop::GetMessageLoopProxy()->PostTask(
FROM_HERE,
base::Bind(&BrowserCdmCast::Initialize,
- base::Unretained(browser_cdm.get()), session_message_cb,
- session_closed_cb, legacy_session_error_cb,
- session_keys_change_cb, session_expiration_update_cb));
+ base::Unretained(browser_cdm.get()),
+ ::media::BindToCurrentLoop(session_message_cb),
+ ::media::BindToCurrentLoop(session_closed_cb),
+ ::media::BindToCurrentLoop(legacy_session_error_cb),
+ ::media::BindToCurrentLoop(session_keys_change_cb),
+ ::media::BindToCurrentLoop(session_expiration_update_cb)));
return make_scoped_ptr(
new BrowserCdmCastUi(browser_cdm.Pass(),
CmaMessageLoop::GetMessageLoopProxy()));
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698