Plumb |use_secure_codecs| through to BrowserCdmFactoryAndroid.

chromecast/browser/media/cast_browser_cdm_factory.cc

Index: chromecast/browser/media/cast_browser_cdm_factory.cc
diff --git a/chromecast/browser/media/cast_browser_cdm_factory.cc b/chromecast/browser/media/cast_browser_cdm_factory.cc
index 10bcfd3247a8d42fe204539d2fd80e166f6c79cc..8f46413df5279b096d7d87008bec3f22a1e718a9 100644
--- a/chromecast/browser/media/cast_browser_cdm_factory.cc
+++ b/chromecast/browser/media/cast_browser_cdm_factory.cc
@@ -17,13 +17,13 @@ namespace media {
 
 scoped_ptr<::media::BrowserCdm> CastBrowserCdmFactory::CreateBrowserCdm(
     const std::string& key_system_name,
-    bool use_secure_surface,
+    bool use_secure_codecs,
     const ::media::SessionMessageCB& session_message_cb,
     const ::media::SessionClosedCB& session_closed_cb,
     const ::media::LegacySessionErrorCB& legacy_session_error_cb,
     const ::media::SessionKeysChangeCB& session_keys_change_cb,
     const ::media::SessionExpirationUpdateCB& session_expiration_update_cb) {
-  DCHECK(!use_secure_surface) << "Chromecast does not use |use_secure_surface|";
+  DCHECK(!use_secure_codecs) << "Chromecast does not use |use_secure_codecs|";

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

I don't know what it would take to make this happen, but it is probably more
appropriate to pass true and say "Chromecast always uses secure codecs."

[gunsch, 2015/05/13 06:52:37]

That would typically be more accurate, though not in all cases.

For example, we have a BrowserCdm-based ClearKey implementation, which is not a
secure codec.

 
   CastKeySystem key_system(GetKeySystemByName(key_system_name));