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

Unified Diff: content/browser/media/android/media_drm_credential_manager.cc

Issue 1427183002: Move MediaDrmBridge provision communication to native side. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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/browser/media/android/media_drm_credential_manager.cc
diff --git a/content/browser/media/android/media_drm_credential_manager.cc b/content/browser/media/android/media_drm_credential_manager.cc
index 8e62770d3f1ae4ca6082f692102bd7d3747f293a..ddd7236b5e64769162be651bfde6919abe083de5 100644
--- a/content/browser/media/android/media_drm_credential_manager.cc
+++ b/content/browser/media/android/media_drm_credential_manager.cc
@@ -13,6 +13,7 @@
#include "base/thread_task_runner_handle.h"
#include "jni/MediaDrmCredentialManager_jni.h"
#include "media/base/android/media_drm_bridge.h"
+#include "media/base/android/provision_fetcher.h"
#include "url/gurl.h"
#include "widevine_cdm_version.h" // In SHARED_INTERMEDIATE_DIR.
@@ -91,8 +92,11 @@ void MediaDrmCredentialManager::OnResetCredentialsCompleted(
// TODO(ddorwin): The key system should be passed in. http://crbug.com/459400
bool MediaDrmCredentialManager::ResetCredentialsInternal(
SecurityLevel security_level) {
- media_drm_bridge_ =
- media::MediaDrmBridge::CreateWithoutSessionSupport(kWidevineKeySystem);
+ scoped_ptr<media::ProvisionFetcher> provision_fetcher =
+ media::ProvisionFetcher::GetFactory()->CreateFetcher();
+
+ media_drm_bridge_ = media::MediaDrmBridge::CreateWithoutSessionSupport(
+ kWidevineKeySystem, provision_fetcher.Pass());
xhwang 2015/11/02 20:37:04 Can we create the ProvisionFetcher directly in MDB
Tima Vaisburd 2015/11/05 02:24:07 Done for this path, but not for the other.
if (!media_drm_bridge_)
return false;

Powered by Google App Engine
This is Rietveld 408576698