Chromium Code Reviews| Index: content/browser/compositor/gpu_process_transport_factory.cc |
| diff --git a/content/browser/compositor/gpu_process_transport_factory.cc b/content/browser/compositor/gpu_process_transport_factory.cc |
| index d94cb12fb1c97d0aab0d76b210cf8cde526f8f97..f4101c250edcb00672c398b6543c59bae1e178cc 100644 |
| --- a/content/browser/compositor/gpu_process_transport_factory.cc |
| +++ b/content/browser/compositor/gpu_process_transport_factory.cc |
| @@ -685,10 +685,9 @@ void GpuProcessTransportFactory::ResizeDisplay(ui::Compositor* compositor, |
| void GpuProcessTransportFactory::SetDisplayColorSpace( |
| ui::Compositor* compositor, |
| const gfx::ColorSpace& color_space) { |
|
ccameron
2016/09/02 21:10:32
danakj, could you verify the correctness of this -
hubbe
2016/09/02 21:35:49
Acknowledged.
|
| - PerCompositorDataMap::iterator it = per_compositor_data_.find(compositor); |
| - if (it == per_compositor_data_.end()) |
| - return; |
| - PerCompositorData* data = it->second.get(); |
| + PerCompositorData* data = per_compositor_data_[compositor].get(); |
| + if (!data) |
| + data = CreatePerCompositorData(compositor); |
|
danakj
2016/09/02 21:11:45
I don't think you want to do this. Instead just st
hubbe
2016/09/02 21:35:49
Added it to ui:Compositor.
Does that mean we don't
danakj
2016/09/02 21:36:47
Ya that'd be ideal.
hubbe
2016/09/02 22:30:47
Gone.
|
| DCHECK(data); |
| data->color_space = color_space; |
| if (data->display) |