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

Unified Diff: media/blink/webmediaplayer_params.h

Issue 783003002: Introduce media::RendererFactory interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments addressed Created 6 years 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
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | media/blink/webmediaplayer_params.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_params.h
diff --git a/media/blink/webmediaplayer_params.h b/media/blink/webmediaplayer_params.h
index 3b7d34321279d8ce0261708ed92c308cfd6c0cfa..121ee1447a168f9f579a729ea546d87c0fb81661 100644
--- a/media/blink/webmediaplayer_params.h
+++ b/media/blink/webmediaplayer_params.h
@@ -20,9 +20,7 @@ class WebMediaPlayerClient;
namespace media {
-class AudioHardwareConfig;
class AudioRendererSink;
-class GpuVideoAcceleratorFactories;
class MediaLog;
// Holds parameters for constructing WebMediaPlayerImpl without having
@@ -36,9 +34,7 @@ class MEDIA_EXPORT WebMediaPlayerParams {
WebMediaPlayerParams(
const DeferLoadCB& defer_load_cb,
const scoped_refptr<AudioRendererSink>& audio_renderer_sink,
- const AudioHardwareConfig& audio_hardware_config,
const scoped_refptr<MediaLog>& media_log,
- const scoped_refptr<GpuVideoAcceleratorFactories>& gpu_factories,
const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner,
blink::WebContentDecryptionModule* initial_cdm);
@@ -53,26 +49,16 @@ class MEDIA_EXPORT WebMediaPlayerParams {
return audio_renderer_sink_;
}
- const AudioHardwareConfig& audio_hardware_config() const {
- return audio_hardware_config_;
- }
-
const scoped_refptr<MediaLog>& media_log() const {
return media_log_;
}
- const scoped_refptr<GpuVideoAcceleratorFactories>&
- gpu_factories() const {
- return gpu_factories_;
- }
-
- const scoped_refptr<base::SingleThreadTaskRunner>&
- media_task_runner() const {
+ const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner() const {
return media_task_runner_;
}
- const scoped_refptr<base::SingleThreadTaskRunner>&
- compositor_task_runner() const {
+ const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner()
+ const {
return compositor_task_runner_;
}
@@ -83,9 +69,7 @@ class MEDIA_EXPORT WebMediaPlayerParams {
private:
base::Callback<void(const base::Closure&)> defer_load_cb_;
scoped_refptr<AudioRendererSink> audio_renderer_sink_;
- const AudioHardwareConfig& audio_hardware_config_;
scoped_refptr<MediaLog> media_log_;
- scoped_refptr<GpuVideoAcceleratorFactories> gpu_factories_;
scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;
scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner_;
blink::WebContentDecryptionModule* initial_cdm_;
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | media/blink/webmediaplayer_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698