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

Side by Side Diff: content/renderer/media/webmediaplayer_params.h

Issue 19638008: Refactor GpuVideoDecoder::Factories out of GpuVideoDecoder. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 7 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_
6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_ 6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_
7 7
8 #include "base/callback.h"
8 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
9 #include "media/filters/gpu_video_decoder.h"
10 10
11 namespace base { 11 namespace base {
12 class MessageLoopProxy; 12 class MessageLoopProxy;
13 } 13 }
14 14
15 namespace media { 15 namespace media {
16 class AudioRendererSink; 16 class AudioRendererSink;
17 class GpuVideoDecoderFactories;
17 class MediaLog; 18 class MediaLog;
18 } 19 }
19 20
20 namespace content { 21 namespace content {
21 22
22 // Holds parameters for constructing WebMediaPlayerImpl without having 23 // Holds parameters for constructing WebMediaPlayerImpl without having
23 // to plumb arguments through various abstraction layers. 24 // to plumb arguments through various abstraction layers.
24 class WebMediaPlayerParams { 25 class WebMediaPlayerParams {
25 public: 26 public:
26 // |message_loop_proxy| and |media_log| are the only required parameters; 27 // |message_loop_proxy| and |media_log| are the only required parameters;
27 // all others may be null. 28 // all others may be null.
28 WebMediaPlayerParams( 29 WebMediaPlayerParams(
29 const scoped_refptr<base::MessageLoopProxy>& message_loop_proxy, 30 const scoped_refptr<base::MessageLoopProxy>& message_loop_proxy,
30 const base::Callback<void(const base::Closure&)>& defer_load_cb, 31 const base::Callback<void(const base::Closure&)>& defer_load_cb,
31 const scoped_refptr<media::AudioRendererSink>& audio_renderer_sink, 32 const scoped_refptr<media::AudioRendererSink>& audio_renderer_sink,
32 const scoped_refptr<media::GpuVideoDecoder::Factories>& gpu_factories, 33 const scoped_refptr<media::GpuVideoDecoderFactories>& gpu_factories,
33 const scoped_refptr<media::MediaLog>& media_log); 34 const scoped_refptr<media::MediaLog>& media_log);
34 ~WebMediaPlayerParams(); 35 ~WebMediaPlayerParams();
35 36
36 const scoped_refptr<base::MessageLoopProxy>& message_loop_proxy() const { 37 const scoped_refptr<base::MessageLoopProxy>& message_loop_proxy() const {
37 return message_loop_proxy_; 38 return message_loop_proxy_;
38 } 39 }
39 40
40 base::Callback<void(const base::Closure&)> defer_load_cb() const { 41 base::Callback<void(const base::Closure&)> defer_load_cb() const {
41 return defer_load_cb_; 42 return defer_load_cb_;
42 } 43 }
43 44
44 const scoped_refptr<media::AudioRendererSink>& audio_renderer_sink() const { 45 const scoped_refptr<media::AudioRendererSink>& audio_renderer_sink() const {
45 return audio_renderer_sink_; 46 return audio_renderer_sink_;
46 } 47 }
47 48
48 const scoped_refptr<media::GpuVideoDecoder::Factories>& 49 const scoped_refptr<media::GpuVideoDecoderFactories>& gpu_factories() const {
49 gpu_factories() const {
50 return gpu_factories_; 50 return gpu_factories_;
51 } 51 }
52 52
53 const scoped_refptr<media::MediaLog>& media_log() const { 53 const scoped_refptr<media::MediaLog>& media_log() const {
54 return media_log_; 54 return media_log_;
55 } 55 }
56 56
57 private: 57 private:
58 scoped_refptr<base::MessageLoopProxy> message_loop_proxy_; 58 scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
59 base::Callback<void(const base::Closure&)> defer_load_cb_; 59 base::Callback<void(const base::Closure&)> defer_load_cb_;
60 scoped_refptr<media::AudioRendererSink> audio_renderer_sink_; 60 scoped_refptr<media::AudioRendererSink> audio_renderer_sink_;
61 scoped_refptr<media::GpuVideoDecoder::Factories> gpu_factories_; 61 scoped_refptr<media::GpuVideoDecoderFactories> gpu_factories_;
62 scoped_refptr<media::MediaLog> media_log_; 62 scoped_refptr<media::MediaLog> media_log_;
63 63
64 DISALLOW_IMPLICIT_CONSTRUCTORS(WebMediaPlayerParams); 64 DISALLOW_IMPLICIT_CONSTRUCTORS(WebMediaPlayerParams);
65 }; 65 };
66 66
67 } // namespace media 67 } // namespace media
68 68
69 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_ 69 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_PARAMS_H_
OLDNEW
« no previous file with comments | « content/renderer/media/webmediaplayer_impl.cc ('k') | content/renderer/media/webmediaplayer_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698