Changeset 112205 in webkit

Timestamp:

Mar 26, 2012 9:41:38 PM (12 years ago)

Author:

scherkus@chromium.org

Message:

[chromium] Remove WebMediaElement from Chromium WebKit API.
​https://bugs.webkit.org/show_bug.cgi?id=82277

Reviewed by James Robinson.

This was added in r63859 but is no longer used by anyone.

• WebKit.gyp:
• public/WebMediaElement.h: Removed.
• src/WebMediaElement.cpp: Removed.

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• WebKit.gyp (modified) (2 diffs)
• public/WebMediaElement.h (deleted)
• src/WebMediaElement.cpp (deleted)
• src/WebMediaPlayerClientImpl.cpp (modified) (2 diffs)
• src/WebMediaPlayerClientImpl.h (modified) (2 diffs)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-03-26  Andrew Scherkus  <scherkus@chromium.org>
+
+        [chromium] Remove WebMediaElement from Chromium WebKit API.
+        https://bugs.webkit.org/show_bug.cgi?id=82277
+
+        Reviewed by James Robinson.
+
+        This was added in r63859 but is no longer used by anyone.
+
+        * WebKit.gyp:
+        * public/WebMediaElement.h: Removed.
+        * src/WebMediaElement.cpp: Removed.
+
 2012-03-26  Scott Byer  <scottbyer@chromium.org>
 

trunk/Source/WebKit/chromium/WebKit.gyp

@@ -200 +200 @@
                 'public/WebKit.h',
                 'public/WebLabelElement.h',
-                'public/WebMediaElement.h',
                 'public/WebMediaHints.h',
                 'public/WebMediaPlayer.h',
@@ -609 +608 @@
                 'src/WebLayerTreeViewImpl.cpp',
                 'src/WebLayerTreeViewImpl.h',
-                'src/WebMediaElement.cpp',
                 'src/WebMediaHints.cpp',
                 'src/WebMediaPlayerClientImpl.cpp',

trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp

@@ -26 +26 @@
 #include "WebFrameImpl.h"
 #include "WebKit.h"
-#include "WebMediaElement.h"
 #include "WebMediaPlayer.h"
 #include "WebViewImpl.h"
@@ -88 +87 @@
 }
 
-WebMediaPlayerClientImpl* WebMediaPlayerClientImpl::fromMediaElement(const WebMediaElement* element)
-{
-    PlatformMedia pm = element->constUnwrap<HTMLMediaElement>()->platformMedia();
-    return static_cast<WebMediaPlayerClientImpl*>(pm.media.chromiumMediaPlayer);
-}
-
 WebMediaPlayer* WebMediaPlayerClientImpl::mediaPlayer() const
 {

trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h

@@ -50 +50 @@
 
 class WebAudioSourceProvider;
-class WebMediaElement;
 class WebMediaPlayer;
 
@@ -66 +65 @@
     static void setIsEnabled(bool);
     static void registerSelf(WebCore::MediaEngineRegistrar);
-
-    static WebMediaPlayerClientImpl* fromMediaElement(const WebMediaElement* element);
 
     // Returns the encapsulated WebKit::WebMediaPlayer.