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

Unified Diff: media/filters/video_renderer_base.cc

Issue 7044008: Initial implementation of stream switching in AdaptiveDemuxer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: _ Created 9 years, 7 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: media/filters/video_renderer_base.cc
diff --git a/media/filters/video_renderer_base.cc b/media/filters/video_renderer_base.cc
index 770594639b8de2748b1f12640615b1a636b6c598..f5ee0399f169916d6a3773b5e4a9b98f16d87116 100644
--- a/media/filters/video_renderer_base.cc
+++ b/media/filters/video_renderer_base.cc
@@ -315,6 +315,14 @@ void VideoRendererBase::ThreadMain() {
timeout_frame = current_frame_;
current_frame_ = frames_queue_ready_.front();
frames_queue_ready_.pop_front();
+
+ if (current_frame_->width() != width_ ||
+ current_frame_->height() != height_) {
Ami GONE FROM CHROMIUM 2011/05/19 20:27:37 I don't imagine we have any test coverage for the
acolwell GONE FROM CHROMIUM 2011/05/20 01:26:37 Removing the frame size changes for now. It was a
+ width_ = current_frame_->width();
+ height_ = current_frame_->height();
+ host()->SetVideoSize(width_, height_);
+ }
+
new_frame_available = true;
} else if (pending_paint_ && frames_queue_ready_.size() >= 2 &&
!frames_queue_ready_[1]->IsEndOfStream()) {

Powered by Google App Engine
This is Rietveld 408576698