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

Unified Diff: content/common/gpu/client/gpu_video_decode_accelerator_host.cc

Issue 1750213002: Fix Android black frames from MSE config changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix cros build Created 4 years, 9 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: content/common/gpu/client/gpu_video_decode_accelerator_host.cc
diff --git a/content/common/gpu/client/gpu_video_decode_accelerator_host.cc b/content/common/gpu/client/gpu_video_decode_accelerator_host.cc
index 1f5ceddb550075385e8e84417dcf60b533d59ed0..0add8df0dd890eeb68eada35135ea828a1885151 100644
--- a/content/common/gpu/client/gpu_video_decode_accelerator_host.cc
+++ b/content/common/gpu/client/gpu_video_decode_accelerator_host.cc
@@ -244,12 +244,13 @@ void GpuVideoDecodeAcceleratorHost::OnPictureReady(
int32_t picture_buffer_id,
int32_t bitstream_buffer_id,
const gfx::Rect& visible_rect,
- bool allow_overlay) {
+ bool allow_overlay,
+ bool size_changed) {
liberato (no reviews please) 2016/03/05 08:51:00 another option is to start sending 'int32_t flags'
chcunningham 2016/03/08 03:19:36 I like this idea. This will involve changing a lot
chcunningham 2016/03/08 22:19:12 Decided not to make it int32. I started to feel ba
DCHECK(CalledOnValidThread());
if (!client_)
return;
media::Picture picture(picture_buffer_id, bitstream_buffer_id, visible_rect,
- allow_overlay);
+ allow_overlay, size_changed);
client_->PictureReady(picture);
}

Powered by Google App Engine
This is Rietveld 408576698