Update MediaKeys interface for EME

content/renderer/media/webcontentdecryptionmodule_impl.cc

Index: content/renderer/media/webcontentdecryptionmodule_impl.cc
diff --git a/content/renderer/media/webcontentdecryptionmodule_impl.cc b/content/renderer/media/webcontentdecryptionmodule_impl.cc
index 9dcb39ee9ca98931c5f49c05035f805ed883e846..124f180c554e292628b9647271624149b8469915 100644
--- a/content/renderer/media/webcontentdecryptionmodule_impl.cc
+++ b/content/renderer/media/webcontentdecryptionmodule_impl.cc
@@ -14,7 +14,9 @@
 #include "base/strings/utf_string_conversions.h"
 #include "content/renderer/media/cdm_session_adapter.h"
 #include "content/renderer/media/crypto/key_systems.h"
+#include "content/renderer/media/webcontentdecryptionmoduleresult_helper.h"
 #include "content/renderer/media/webcontentdecryptionmodulesession_impl.h"
+#include "media/base/cdm_promise.h"
 #include "media/base/media_keys.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/web/WebSecurityOrigin.h"
@@ -78,7 +80,8 @@ WebContentDecryptionModuleImpl* WebContentDecryptionModuleImpl::Create(
 
 WebContentDecryptionModuleImpl::WebContentDecryptionModuleImpl(
     scoped_refptr<CdmSessionAdapter> adapter)
-    : adapter_(adapter) {}
+    : adapter_(adapter) {
+}
 
 WebContentDecryptionModuleImpl::~WebContentDecryptionModuleImpl() {
 }
@@ -97,6 +100,17 @@ WebContentDecryptionModuleImpl::createSession(
   return session;
 }
 
+void WebContentDecryptionModuleImpl::setServerCertificate(
+    const uint8* server_certificate,
+    size_t server_certificate_length,
+    blink::WebContentDecryptionModuleResult result) {
+  DCHECK(server_certificate);
+  adapter_->SetServerCertificate(server_certificate,
+                                 server_certificate_length,
+                                 make_scoped_ptr<media::SimpleCdmPromise>(

[ddorwin, 2014/09/24 17:37:46]

According to
https://code.google.com/p/chromium/codesearch#chromium/src/base/memory/scoped...,
you don't need the <>. That makes this simpler and probably makes the next
comment obsolete.

[jrummell, 2014/09/24 22:32:32]

Without the type make_scoped_ptr returns scoped_ptr<WebSimpleCdmPromise>, and it
can't convert that to scoped_ptr<SimpleCdmPromise>.

+                                     new WebSimpleCdmPromise(result)));

[ddorwin, 2014/09/24 17:37:46]

Would this be simpler if we made a templated Create() method for the Web*
promises?

[jrummell, 2014/09/24 22:32:32]

I could, but Create() would return scoped_ptr<WebSimpleCdmPromise>, so I would
have to write WebSimpleCdmPromise::Create(result).PassAs<SimpleCdmPromise>(),
which is no shorter. Or switch the template to <T, U>, with U being the type
returned by Create().

+}
+
 media::Decryptor* WebContentDecryptionModuleImpl::GetDecryptor() {
   return adapter_->GetDecryptor();
 }