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

Unified Diff: content/common/gpu/media/gpu_video_decode_accelerator.cc

Issue 1154053002: gpu: Use a rectangle to keep track of the cleared area of each texture level. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: restore scissor state in GLES2DecoderImpl::ClearLevel and update GLES2DecoderManualInitTest.DrawCle… Created 5 years, 6 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/media/gpu_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/gpu_video_decode_accelerator.cc b/content/common/gpu/media/gpu_video_decode_accelerator.cc
index c4d7eb9f88639e70743658ab57f96a7ca47c1734..99a782f54c48f52572501ed188d78fba34a0b788 100644
--- a/content/common/gpu/media/gpu_video_decode_accelerator.cc
+++ b/content/common/gpu/media/gpu_video_decode_accelerator.cc
@@ -479,17 +479,9 @@ void GpuVideoDecodeAccelerator::OnAssignPictureBuffers(
texture_target_ == GL_TEXTURE_RECTANGLE_ARB) {
// These textures have their dimensions defined by the underlying storage.
// Use |texture_dimensions_| for this size.
- texture_manager->SetLevelInfo(texture_ref,
- texture_target_,
- 0,
- GL_RGBA,
- texture_dimensions_.width(),
- texture_dimensions_.height(),
- 1,
- 0,
- GL_RGBA,
- 0,
- false);
+ texture_manager->SetLevelInfo(
+ texture_ref, texture_target_, 0, GL_RGBA, texture_dimensions_.width(),
+ texture_dimensions_.height(), 1, 0, GL_RGBA, 0, gfx::Rect());
} else {
// For other targets, texture dimensions should already be defined.
GLsizei width = 0, height = 0;
@@ -506,7 +498,8 @@ void GpuVideoDecodeAccelerator::OnAssignPictureBuffers(
video_decode_accelerator_.get()->GetSurfaceInternalFormat();
if (format != GL_RGBA) {
texture_manager->SetLevelInfo(texture_ref, texture_target_, 0, format,
- width, height, 1, 0, format, 0, false);
+ width, height, 1, 0, format, 0,
+ gfx::Rect());
}
}
buffers.push_back(media::PictureBuffer(buffer_ids[i], texture_dimensions_,
« no previous file with comments | « no previous file | content/common/gpu/stream_texture_android.cc » ('j') | gpu/command_buffer/service/gles2_cmd_decoder.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698