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

Side by Side 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chromecast/browser/media/cast_browser_cdm_factory.h" 5 #include "chromecast/browser/media/cast_browser_cdm_factory.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/message_loop/message_loop_proxy.h" 9 #include "base/message_loop/message_loop_proxy.h"
10 #include "chromecast/browser/media/cma_message_loop.h" 10 #include "chromecast/browser/media/cma_message_loop.h"
11 #include "chromecast/media/cdm/browser_cdm_cast.h" 11 #include "chromecast/media/cdm/browser_cdm_cast.h"
12 #include "media/base/bind_to_current_loop.h"
13 #include "media/base/cdm_key_information.h"
12 14
13 namespace chromecast { 15 namespace chromecast {
14 namespace media { 16 namespace media {
15 17
16 scoped_ptr<::media::BrowserCdm> CastBrowserCdmFactory::CreateBrowserCdm( 18 scoped_ptr<::media::BrowserCdm> CastBrowserCdmFactory::CreateBrowserCdm(
17 const std::string& key_system_name, 19 const std::string& key_system_name,
18 bool use_secure_surface, 20 bool use_secure_surface,
19 const ::media::SessionMessageCB& session_message_cb, 21 const ::media::SessionMessageCB& session_message_cb,
20 const ::media::SessionClosedCB& session_closed_cb, 22 const ::media::SessionClosedCB& session_closed_cb,
21 const ::media::LegacySessionErrorCB& legacy_session_error_cb, 23 const ::media::LegacySessionErrorCB& legacy_session_error_cb,
22 const ::media::SessionKeysChangeCB& session_keys_change_cb, 24 const ::media::SessionKeysChangeCB& session_keys_change_cb,
23 const ::media::SessionExpirationUpdateCB& session_expiration_update_cb) { 25 const ::media::SessionExpirationUpdateCB& session_expiration_update_cb) {
24 DCHECK(!use_secure_surface) << "Chromecast does not use |use_secure_surface|"; 26 DCHECK(!use_secure_surface) << "Chromecast does not use |use_secure_surface|";
25 27
26 CastKeySystem key_system(GetKeySystemByName(key_system_name)); 28 CastKeySystem key_system(GetKeySystemByName(key_system_name));
27 29
28 scoped_ptr<chromecast::media::BrowserCdmCast> browser_cdm; 30 scoped_ptr<chromecast::media::BrowserCdmCast> browser_cdm;
29 if (key_system == chromecast::media::KEY_SYSTEM_CLEAR_KEY) { 31 if (key_system == chromecast::media::KEY_SYSTEM_CLEAR_KEY) {
30 // TODO(gunsch): handle ClearKey decryption. See crbug.com/441957 32 // TODO(gunsch): handle ClearKey decryption. See crbug.com/441957
31 } else { 33 } else {
32 browser_cdm = CreatePlatformBrowserCdm(key_system); 34 browser_cdm = CreatePlatformBrowserCdm(key_system);
33 } 35 }
34 36
35 if (browser_cdm) { 37 if (browser_cdm) {
36 CmaMessageLoop::GetMessageLoopProxy()->PostTask( 38 CmaMessageLoop::GetMessageLoopProxy()->PostTask(
37 FROM_HERE, 39 FROM_HERE,
38 base::Bind(&BrowserCdmCast::Initialize, 40 base::Bind(&BrowserCdmCast::Initialize,
39 base::Unretained(browser_cdm.get()), session_message_cb, 41 base::Unretained(browser_cdm.get()),
40 session_closed_cb, legacy_session_error_cb, 42 ::media::BindToCurrentLoop(session_message_cb),
41 session_keys_change_cb, session_expiration_update_cb)); 43 ::media::BindToCurrentLoop(session_closed_cb),
44 ::media::BindToCurrentLoop(legacy_session_error_cb),
45 ::media::BindToCurrentLoop(session_keys_change_cb),
46 ::media::BindToCurrentLoop(session_expiration_update_cb)));
42 return make_scoped_ptr( 47 return make_scoped_ptr(
43 new BrowserCdmCastUi(browser_cdm.Pass(), 48 new BrowserCdmCastUi(browser_cdm.Pass(),
44 CmaMessageLoop::GetMessageLoopProxy())); 49 CmaMessageLoop::GetMessageLoopProxy()));
45 } 50 }
46 51
47 LOG(INFO) << "No matching key system found."; 52 LOG(INFO) << "No matching key system found.";
48 return scoped_ptr< ::media::BrowserCdm>(); 53 return scoped_ptr< ::media::BrowserCdm>();
49 } 54 }
50 55
51 } // namespace media 56 } // namespace media
52 } // namespace chromecast 57 } // namespace chromecast
OLDNEW
« 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