Make unprefixed EME APIs use platform and Chromium.

Source/core/platform/graphics/ContentDecryptionModule.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 11 matching lines @@
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
-#include "core/html/ime/InputMethodContext.h"
+#include "core/platform/graphics/ContentDecryptionModule.h"
 
-#include "core/html/ime/Composition.h"
+#include "core/platform/NotImplemented.h"
+#include "core/platform/graphics/ContentDecryptionModuleSession.h"
+#include "public/platform/Platform.h"
 
 namespace WebCore {
 
-PassOwnPtr<InputMethodContext> InputMethodContext::create(HTMLElement* element)
+bool ContentDecryptionModule::supportsKeySystem(const String& keySystem)
 {
-    return adoptPtr(new InputMethodContext(element));
+    // FIXME: Chromium should handle this, possibly using
+    // MIMETypeRegistry::isSupportedEncryptedMediaMIMEType().
+    notImplemented();
+    if (keySystem == "org.w3.clearkey")
+        return true;
+
+    return false;

[abarth-chromium, 2013/06/12 23:03:51]

You can just write:

return keySystem == "org.w3.clearkey";

[ddorwin, 2013/06/12 23:55:54]

Done.

 }
 
-InputMethodContext::InputMethodContext(HTMLElement* element)
-    : m_enabled(false)
-    , m_composition(0)
-    , m_element(element)
+// Factory method: Chromium-implementation

[abarth-chromium, 2013/06/12 23:03:51]

No need for this comment.

[ddorwin, 2013/06/12 23:55:54]

Done.

+PassOwnPtr<ContentDecryptionModule> ContentDecryptionModule::create(const String& keySystem)
 {
-    ScriptWrappable::init(this);
+    ASSERT(!keySystem.isNull() && !keySystem.isEmpty());

[abarth-chromium, 2013/06/12 23:03:51]

isEmpty is a superset of isNull. You can just write:

ASSERT(!keySystem.isEmpty());

[ddorwin, 2013/06/12 23:55:54]

Done here and in MediaKeys.cpp.

+    OwnPtr<WebKit::WebContentDecryptionModule> cdm = adoptPtr(WebKit::Platform::current()->createContentDecryptionModule(keySystem));
+    if (!cdm)
+        return nullptr;
+    return adoptPtr(new ContentDecryptionModule(cdm.release()));

[abarth-chromium, 2013/06/12 23:03:51]

No need to call adoptPtr here.  You've already adopted the object. 
cdm.release() should work correctly.  (I'm surprised what you've written here
compiles, actually.)

[abarth-chromium, 2013/06/12 23:26:38]

Oops, sorry.  I misread the code.  You're right.  ;)

 }
 
-InputMethodContext::~InputMethodContext()
+ContentDecryptionModule::ContentDecryptionModule(PassOwnPtr<WebKit::WebContentDecryptionModule> cdm)
+    : m_cdm(cdm)
+{
+    ASSERT(m_cdm);
+}
+
+ContentDecryptionModule::~ContentDecryptionModule()
 {
 }
 
-Composition* InputMethodContext::composition() const
+bool ContentDecryptionModule::supportsMIMEType(const String& mimeType)
 {
-    // FIXME: Implement this. This should lazily update the composition object
-    // here.
-    return m_composition.get();
+    // FIXME: Chromium should handle this, possibly using
+    // MIMETypeRegistry::isSupportedEncryptedMediaMIMEType().
+    notImplemented();
+    return mimeType == "video/webm";
 }
 
-bool InputMethodContext::enabled() const
+PassOwnPtr<ContentDecryptionModuleSession> ContentDecryptionModule::createSession(ContentDecryptionModuleSessionClient* client)
 {
-    // FIXME: Implement this. Enabled state may change between calls from user
-    // action and the status should be retrieved here.
-    return m_enabled;
-}
-
-void InputMethodContext::setEnabled(bool enabled)
-{
-    // FIXME: Implement this. The enabled state should propagate to IME.
-    m_enabled = enabled;
-}
-
-String InputMethodContext::locale() const
-{
-    // FIXME: Implement this.
-    return emptyString();
-}
-
-void InputMethodContext::confirmComposition()
-{
-    // FIXME: Implement this.
-}
-
-void InputMethodContext::setCaretRectangle(Node* anchor, int x, int y, int w, int h)
-{
-    // FIXME: Implement this.
-}
-
-void InputMethodContext::setExclusionRectangle(Node* anchor, int x, int y, int w, int h)
-{
-    // FIXME: Implement this.
-}
-
-bool InputMethodContext::open()
-{
-    // FIXME: Implement this.
-    return false;
+    return adoptPtr(new ContentDecryptionModuleSession(*m_cdm, client));
 }
 
 } // namespace WebCore