Chromium Code Reviews| Index: media/gpu/ipc/service/gpu_video_decode_accelerator.cc |
| diff --git a/media/gpu/ipc/service/gpu_video_decode_accelerator.cc b/media/gpu/ipc/service/gpu_video_decode_accelerator.cc |
| index 9ed062425b8dcd79d368431f901a9f3cea02cb3f..391215da66e82d3a9e150318f3904c0cca7d3844 100644 |
| --- a/media/gpu/ipc/service/gpu_video_decode_accelerator.cc |
| +++ b/media/gpu/ipc/service/gpu_video_decode_accelerator.cc |
| @@ -450,7 +450,7 @@ void GpuVideoDecodeAccelerator::OnAssignPictureBuffers( |
| texture_manager->SetLevelInfo(texture_ref, texture_target_, 0, GL_RGBA, |
| texture_dimensions_.width(), |
| texture_dimensions_.height(), 1, 0, |
| - GL_RGBA, 0, gfx::Rect()); |
| + GL_RGBA, GL_UNSIGNED_BYTE, gfx::Rect()); |
|
qiankun
2017/02/23 09:28:19
I am not sure can this type be GL_FLOAT?
Ken Russell (switch to Gerrit)
2017/03/02 02:38:48
There's ongoing work by aleksandar.stojiljkovic in
|
| } else { |
| // For other targets, texture dimensions should already be defined. |
| GLsizei width = 0, height = 0; |
| @@ -468,8 +468,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, |
| - gfx::Rect()); |
| + width, height, 1, 0, format, |
| + GL_UNSIGNED_BYTE, gfx::Rect()); |
| } |
| } |
| service_ids.push_back(texture_ref->service_id()); |