Remove unused methods from MediaPlayerClient
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Sep 2014 11:16:23 +0000 (11:16 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Sep 2014 11:16:23 +0000 (11:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=136884

Reviewed by Eric Carlson.

Remove unused methods from MediaPlayerClient that introduce
dependencies on WebCore types.

* html/HTMLMediaElement.cpp:
* html/HTMLMediaElement.h:
* platform/graphics/MediaPlayer.h:
(WebCore::MediaPlayerClient::mediaPlayerOwningDocument): Deleted.
(WebCore::MediaPlayerClient::mediaPlayerHostWindow): Deleted.
(WebCore::MediaPlayerClient::mediaPlayerWindowClipRect): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@173719 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLMediaElement.cpp
Source/WebCore/html/HTMLMediaElement.h
Source/WebCore/platform/graphics/MediaPlayer.h

index 5526b81..8dc30fc 100644 (file)
@@ -1,3 +1,20 @@
+2014-09-18  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        Remove unused methods from MediaPlayerClient
+        https://bugs.webkit.org/show_bug.cgi?id=136884
+
+        Reviewed by Eric Carlson.
+
+        Remove unused methods from MediaPlayerClient that introduce
+        dependencies on WebCore types.
+
+        * html/HTMLMediaElement.cpp:
+        * html/HTMLMediaElement.h:
+        * platform/graphics/MediaPlayer.h:
+        (WebCore::MediaPlayerClient::mediaPlayerOwningDocument): Deleted.
+        (WebCore::MediaPlayerClient::mediaPlayerHostWindow): Deleted.
+        (WebCore::MediaPlayerClient::mediaPlayerWindowClipRect): Deleted.
+
 2014-09-17  Iago Toral  <itoral@igalia.com> and Zan Dobersek  <zdobersek@igalia.com>
 
         [GTK] Add Wayland support to GLContextEGL
 2014-09-17  Iago Toral  <itoral@igalia.com> and Zan Dobersek  <zdobersek@igalia.com>
 
         [GTK] Add Wayland support to GLContextEGL
index 82f5dda..32c9ef1 100644 (file)
@@ -1812,11 +1812,6 @@ void HTMLMediaElement::cancelPendingEventsAndCallbacks()
         source.cancelPendingErrorEvent();
 }
 
         source.cancelPendingErrorEvent();
 }
 
-Document* HTMLMediaElement::mediaPlayerOwningDocument()
-{
-    return &document();
-}
-
 void HTMLMediaElement::mediaPlayerNetworkStateChanged(MediaPlayer*)
 {
     beginProcessingMediaPlayerCallback();
 void HTMLMediaElement::mediaPlayerNetworkStateChanged(MediaPlayer*)
 {
     beginProcessingMediaPlayerCallback();
@@ -5663,19 +5658,9 @@ bool HTMLMediaElement::mediaPlayerIsLooping() const
     return loop();
 }
 
     return loop();
 }
 
-HostWindow* HTMLMediaElement::mediaPlayerHostWindow()
-{
-    return mediaPlayerOwningDocument()->view()->hostWindow();
-}
-
-IntRect HTMLMediaElement::mediaPlayerWindowClipRect()
-{
-    return mediaPlayerOwningDocument()->view()->windowClipRect();
-}
-
 CachedResourceLoader* HTMLMediaElement::mediaPlayerCachedResourceLoader()
 {
 CachedResourceLoader* HTMLMediaElement::mediaPlayerCachedResourceLoader()
 {
-    return mediaPlayerOwningDocument()->cachedResourceLoader();
+    return document().cachedResourceLoader();
 }
 
 bool HTMLMediaElement::mediaPlayerShouldWaitForResponseToAuthenticationChallenge(const AuthenticationChallenge& challenge)
 }
 
 bool HTMLMediaElement::mediaPlayerShouldWaitForResponseToAuthenticationChallenge(const AuthenticationChallenge& challenge)
index 4834814..d867f75 100644 (file)
@@ -514,7 +514,6 @@ private:
 
     double effectivePlaybackRate() const;
 
 
     double effectivePlaybackRate() const;
 
