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

Unified Diff: media/filters/video_renderer_base.cc

Issue 155230: Made MediaFilter::host_ and MediaFilter::message_loop_ private. (Closed)
Patch Set: Merge with ToT Created 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/filters/file_data_source_unittest.cc ('k') | webkit/glue/media/simple_data_source.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/video_renderer_base.cc
diff --git a/media/filters/video_renderer_base.cc b/media/filters/video_renderer_base.cc
index 094c92aa447fedcf6ad505ef5d2a0c3802b242a2..7eace6f71833779124a17f07eca6a63812569772 100644
--- a/media/filters/video_renderer_base.cc
+++ b/media/filters/video_renderer_base.cc
@@ -104,7 +104,7 @@ bool VideoRendererBase::Initialize(VideoDecoder* decoder) {
int height = 0;
if (!ParseMediaFormat(decoder->media_format(), &width, &height))
return false;
- host_->SetVideoSize(width, height);
+ host()->SetVideoSize(width, height);
// Initialize the subclass.
// TODO(scherkus): do we trust subclasses not to do something silly while
@@ -188,7 +188,7 @@ void VideoRendererBase::ThreadMain() {
OnFrameAvailable();
// Determine the current and next presentation timestamps.
- base::TimeDelta now = host_->GetTime();
+ base::TimeDelta now = host()->GetTime();
base::TimeDelta this_pts = current_frame_->GetTimestamp();
base::TimeDelta next_pts;
if (next_frame) {
@@ -248,11 +248,11 @@ void VideoRendererBase::OnReadComplete(VideoFrame* frame) {
if (frames_.empty()) {
// We should have initialized but there's no decoded frames in the queue.
// Raise an error.
- host_->Error(PIPELINE_ERROR_NO_DATA);
+ host()->Error(PIPELINE_ERROR_NO_DATA);
} else {
state_ = INITIALIZED;
current_frame_ = frames_.front();
- host_->InitializationComplete();
+ host()->InitializationComplete();
}
}
}
« no previous file with comments | « media/filters/file_data_source_unittest.cc ('k') | webkit/glue/media/simple_data_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698