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

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

Issue 1407703003: Reland: Use GpuMemoryBufferVideoFramePool for WebMediaPlayerMS and MediaStreamVideoRendererSink (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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_ms.h
diff --git a/content/renderer/media/webmediaplayer_ms.h b/content/renderer/media/webmediaplayer_ms.h
index 15cd854d4db454bb97a2bd78d584b18a6a22e173..ba0a48810f31d36486f86516cf5f27a1764a4e2a 100644
--- a/content/renderer/media/webmediaplayer_ms.h
+++ b/content/renderer/media/webmediaplayer_ms.h
@@ -14,6 +14,7 @@
#include "media/blink/skcanvas_video_renderer.h"
#include "media/blink/webmediaplayer_util.h"
#include "media/filters/video_renderer_algorithm.h"
+#include "media/renderers/gpu_video_accelerator_factories.h"
#include "skia/ext/platform_canvas.h"
#include "third_party/WebKit/public/platform/WebMediaPlayer.h"
#include "url/gurl.h"
@@ -58,13 +59,16 @@ class WebMediaPlayerMS
public:
// Construct a WebMediaPlayerMS with reference to the client, and
// a MediaStreamClient which provides VideoFrameProvider.
- WebMediaPlayerMS(blink::WebFrame* frame,
- blink::WebMediaPlayerClient* client,
- base::WeakPtr<media::WebMediaPlayerDelegate> delegate,
- media::MediaLog* media_log,
- scoped_ptr<MediaStreamRendererFactory> factory,
- const scoped_refptr<base::SingleThreadTaskRunner>&
- compositor_task_runner);
+ WebMediaPlayerMS(
+ blink::WebFrame* frame,
+ blink::WebMediaPlayerClient* client,
+ base::WeakPtr<media::WebMediaPlayerDelegate> delegate,
+ media::MediaLog* media_log,
+ scoped_ptr<MediaStreamRendererFactory> factory,
+ const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner,
+ const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
+ const scoped_refptr<base::TaskRunner>& worker_task_runner,
+ const scoped_refptr<media::GpuVideoAcceleratorFactories>& gpu_factories);
~WebMediaPlayerMS() override;
@@ -268,6 +272,9 @@ class WebMediaPlayerMS
scoped_ptr<MediaStreamRendererFactory> renderer_factory_;
+ const scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;
+ const scoped_refptr<base::TaskRunner> worker_task_runner_;
+ const scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_;
// Used for DCHECKs to ensure methods calls executed in the correct thread.
base::ThreadChecker thread_checker_;
« no previous file with comments | « content/renderer/media/media_stream_video_renderer_sink_unittest.cc ('k') | content/renderer/media/webmediaplayer_ms.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698