diff options
author | scherkus@chromium.org <scherkus@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-18 23:52:46 +0000 |
---|---|---|
committer | scherkus@chromium.org <scherkus@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-18 23:52:46 +0000 |
commit | 86b3f4b2779d33edc06a7e72c768868316d34b6f (patch) | |
tree | e68a0be69411bc298e4409cf354be265b2c87604 /webkit/media/webmediaplayer_impl.h | |
parent | b62eab12bcf6e0c3c3d6ccaa91f5ddfd4b0b1ead (diff) | |
download | chromium_src-86b3f4b2779d33edc06a7e72c768868316d34b6f.zip chromium_src-86b3f4b2779d33edc06a7e72c768868316d34b6f.tar.gz chromium_src-86b3f4b2779d33edc06a7e72c768868316d34b6f.tar.bz2 |
Remove --enable-video-logging and WebVideoRenderer.
WebVideoRenderer was added in r40137 as part of my IPC-based video rendering experiment.
I later added --enable-video-logging in r43038 which was intended to log timestamps and do logs analysis to determine playback smoothness. This again was experimental and never used.
Also removed WebMediaPlayerProxy::SetRect() and VideoRendererImpl::SetRect() as they were not implemented.
Review URL: http://codereview.chromium.org/8601004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110786 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/media/webmediaplayer_impl.h')
-rw-r--r-- | webkit/media/webmediaplayer_impl.h | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/webkit/media/webmediaplayer_impl.h b/webkit/media/webmediaplayer_impl.h index e38bd9f..ef6c2ed 100644 --- a/webkit/media/webmediaplayer_impl.h +++ b/webkit/media/webmediaplayer_impl.h @@ -14,7 +14,7 @@ // media::PipelineImpl // The media playback pipeline. // -// WebVideoRenderer +// VideoRendererImpl // Video renderer object. // // WebKit::WebMediaPlayerClient @@ -29,12 +29,12 @@ // WebMediaPlayerImpl ---> PipelineImpl // | ^ | // | | v r -// | | WebVideoRenderer +// | | VideoRendererImpl // | | | ^ r // | r | v r | // '---> WebMediaPlayerProxy --' // -// Notice that WebMediaPlayerProxy and WebVideoRenderer are referencing each +// Notice that WebMediaPlayerProxy and VideoRendererImpl are referencing each // other. This interdependency has to be treated carefully. // // Other issues: @@ -72,7 +72,6 @@ namespace webkit_media { class MediaStreamClient; class WebMediaPlayerDelegate; class WebMediaPlayerProxy; -class WebVideoRenderer; class WebMediaPlayerImpl : public WebKit::WebMediaPlayer, @@ -110,10 +109,7 @@ class WebMediaPlayerImpl virtual ~WebMediaPlayerImpl(); // Finalizes initialization of the object. - bool Initialize( - WebKit::WebFrame* frame, - bool use_simple_data_source, - scoped_refptr<WebVideoRenderer> web_video_renderer); + bool Initialize(WebKit::WebFrame* frame, bool use_simple_data_source); virtual void load(const WebKit::WebURL& url); virtual void cancelLoad(); |