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

Unified Diff: content/renderer/media/webmediaplayer_impl.h

Issue 237353007: Refactor VideoRendererImpl to use VideoFrameScheduler. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: pretty much done Created 6 years, 8 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: content/renderer/media/webmediaplayer_impl.h
diff --git a/content/renderer/media/webmediaplayer_impl.h b/content/renderer/media/webmediaplayer_impl.h
index 1c3cb51396cfcbb2d8b6aeb960098239b53bbb94..3c6b6c39793b3f945123314190a4600277061ce8 100644
--- a/content/renderer/media/webmediaplayer_impl.h
+++ b/content/renderer/media/webmediaplayer_impl.h
@@ -45,6 +45,7 @@ namespace media {
class ChunkDemuxer;
class GpuVideoAcceleratorFactories;
class MediaLog;
+class VideoFrameSchedulerImpl;
}
namespace webkit {
@@ -332,8 +333,11 @@ class WebMediaPlayerImpl
std::string init_data_type_;
// Video rendering members.
+ scoped_refptr<base::SingleThreadTaskRunner> compositor_runner_;
VideoFrameCompositor compositor_;
media::SkCanvasVideoRenderer skcanvas_video_renderer_;
+ media::VideoFrameSchedulerImpl*
+ scheduler_; // Deleted on |compositor_runner_|.
xhwang 2014/04/24 18:48:44 nit: I am not a fan of this format. Is this done b
scherkus (not reviewing) 2014/04/25 02:04:47 Done. (it was clang-format)
// The compositor layer for displaying the video content when using composited
// playback.
« no previous file with comments | « no previous file | content/renderer/media/webmediaplayer_impl.cc » ('j') | content/renderer/media/webmediaplayer_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698