Media Capabilities: add plumbing between Blink and media/blink/.

third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp

Index: third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp
diff --git a/third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp b/third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp
index a91b81d967cffb7766f9bd9d6cc95d9f0269ae4e..43dce182c3a59696f062fefa5fc22e10bc0482e1 100644
--- a/third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp
+++ b/third_party/WebKit/Source/modules/media_capabilities/MediaDecodingAbility.cpp
@@ -6,20 +6,30 @@
 
 namespace blink {
 
-MediaDecodingAbility::MediaDecodingAbility() = default;
+// static
+MediaDecodingAbility* MediaDecodingAbility::take(
+    ScriptPromiseResolver*,
+    std::unique_ptr<WebMediaDecodingAbility> webMediaDecodingAbility) {
+  DCHECK(webMediaDecodingAbility);
+  return new MediaDecodingAbility(std::move(webMediaDecodingAbility));

[chcunningham, 2017/03/08 02:43:10]

I notice this class breaks the pattern of the *Configuration classes where we
are nesting the Web* object here rather than doing a converter function. Perhaps
thats standard practice for objects that originate in blink/platform and bubble
up - but figured I'd ask.

[mlamouri (slow - plz ping), 2017/03/10 11:26:42]

That's following the pattern of CallbackPromiseAdapter. ::take() is called with
::WebType automatically,

+}
 
 bool MediaDecodingAbility::supported() const {
-  return true;
+  return m_webMediaDecodingAbility->supported;
 }
 
 bool MediaDecodingAbility::smooth() const {
-  return true;
+  return m_webMediaDecodingAbility->smooth;
 }
 
 bool MediaDecodingAbility::powerEfficient() const {
-  return true;
+  return m_webMediaDecodingAbility->powerEfficient;
 }
 
 DEFINE_TRACE(MediaDecodingAbility) {}
 
+MediaDecodingAbility::MediaDecodingAbility(
+    std::unique_ptr<WebMediaDecodingAbility> webMediaDecodingAbility)
+    : m_webMediaDecodingAbility(std::move(webMediaDecodingAbility)) {}
+
 }  // namespace blink