Encrypted Media: Implement IPC based SetCdm().

content/renderer/media/webcontentdecryptionmodule_impl.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/webcontentdecryptionmodule_impl.h"
 
 #include <map>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/strings/string_util.h"
 #include "content/renderer/media/cdm_session_adapter.h"
 #include "content/renderer/media/webcontentdecryptionmodulesession_impl.h"
 #include "media/base/media_keys.h"
 
 #if defined(ENABLE_PEPPER_CDMS)
 #include "content/renderer/media/crypto/pepper_cdm_wrapper_impl.h"
+#elif defined(OS_ANDROID)
+#include "content/renderer/media/android/renderer_media_player_manager.h"
 #endif
 
 namespace content {
 
 WebContentDecryptionModuleImpl* WebContentDecryptionModuleImpl::Create(
     const base::string16& key_system) {
   // TODO(ddorwin): Guard against this in supported types check and remove this.
   // Chromium only supports ASCII key systems.
   if (!IsStringASCII(key_system)) {
     NOTREACHED();
     return NULL;
   }
 
   scoped_refptr<CdmSessionAdapter> adapter(new CdmSessionAdapter());
-  if (!adapter->Initialize(
+
+#if defined(OS_ANDROID)
+  int cdm_id = RendererMediaPlayerManager::kInvalidCdmId;
+  if (adapter->Initialize(&cdm_id, UTF16ToASCII(key_system)))
+    return new WebContentDecryptionModuleImpl(adapter, cdm_id);
+#else  // defined(OS_ANDROID)
+  if (adapter->Initialize(
 #if defined(ENABLE_PEPPER_CDMS)
           // TODO(jrummell): Figure out how to get a WebFrame from Blink (or
           // something equivalent) so the plugin can actually get created.
           // http://crbug.com/250049
           base::Bind(&PepperCdmWrapperImpl::Create,
                      static_cast<blink::WebFrame*>(NULL)),
-#endif
+#endif  // defined(ENABLE_PEPPER_CDMS)
           UTF16ToASCII(key_system))) {
-    return NULL;
+    return new WebContentDecryptionModuleImpl(adapter);
   }
-
-  return new WebContentDecryptionModuleImpl(adapter);
+#endif  // defined(OS_ANDROID)
+  return NULL;
 }
 
+#if defined(OS_ANDROID)
+WebContentDecryptionModuleImpl::WebContentDecryptionModuleImpl(
+    scoped_refptr<CdmSessionAdapter> adapter, int cdm_id)
+    : adapter_(adapter), cdm_id_(cdm_id) {}
+#else
 WebContentDecryptionModuleImpl::WebContentDecryptionModuleImpl(
     scoped_refptr<CdmSessionAdapter> adapter)
-    : adapter_(adapter) {
-}
+    : adapter_(adapter) {}
+#endif

[xhwang, 2014/03/12 01:20:11]

This code (l.37-65) is really messy with #ifdefs. And I don't find a cleaner way
to do this. Note that:
- Only for Android we need line 38.
- WebCDMImpl::Ctor takes one or two parameters, so that:
  * The "return" code on l.40 and l.51 are different.
  * The ctor definition (l.57-65) is hard to write cleanly with inline #ifdefs.
That's why I have them separate (58-60 and 62-64, well I may reuse line 58 and
62, but still...)

I feel having the adapter to store the cdm_id_ and return it via a
CdmSessionAdapter::GetCdmId() call makes the code a bit cleaner. WDYT?

[ddorwin, 2014/03/18 22:51:08]

How much smaller is the diff if we track this in the adapter? We still need line
80-85 for WMPA to call, so adding that to the adapter would be worse.

I would try to avoid duplicating calls and just ifdef individual parameters
where possible.

[xhwang, 2014/03/25 21:49:50]

ifdef'ed individual parameters where possible.

 
 WebContentDecryptionModuleImpl::~WebContentDecryptionModuleImpl() {
 }
 
 // The caller owns the created session.
 blink::WebContentDecryptionModuleSession*
 WebContentDecryptionModuleImpl::createSession(
     blink::WebContentDecryptionModuleSession::Client* client) {
   return adapter_->CreateSession(client);
 }
 
 media::Decryptor* WebContentDecryptionModuleImpl::GetDecryptor() {
   return adapter_->GetDecryptor();
 }
 
+#if defined(OS_ANDROID)
+int WebContentDecryptionModuleImpl::GetCdmId() {
+  return cdm_id_;
+}
+#endif  // defined(OS_ANDROID)
+
 }  // namespace content