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

Side by Side Diff: media/base/android/android_cdm_factory.cc

Issue 2823513002: [Clank] Load origin id before create MediaDrmBridge (Closed)
Patch Set: Rebase + Nit Created 3 years, 5 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "media/base/android/android_cdm_factory.h" 5 #include "media/base/android/android_cdm_factory.h"
6 6
7 #include "base/feature_list.h" 7 #include "base/feature_list.h"
8 #include "base/strings/string_number_conversions.h" 8 #include "base/strings/string_number_conversions.h"
9 #include "media/base/android/media_drm_bridge.h" 9 #include "media/base/android/media_drm_bridge.h"
10 #include "media/base/bind_to_current_loop.h" 10 #include "media/base/bind_to_current_loop.h"
11 #include "media/base/cdm_config.h" 11 #include "media/base/cdm_config.h"
12 #include "media/base/content_decryption_module.h" 12 #include "media/base/content_decryption_module.h"
13 #include "media/base/key_system_names.h" 13 #include "media/base/key_system_names.h"
14 #include "media/base/key_systems.h" 14 #include "media/base/key_systems.h"
15 #include "media/base/media_switches.h" 15 #include "media/base/media_switches.h"
16 #include "media/cdm/aes_decryptor.h" 16 #include "media/cdm/aes_decryptor.h"
17 #include "third_party/widevine/cdm/widevine_cdm_common.h" 17 #include "third_party/widevine/cdm/widevine_cdm_common.h"
18 #include "url/gurl.h" 18 #include "url/gurl.h"
19 19
20 namespace media { 20 namespace media {
21 namespace {
22 void OnCdmCreated(const std::string& key_system,
23 MediaDrmBridge::SecurityLevel security_level,
24 const CdmCreatedCB& bound_cdm_created_cb,
25 scoped_refptr<MediaDrmBridge> cdm) {
26 if (!cdm) {
27 std::string error_message = "MediaDrmBridge cannot be created for " +
28 key_system + " with security level " +
29 base::IntToString(security_level);
30 LOG(ERROR) << error_message;
31 bound_cdm_created_cb.Run(nullptr, error_message);
32 return;
33 }
34
35 // Success!
36 bound_cdm_created_cb.Run(cdm, "");
37 }
38 } // namespace
21 39
22 AndroidCdmFactory::AndroidCdmFactory(const CreateFetcherCB& create_fetcher_cb, 40 AndroidCdmFactory::AndroidCdmFactory(const CreateFetcherCB& create_fetcher_cb,
23 const CreateStorageCB& create_storage_cb) 41 const CreateStorageCB& create_storage_cb)
24 : create_fetcher_cb_(create_fetcher_cb), 42 : create_fetcher_cb_(create_fetcher_cb),
25 create_storage_cb_(create_storage_cb) {} 43 create_storage_cb_(create_storage_cb) {}
26 44
27 AndroidCdmFactory::~AndroidCdmFactory() {} 45 AndroidCdmFactory::~AndroidCdmFactory() {}
28 46
29 void AndroidCdmFactory::Create( 47 void AndroidCdmFactory::Create(
30 const std::string& key_system, 48 const std::string& key_system,
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 // Assume other key systems require hardware-secure codecs and thus do not 91 // Assume other key systems require hardware-secure codecs and thus do not
74 // support full compositing. 92 // support full compositing.
75 error_message = 93 error_message =
76 key_system + 94 key_system +
77 " may require use_video_overlay_for_embedded_encrypted_video"; 95 " may require use_video_overlay_for_embedded_encrypted_video";
78 NOTREACHED() << error_message; 96 NOTREACHED() << error_message;
79 bound_cdm_created_cb.Run(nullptr, error_message); 97 bound_cdm_created_cb.Run(nullptr, error_message);
80 return; 98 return;
81 } 99 }
82 100
83 scoped_refptr<MediaDrmBridge> cdm(MediaDrmBridge::Create( 101 MediaDrmBridge::Create(key_system, security_origin, security_level,
84 key_system, security_origin, security_level, create_fetcher_cb_, 102 create_fetcher_cb_, create_storage_cb_,
85 create_storage_cb_, session_message_cb, session_closed_cb, 103 session_message_cb, session_closed_cb,
86 session_keys_change_cb, session_expiration_update_cb)); 104 session_keys_change_cb, session_expiration_update_cb,
87 if (!cdm) { 105 base::BindOnce(&OnCdmCreated, key_system,
88 error_message = "MediaDrmBridge cannot be created for " + key_system + 106 security_level, bound_cdm_created_cb));
89 " with security level " + base::IntToString(security_level);
90 LOG(ERROR) << error_message;
91 bound_cdm_created_cb.Run(nullptr, error_message);
92 return;
93 }
94
95 // Success!
96 bound_cdm_created_cb.Run(cdm, "");
97 } 107 }
98 108
99 } // namespace media 109 } // namespace media
OLDNEW
« no previous file with comments | « chrome/browser/media/android/cdm/media_drm_credential_manager.cc ('k') | media/base/android/media_drm_bridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698