Move MediaDrmBridge provision communication to native side.

content/browser/media/cdm/browser_cdm_manager.cc

Index: content/browser/media/cdm/browser_cdm_manager.cc
diff --git a/content/browser/media/cdm/browser_cdm_manager.cc b/content/browser/media/cdm/browser_cdm_manager.cc
index 7294ef40c4d78752c14987e4b71c9edc56024b3f..04e983567d1b4bcf4e5251c1a483860ed7c7b7e1 100644
--- a/content/browser/media/cdm/browser_cdm_manager.cc
+++ b/content/browser/media/cdm/browser_cdm_manager.cc
@@ -25,6 +25,7 @@
 #include "media/base/limits.h"
 
 #if defined(OS_ANDROID)
+#include "content/public/browser/android/cdm_provision_fetcher.h"
 #include "content/public/common/renderer_preferences.h"
 #endif
 
@@ -539,11 +540,22 @@ void BrowserCdmManager::AddCdm(int render_frame_id,
   DCHECK(task_runner_->RunsTasksOnCurrentThread());
   DCHECK(!GetCdm(render_frame_id, cdm_id));
 
+  scoped_ptr<media::ProvisionFetcher> provision_fetcher;
+
+#if defined(OS_ANDROID)
+  net::URLRequestContextGetter* context =
+      RenderProcessHost::FromID(render_process_id_)
+          ->GetBrowserContext()
+          ->GetRequestContext();
+  DCHECK(context);
+  provision_fetcher = CDMProvisionFetcher::CreateWithURLContext(context);

[xhwang, 2015/11/06 23:08:18]

Can you just create a URLProvisionFetch directly here, without going through
CDMProvisionFetcher?

[Tima Vaisburd, 2015/11/11 03:03:34]

Done.

+#endif
+
   scoped_ptr<SimplePromise> promise(new SimplePromise(
       weak_ptr_factory_.GetWeakPtr(), render_frame_id, cdm_id, promise_id));
 
   ScopedBrowserCdmPtr cdm(media::CreateBrowserCdm(
-      key_system, use_hw_secure_codecs,
+      key_system, use_hw_secure_codecs, provision_fetcher.Pass(),

[Tima Vaisburd, 2015/11/05 02:24:07]

I pass |provision_fetcher| here as a parameter, and so I had to move the
ProvisionFetcher definition to media/base/, although it is used only for
Android. I wonder whether there is a better way.

[Tima Vaisburd, 2015/11/05 02:28:54]

An alternative could be adding another factory method in MediaClientAndroid,
e.g. CreateFetcherForRenderProcess(int id) and implementing it in every subclass
of MediaClientAndroid. WDYT?

[xhwang, 2015/11/06 23:08:19]

See comments above.

ProvisionFetcher is only used by MDB, we probably still want to keep it in
media/base/android. See my comment above, we don't want this parameter in
Create() call, which is used for all CDM creations.

[Tima Vaisburd, 2015/11/11 03:03:34]

Done.

[xhwang, 2015/11/11 09:53:21]

MediaClients are consumed by media/, which has no knowledge of what a render
process is.

[Tima Vaisburd, 2015/11/11 23:26:35]

MediaClient is not used for the provisioning purpose any more, I think my
question was outdated. Or did I miss your point here?

       BROWSER_CDM_MANAGER_CB(OnSessionMessage),
       BROWSER_CDM_MANAGER_CB(OnSessionClosed),
       BROWSER_CDM_MANAGER_CB(OnLegacySessionError),