Introduce CdmFactory interface.

content/renderer/media/crypto/render_cdm_factory.cc

Index: content/renderer/media/crypto/render_cdm_factory.cc
diff --git a/content/renderer/media/crypto/content_decryption_module_factory.cc b/content/renderer/media/crypto/render_cdm_factory.cc
similarity index 82%
rename from content/renderer/media/crypto/content_decryption_module_factory.cc
rename to content/renderer/media/crypto/render_cdm_factory.cc
index 45db511c5247d5d3df2e8bb596a1e9fa41c4fdc7..244e48f50c232edede87ed15a4d026ce00c65fd9 100644
--- a/content/renderer/media/crypto/content_decryption_module_factory.cc
+++ b/content/renderer/media/crypto/render_cdm_factory.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/renderer/media/crypto/content_decryption_module_factory.h"
+#include "content/renderer/media/crypto/render_cdm_factory.h"
 
 #include "base/logging.h"
 #include "content/renderer/media/crypto/key_systems.h"
@@ -18,21 +18,36 @@
 
 namespace content {
 
-scoped_ptr<media::MediaKeys> ContentDecryptionModuleFactory::Create(
-    const std::string& key_system,
-    const GURL& security_origin,
 #if defined(ENABLE_PEPPER_CDMS)
-    const CreatePepperCdmCB& create_pepper_cdm_cb,
+RenderCdmFactory::RenderCdmFactory(
+    const CreatePepperCdmCB& create_pepper_cdm_cb)
+    : create_pepper_cdm_cb_(create_pepper_cdm_cb) {
+}
 #elif defined(ENABLE_BROWSER_CDMS)
-    RendererCdmManager* manager,
-    int* cdm_id,
+RenderCdmFactory::RenderCdmFactory(RendererCdmManager* manager)
+    : manager_(manager) {
+}
+#else
+RenderCdmFactory::RenderCdmFactory() {
+}
 #endif  // defined(ENABLE_PEPPER_CDMS)
+
+RenderCdmFactory::~RenderCdmFactory() {
+}
+
+scoped_ptr<media::MediaKeys> RenderCdmFactory::Create(
+    const std::string& key_system,
+    const GURL& security_origin,
+#if defined(ENABLE_BROWSER_CDMS)
+    int* cdm_id,
+#endif
     const media::SessionMessageCB& session_message_cb,
     const media::SessionReadyCB& session_ready_cb,
     const media::SessionClosedCB& session_closed_cb,
     const media::SessionErrorCB& session_error_cb,
     const media::SessionKeysChangeCB& session_keys_change_cb,
-    const media::SessionExpirationUpdateCB& session_expiration_update_cb) {
+    const media::SessionExpirationUpdateCB& session_expiration_update_cb)
+    const {
   // TODO(jrummell): Pass |security_origin| to all constructors.

[ddorwin, 2014/10/16 20:33:20]

jrummell: I think this now just means AesDecryptor, which will eat it.

[jrummell, 2014/10/16 22:40:32]

Looks that way.

   // TODO(jrummell): Enable the following line once blink code updated to
   // check the security origin before calling.
@@ -46,11 +61,12 @@ scoped_ptr<media::MediaKeys> ContentDecryptionModuleFactory::Create(
     return scoped_ptr<media::MediaKeys>(new media::AesDecryptor(
         session_message_cb, session_closed_cb, session_keys_change_cb));
   }
+
 #if defined(ENABLE_PEPPER_CDMS)
   return scoped_ptr<media::MediaKeys>(
       PpapiDecryptor::Create(key_system,
                              security_origin,
-                             create_pepper_cdm_cb,
+                             create_pepper_cdm_cb_,
                              session_message_cb,
                              session_ready_cb,
                              session_closed_cb,
@@ -61,7 +77,7 @@ scoped_ptr<media::MediaKeys> ContentDecryptionModuleFactory::Create(
   scoped_ptr<ProxyMediaKeys> proxy_media_keys =
       ProxyMediaKeys::Create(key_system,
                              security_origin,
-                             manager,
+                             manager_,
                              session_message_cb,
                              session_ready_cb,
                              session_closed_cb,