Plumb |use_secure_codecs| through to BrowserCdmFactoryAndroid.

content/renderer/media/crypto/render_cdm_factory.h

Index: content/renderer/media/crypto/render_cdm_factory.h
diff --git a/content/renderer/media/crypto/render_cdm_factory.h b/content/renderer/media/crypto/render_cdm_factory.h
index 1be9dcfe306a77ee82b9c8914bad2a8b72ee39c1..80502db933288d8bbf786b8ec51018cd8dfbba60 100644
--- a/content/renderer/media/crypto/render_cdm_factory.h
+++ b/content/renderer/media/crypto/render_cdm_factory.h
@@ -19,6 +19,10 @@
 
 class GURL;
 
+namespace media {
+struct CdmConfig;
+}  // namespace media
+
 namespace content {
 
 #if defined(ENABLE_BROWSER_CDMS)
@@ -42,9 +46,8 @@ class RenderCdmFactory : public media::CdmFactory, public RenderFrameObserver {
   // CdmFactory implementation.
   void Create(
       const std::string& key_system,
-      bool allow_distinctive_identifier,
-      bool allow_persistent_state,
       const GURL& security_origin,
+      const media::CdmConfig& cdm_config,

[ddorwin, 2015/05/13 06:03:20]

nit: The config seems more related to the key_system than the origin. What was
your reasoning? Consistency for the first two parameters (because the third type
may change)?

[sandersd (OOO until July 31), 2015/05/13 18:17:14]

If anything, |security_origin| would make sense as a field in CdmConfig. I based
the ordering from most-to-least important/descriptive.

       const media::SessionMessageCB& session_message_cb,
       const media::SessionClosedCB& session_closed_cb,
       const media::LegacySessionErrorCB& legacy_session_error_cb,