Commit 3cf3eea9 authored by Christopher Cameron's avatar Christopher Cameron Committed by Commit Bot

cc: Add missed GpuMemoryBuffer:SetColorSpace call

This call is necessary to set the color space property on the backing
IOSurface.

It may be that we should change this API to require specifying a color
space a GpuMemoryBuffer creation time. That would be an artifical
constraint, but would make bugs of this sort less likely.

This needs a merge to M66, so I'm not considering it in this patch.

Bug: 827670
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: If5ecdc5a8efcefcf1fcbc9c8a230cd716005ac2e
Reviewed-on: https://chromium-review.googlesource.com/988826Reviewed-by: default avatarvmpstr <vmpstr@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547872}
parent 8cce6adb
...@@ -168,6 +168,7 @@ class ZeroCopyRasterBufferImpl : public RasterBuffer { ...@@ -168,6 +168,7 @@ class ZeroCopyRasterBufferImpl : public RasterBuffer {
// GpuMemoryBuffer allocation can fail (https://crbug.com/554541). // GpuMemoryBuffer allocation can fail (https://crbug.com/554541).
if (!gpu_memory_buffer_) if (!gpu_memory_buffer_)
return; return;
gpu_memory_buffer_->SetColorSpace(resource_color_space_);
} }
DCHECK_EQ(1u, gfx::NumberOfPlanesForBufferFormat( DCHECK_EQ(1u, gfx::NumberOfPlanesForBufferFormat(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment