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

Unified Diff: media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc

Issue 2881553002: Add |texture_target_| and |pixel_format_| to media::PictureBuffer (Closed)
Patch Set: addressed review comments Created 3 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/gpu/ipc/client/gpu_video_decode_accelerator_host.cc
diff --git a/media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc b/media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc
index 6820ea1db7964ad7094d665da4313859733e3d7a..36daa91d84262f41b9e2b898fc786d30df1542f9 100644
--- a/media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc
+++ b/media/gpu/ipc/client/gpu_video_decode_accelerator_host.cc
@@ -130,6 +130,9 @@ void GpuVideoDecodeAcceleratorHost::AssignPictureBuffers(
// Rearrange data for IPC command.
std::vector<int32_t> buffer_ids;
std::vector<PictureBuffer::TextureIds> texture_ids;
+ std::vector<uint32_t> texture_targets;
+ std::vector<VideoPixelFormat> formats;
+ std::vector<gfx::Size> sizes;
for (uint32_t i = 0; i < buffers.size(); i++) {
const PictureBuffer& buffer = buffers[i];
if (buffer.size() != picture_buffer_dimensions_) {
@@ -141,9 +144,13 @@ void GpuVideoDecodeAcceleratorHost::AssignPictureBuffers(
}
texture_ids.push_back(buffer.client_texture_ids());
buffer_ids.push_back(buffer.id());
+ texture_targets.push_back(buffer.texture_target());
+ formats.push_back(buffer.pixel_format());
+ sizes.push_back(buffer.size());
}
Send(new AcceleratedVideoDecoderMsg_AssignPictureBuffers(
- decoder_route_id_, buffer_ids, texture_ids));
+ decoder_route_id_, buffer_ids, texture_ids, texture_targets, formats,
+ sizes));
}
void GpuVideoDecodeAcceleratorHost::ReusePictureBuffer(

Powered by Google App Engine
This is Rietveld 408576698