-    virtual Document* mediaPlayerOwningDocument() override;
     virtual void mediaPlayerNetworkStateChanged(MediaPlayer*) override;
     virtual void mediaPlayerReadyStateChanged(MediaPlayer*) override;
     virtual void mediaPlayerTimeChanged(MediaPlayer*) override;
     virtual void mediaPlayerNetworkStateChanged(MediaPlayer*) override;
     virtual void mediaPlayerReadyStateChanged(MediaPlayer*) override;
     virtual void mediaPlayerTimeChanged(MediaPlayer*) override;
@@ -570,8 +569,6 @@ private:
     virtual bool mediaPlayerPlatformVolumeConfigurationRequired() const override;
     virtual bool mediaPlayerIsPaused() const override;
     virtual bool mediaPlayerIsLooping() const override;
     virtual bool mediaPlayerPlatformVolumeConfigurationRequired() const override;
     virtual bool mediaPlayerIsPaused() const override;
     virtual bool mediaPlayerIsLooping() const override;
-    virtual HostWindow* mediaPlayerHostWindow() override;
-    virtual IntRect mediaPlayerWindowClipRect() override;
     virtual CachedResourceLoader* mediaPlayerCachedResourceLoader() override;
 
 #if PLATFORM(WIN) && USE(AVFOUNDATION)
     virtual CachedResourceLoader* mediaPlayerCachedResourceLoader() override;
 
 #if PLATFORM(WIN) && USE(AVFOUNDATION)
index 5ce7514..2939917 100644 (file)
@@ -70,7 +70,6 @@ namespace WebCore {
 
 class AudioSourceProvider;
 class AuthenticationChallenge;
 
 class AudioSourceProvider;
 class AuthenticationChallenge;
-class Document;
 #if ENABLE(MEDIA_SOURCE)
 class MediaSourcePrivateClient;
 #endif
 #if ENABLE(MEDIA_SOURCE)
 class MediaSourcePrivateClient;
 #endif
@@ -133,7 +132,6 @@ class ContentType;
 class FrameView;
 class GraphicsContext;
 class GraphicsContext3D;
 class FrameView;
 class GraphicsContext;
 class GraphicsContext3D;
-class HostWindow;
 class IntRect;
 class IntSize;
 class MediaPlayer;
 class IntRect;
 class IntSize;
 class MediaPlayer;
@@ -150,9 +148,6 @@ public:
 
     virtual ~MediaPlayerClient() { }
 
 
     virtual ~MediaPlayerClient() { }
 
-    // Get the document which the media player is owned by
-    virtual Document* mediaPlayerOwningDocument() { return 0; }
-
     // the network state has changed
     virtual void mediaPlayerNetworkStateChanged(MediaPlayer*) { }
 
     // the network state has changed
     virtual void mediaPlayerNetworkStateChanged(MediaPlayer*) { }
 
@@ -244,8 +239,6 @@ public:
     virtual bool mediaPlayerPlatformVolumeConfigurationRequired() const { return false; }
     virtual bool mediaPlayerIsPaused() const { return true; }
     virtual bool mediaPlayerIsLooping() const { return false; }
     virtual bool mediaPlayerPlatformVolumeConfigurationRequired() const { return false; }
     virtual bool mediaPlayerIsPaused() const { return true; }
     virtual bool mediaPlayerIsLooping() const { return false; }
-    virtual HostWindow* mediaPlayerHostWindow() { return 0; }
-    virtual IntRect mediaPlayerWindowClipRect() { return IntRect(); }
     virtual CachedResourceLoader* mediaPlayerCachedResourceLoader() { return 0; }
     virtual bool doesHaveAttribute(const AtomicString&, AtomicString* = 0) const { return false; }
 
     virtual CachedResourceLoader* mediaPlayerCachedResourceLoader() { return 0; }
     virtual bool doesHaveAttribute(const AtomicString&, AtomicString* = 0) const { return false; }