cc: Fix the format of GpuMemoryBuffer for SurfaceTexture

content/browser/renderer_host/render_process_host_impl.cc

Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index e6df625995260a1880ca0af522c5ceaff11abee8..946faec62ea0e24f1baeb83096e1650be972f021 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -1146,7 +1146,11 @@ static void AppendCompositorCommandLineFlags(base::CommandLine* command_line) {
           // compositor for each usage.
           // crbug.com/490362
           BrowserGpuMemoryBufferManager::GetImageTextureTarget(
+#if defined(OS_ANDROID)
+              gfx::GpuMemoryBuffer::RGBA_8888,
+#else

[reveman, 2015/07/23 07:48:39]

Let's get this fixed properly instead. The problem is that we're trying to
determine what target to use here without knowing the format that the compositor
will use. It's hard to know the format here as it depends on available
extensions.

I think the best solution is to let the compositor know what the target is for
each format. This way the compositor can decide what format it wants to use
without having to worry about the target being wrong.

Let's change the value of switches::kContentImageTextureTarget into a list of
ints. Each int will be the target to use for a supported GMB format. Something
similar to this:

https://code.google.com/p/chromium/codesearch#chromium/src/gpu/config/gpu_uti...

               gfx::GpuMemoryBuffer::BGRA_8888,
+#endif
               // TODO(danakj): When one-copy supports partial update, change
               // this usage to PERSISTENT_MAP for one-copy.
               gfx::GpuMemoryBuffer::MAP)));