Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: webkit/media/webmediaplayer_ms.h

Issue 12902002: Remove WebVideoFrame, WebVideoFrameProvider, and WebVideoLayer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: No provider client lock Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/media/webmediaplayer_ms.h
diff --git a/webkit/media/webmediaplayer_ms.h b/webkit/media/webmediaplayer_ms.h
index 9d2d43e33510e710d136d0e0612e2d3ae55ca6ad..e97a71e1263728c583478b4051672303c0923607 100644
--- a/webkit/media/webmediaplayer_ms.h
+++ b/webkit/media/webmediaplayer_ms.h
@@ -9,6 +9,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
#include "base/synchronization/lock.h"
+#include "cc/layers/video_frame_provider.h"
#include "googleurl/src/gurl.h"
#include "media/filters/skcanvas_video_renderer.h"
#include "skia/ext/platform_canvas.h"
@@ -24,6 +25,10 @@ namespace media {
class MediaLog;
}
+namespace webkit {
+class WebLayerImpl;
+}
+
namespace webkit_media {
class MediaStreamAudioRenderer;
@@ -47,6 +52,7 @@ class WebMediaPlayerDelegate;
// WebKit client of this media player object.
class WebMediaPlayerMS
: public WebKit::WebMediaPlayer,
+ public cc::VideoFrameProvider,
public base::SupportsWeakPtr<WebMediaPlayerMS> {
public:
// Construct a WebMediaPlayerMS with reference to the client, and
@@ -120,8 +126,12 @@ class WebMediaPlayerMS
virtual unsigned audioDecodedByteCount() const OVERRIDE;
virtual unsigned videoDecodedByteCount() const OVERRIDE;
- virtual WebKit::WebVideoFrame* getCurrentFrame() OVERRIDE;
- virtual void putCurrentFrame(WebKit::WebVideoFrame* web_video_frame) OVERRIDE;
+ // VideoFrameProvider implementation.
+ virtual void SetVideoFrameProviderClient(
+ cc::VideoFrameProvider::Client* client) OVERRIDE;
+ virtual scoped_refptr<media::VideoFrame> GetCurrentFrame() OVERRIDE;
+ virtual void PutCurrentFrame(const scoped_refptr<media::VideoFrame>& frame)
+ OVERRIDE;
private:
// The callback for VideoFrameProvider to signal a new frame is available.
@@ -155,8 +165,9 @@ class WebMediaPlayerMS
base::WeakPtr<WebMediaPlayerDelegate> delegate_;
MediaStreamClient* media_stream_client_;
- scoped_refptr<VideoFrameProvider> video_frame_provider_;
+ scoped_refptr<webkit_media::VideoFrameProvider> video_frame_provider_;
bool paused_;
+
// |current_frame_| is updated only on main thread.
scoped_refptr<media::VideoFrame> current_frame_;
// |current_frame_used_| is updated on both main and compositing thread.
@@ -165,6 +176,11 @@ class WebMediaPlayerMS
bool current_frame_used_;
base::Lock current_frame_lock_;
bool pending_repaint_;
+
+ scoped_ptr<webkit::WebLayerImpl> video_weblayer_;
+
+ cc::VideoFrameProvider::Client* video_frame_provider_client_;
+
bool received_first_frame_;
bool sequence_started_;
base::TimeDelta start_time_;

Powered by Google App Engine
This is Rietveld 408576698