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

Unified Diff: chrome/browser/media/android/cdm/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: Bound network context into callback function instead of using factory 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: chrome/browser/media/android/cdm/media_drm_credential_manager.cc
diff --git a/chrome/browser/media/android/cdm/media_drm_credential_manager.cc b/chrome/browser/media/android/cdm/media_drm_credential_manager.cc
index 26cbeca45035972ecee2839194a45eef8c5042f2..f7ab634e3d88b3470f2d2e11370ac7c31d3bf4c2 100644
--- a/chrome/browser/media/android/cdm/media_drm_credential_manager.cc
+++ b/chrome/browser/media/android/cdm/media_drm_credential_manager.cc
@@ -11,8 +11,11 @@
#include "base/location.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
+#include "chrome/browser/browser_process.h"
+#include "content/public/browser/android/provision_fetcher_factory.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.
@@ -89,8 +92,14 @@ 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);
+ // Create provision fetcher for the default browser http request context.
+ DCHECK(g_browser_process);
qinmin 2015/11/13 19:07:40 nit: no need for this, if this is null, the follow
Tima Vaisburd 2015/11/13 21:04:43 Removed.
+ scoped_ptr<media::ProvisionFetcher> provision_fetcher =
+ content::CreateProvisionFetcher(
+ g_browser_process->system_request_context());
+
+ media_drm_bridge_ = media::MediaDrmBridge::CreateWithoutSessionSupport(
+ kWidevineKeySystem, provision_fetcher.Pass());
if (!media_drm_bridge_)
return false;

Powered by Google App Engine
This is Rietveld 408576698