Cleanup dead MediaSource code now that the Chromium-side refactor is complete.
authoracolwell@chromium.org <acolwell@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Mar 2013 15:37:57 +0000 (15:37 +0000)
committeracolwell@chromium.org <acolwell@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Mar 2013 15:37:57 +0000 (15:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110371

Reviewed by Adam Barth.

* public/WebMediaPlayer.h:
(WebKit):
(WebMediaPlayer): Remove old MediaSource methods and default MediaSource load() implementation.
* public/WebMediaPlayerClient.h: Remove old MediaSource methods.
* src/WebMediaPlayerClientImpl.cpp: Remove WebMediaSourceClientImpl & WebSourceBufferImpl classes that
                                    are no longer needed.
* src/WebMediaPlayerClientImpl.h: Remove old MediaSource methods.
(WebMediaPlayerClientImpl):

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

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebMediaPlayer.h
Source/WebKit/chromium/public/WebMediaPlayerClient.h
Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp
Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h

index dfaff21..655c8e5 100644 (file)
@@ -1,3 +1,19 @@
+2013-03-13  Aaron Colwell  <acolwell@chromium.org>
+
+        Cleanup dead MediaSource code now that the Chromium-side refactor is complete.
+        https://bugs.webkit.org/show_bug.cgi?id=110371
+
+        Reviewed by Adam Barth.
+
+        * public/WebMediaPlayer.h:
+        (WebKit):
+        (WebMediaPlayer): Remove old MediaSource methods and default MediaSource load() implementation.
+        * public/WebMediaPlayerClient.h: Remove old MediaSource methods.
+        * src/WebMediaPlayerClientImpl.cpp: Remove WebMediaSourceClientImpl & WebSourceBufferImpl classes that
+                                            are no longer needed.
+        * src/WebMediaPlayerClientImpl.h: Remove old MediaSource methods.
+        (WebMediaPlayerClientImpl):
+
 2013-03-13  Pavel Feldman  <pfeldman@chromium.org>
 
         Web Inspector: get rid of hiddenPanels filter
index 6c41019..ad56c1b 100644 (file)
@@ -34,7 +34,6 @@
 #include "../../../Platform/chromium/public/WebCanvas.h"
 #include "../../../Platform/chromium/public/WebString.h"
 #include "../../../Platform/chromium/public/WebVideoFrame.h"
-#include "WebMediaSource.h"
 #include "WebTimeRange.h"
 
 namespace WebKit {
@@ -42,6 +41,7 @@ namespace WebKit {
 class WebAudioSourceProvider;
 class WebAudioSourceProviderClient;
 class WebMediaPlayerClient;
+class WebMediaSource;
 class WebStreamTextureClient;
 class WebString;
 class WebURL;
@@ -82,18 +82,6 @@ public:
         PreloadAuto,
     };
 
-    enum AddIdStatus {
-        AddIdStatusOk,
-        AddIdStatusNotSupported,
-        AddIdStatusReachedIdLimit
-    };
-
-    enum EndOfStreamStatus {
-        EndOfStreamStatusNoError,
-        EndOfStreamStatusNetworkError,
-        EndOfStreamStatusDecodeError,
-    };
-
     // Represents synchronous exceptions that can be thrown from the Encrypted
     // Media methods. This is different from the asynchronous MediaKeyError.
     enum MediaKeyException {
@@ -111,9 +99,7 @@ public:
     virtual ~WebMediaPlayer() {}
 
     virtual void load(const WebURL&, CORSMode) = 0;
-    // FIXME: Remove the default implementation once the Chromium code implements this method.
-    // https://bugs.webkit.org/show_bug.cgi?id=110371
-    virtual void load(const WebURL& url, WebMediaSource* mediaSource, CORSMode corsMode) { delete mediaSource; load(url, corsMode); };
+    virtual void load(const WebURL&, WebMediaSource*, CORSMode) = 0;
     virtual void cancelLoad() = 0;
 
     // Playback controls.
@@ -188,16 +174,6 @@ public:
 
     virtual WebAudioSourceProvider* audioSourceProvider() { return 0; }
 
-    virtual AddIdStatus sourceAddId(const WebString& id, const WebString& type,
-                                    const WebVector<WebString>& codecs) { return AddIdStatusNotSupported; }
-    virtual bool sourceRemoveId(const WebString& id) { return false; }
-    virtual WebTimeRanges sourceBuffered(const WebString& id) { return WebTimeRanges(); };
-    virtual bool sourceAppend(const WebString& id, const unsigned char* data, unsigned length) { return false; }
-    virtual bool sourceAbort(const WebString& id) { return false; }
-    virtual void sourceSetDuration(double) { }
-    virtual void sourceEndOfStream(EndOfStreamStatus)  { }
-    virtual bool sourceSetTimestampOffset(const WebString& id, double offset) { return false; }
-
     // Returns whether keySystem is supported. If true, the result will be
     // reported by an event.
     virtual MediaKeyException generateKeyRequest(const WebString& keySystem, const unsigned char* initData, unsigned initDataLength) { return MediaKeyExceptionKeySystemNotSupported; }
index c82fd3a..089937c 100644 (file)
@@ -72,8 +72,6 @@ public:
     virtual float volume() const = 0;
     virtual void playbackStateChanged() = 0;
     virtual WebMediaPlayer::Preload preload() const = 0;
-    virtual void sourceOpened() = 0;
-    virtual WebKit::WebURL sourceURL() const = 0;
     virtual void keyAdded(const WebString& keySystem, const WebString& sessionId) = 0;
     virtual void keyError(const WebString& keySystem, const WebString& sessionId, MediaKeyErrorCode, unsigned short systemCode) = 0;
     virtual void keyMessage(const WebString& keySystem, const WebString& sessionId, const unsigned char* message, unsigned messageLength, const WebURL& defaultURL) = 0;
index 75cb0f2..aab74e2 100644 (file)
 #include "PlatformContextSkia.h"
 #include "RenderView.h"
 #include "TimeRanges.h"
-#include "UUID.h"
 #include "WebAudioSourceProvider.h"
 #include "WebDocument.h"
 #include "WebFrameClient.h"
 #include "WebFrameImpl.h"
 #include "WebHelperPluginImpl.h"
 #include "WebMediaPlayer.h"
-#include "WebMediaSourceClient.h"
 #include "WebMediaSourceImpl.h"
-#include "WebSourceBuffer.h"
 #include "WebViewImpl.h"
 #include <public/Platform.h>
 #include <public/WebCString.h>
@@ -55,113 +52,6 @@ using namespace WebCore;
 
 namespace WebKit {
 
-// FIXME: Remove this class once the Chromium code implements its own
-// version of WebMediaPlayer::load(WebURL, WebMediaSource, CORSMode).
-// https://bugs.webkit.org/show_bug.cgi?id=110371
-class WebMediaSourceClientImpl : public WebMediaSourceClient {
-public:
-    explicit WebMediaSourceClientImpl(WebMediaPlayer*);
-    virtual ~WebMediaSourceClientImpl();
-
-    // WebMediaSourceClient methods.
-    virtual AddStatus addSourceBuffer(const WebString& type, const WebVector<WebString>& codecs, WebSourceBuffer**);
-    virtual double duration();
-    virtual void setDuration(double);
-    virtual void endOfStream(EndOfStreamStatus);
-
-private:
-    WebMediaPlayer* m_webMediaPlayer;
-};
-
-// FIXME: Remove this class once the Chromium code implements its own
-// version of WebMediaPlayer::load(WebURL, WebMediaSource, CORSMode).
-// https://bugs.webkit.org/show_bug.cgi?id=110371
-class WebSourceBufferImpl : public WebSourceBuffer {
-public:
-    WebSourceBufferImpl(WebMediaPlayer*, const WebString& id);
-
-    // WebSourceBuffer methods.
-    virtual WebTimeRanges buffered();
-    virtual void append(const unsigned char* data, unsigned length);
-    virtual void abort();
-    virtual bool setTimestampOffset(double);
-    virtual void removedFromMediaSource();
-
-private:
-    WebMediaPlayer* m_webMediaPlayer;
-    WebString m_id;
-};
-
-WebMediaSourceClientImpl::WebMediaSourceClientImpl(WebMediaPlayer* webMediaPlayer)
-    : m_webMediaPlayer(webMediaPlayer)
-{
-}
-
-WebMediaSourceClientImpl::~WebMediaSourceClientImpl()
-{
-}
-
-WebMediaSourceClient::AddStatus WebMediaSourceClientImpl::addSourceBuffer(const WebString& type, const WebVector<WebString>& codecs,
-    WebSourceBuffer** sourceBuffer)
-{
-    WebString id = WebCore::createCanonicalUUIDString();
-    WebMediaSourceClient::AddStatus result = static_cast<WebMediaSourceClient::AddStatus>(m_webMediaPlayer->sourceAddId(id, type, codecs));
-    if (result == WebMediaSourceClient::AddStatusOk)
-        *sourceBuffer = new WebSourceBufferImpl(m_webMediaPlayer, id);
-
-    return result;
-}
-
-double WebMediaSourceClientImpl::duration()
-{
-    return m_webMediaPlayer->duration();
-}
-
-void WebMediaSourceClientImpl::setDuration(double duration)
-{
-    return m_webMediaPlayer->sourceSetDuration(duration);
-}
-
-void WebMediaSourceClientImpl::endOfStream(EndOfStreamStatus status)
-{
-    m_webMediaPlayer->sourceEndOfStream(static_cast<WebMediaPlayer::EndOfStreamStatus>(status));
-}
-
-WebSourceBufferImpl::WebSourceBufferImpl(WebMediaPlayer* webMediaPlayer, const WebString& id)
-    : m_webMediaPlayer(webMediaPlayer)
-    , m_id(id)
-{
-}
-
-WebTimeRanges WebSourceBufferImpl::buffered()
-{
-    return m_webMediaPlayer->sourceBuffered(m_id);
-}
-
-void WebSourceBufferImpl::append(const unsigned char* data, unsigned length)
-{
-    m_webMediaPlayer->sourceAppend(m_id, data, length);
-}
-
-void WebSourceBufferImpl::abort()
-{
-    m_webMediaPlayer->sourceAbort(m_id);
-}
-
-bool WebSourceBufferImpl::setTimestampOffset(double offset)
-{
-    return m_webMediaPlayer->sourceSetTimestampOffset(m_id, offset);
-}
-
-void WebSourceBufferImpl::removedFromMediaSource()
-{
-    if (!m_webMediaPlayer)
-        return;
-
-    m_webMediaPlayer->sourceRemoveId(m_id);
-    m_webMediaPlayer = 0;
-}
-
 static PassOwnPtr<WebMediaPlayer> createWebMediaPlayer(WebMediaPlayerClient* client, const WebURL& url, Frame* frame)
 {
     WebFrameImpl* webFrame = WebFrameImpl::fromFrame(frame);
@@ -331,29 +221,6 @@ WebMediaPlayer::Preload WebMediaPlayerClientImpl::preload() const
     return static_cast<WebMediaPlayer::Preload>(m_preload);
 }
 
-// FIXME: Remove this and sourceURL() once the Chromium code implements
-// its own version of WebMediaPlayer::load(WebURL, WebMediaSource, CORSMode).
-// https://bugs.webkit.org/show_bug.cgi?id=110371
-void WebMediaPlayerClientImpl::sourceOpened()
-{
-#if ENABLE(MEDIA_SOURCE)
-    ASSERT(m_webMediaPlayer);
-    if (m_mediaSource) {
-        OwnPtr<WebMediaSource> mediaSource = adoptPtr(new WebMediaSourceImpl(m_mediaSource));
-        mediaSource->open(new WebMediaSourceClientImpl(m_webMediaPlayer.get()));
-    }
-#endif
-}
-
-WebKit::WebURL WebMediaPlayerClientImpl::sourceURL() const
-{
-#if ENABLE(MEDIA_SOURCE)
-    if (m_mediaSource)
-        return m_url;
-#endif
-    return KURL();
-}
-
 void WebMediaPlayerClientImpl::keyAdded(const WebString& keySystem, const WebString& sessionId)
 {
 #if ENABLE(ENCRYPTED_MEDIA)
index 578f8e4..82d8f27 100644 (file)
@@ -84,8 +84,6 @@ public:
     virtual float volume() const;
     virtual void playbackStateChanged();
     virtual WebMediaPlayer::Preload preload() const;
-    virtual void sourceOpened();
-    virtual WebKit::WebURL sourceURL() const;
     virtual void keyAdded(const WebString& keySystem, const WebString& sessionId);
     virtual void keyError(const WebString& keySystem, const WebString& sessionId, MediaKeyErrorCode, unsigned short systemCode);
     virtual void keyMessage(const WebString& keySystem, const WebString& sessionId, const unsigned char* message, unsigned messageLength, const WebURL& defaultURL);