gpu: Associate all GpuMemoryBuffers with unique IDs.

content/browser/android/browser_surface_texture_manager.cc

Index: content/browser/android/browser_surface_texture_manager.cc
diff --git a/content/browser/android/browser_surface_texture_manager.cc b/content/browser/android/browser_surface_texture_manager.cc
index ee154f1390f6485115de8b63cec56f8fde4d6d16..cdb11d4eda0daa208deb478eb013f3674e92b11d 100644
--- a/content/browser/android/browser_surface_texture_manager.cc
+++ b/content/browser/android/browser_surface_texture_manager.cc
@@ -95,10 +95,10 @@ void BrowserSurfaceTextureManager::UnregisterSurfaceTexture(
 }
 
 gfx::AcceleratedWidget BrowserSurfaceTextureManager::AcquireNativeWidget(
-    int surface_texture_id,
-    int client_id) {
+    int surface_texture_id) {
+  const int kBrowserClientId = 0;

[piman, 2014/11/04 01:07:26]

Why 0, where does it come from?
Can RegisterSurfaceTexture be called with client_id == 0? From where?

If/when Android uses out-of-process gpu, doesn't this need to be
BrowserGpuChannelHostFactory::GetGpuChannelId() ?

[piman, 2014/11/04 01:22:46]

Double-checked: this should be
BrowserGpuChannelHostFactory::instance()->GetGpuChannelId() even for Android
without a GPU process.

[reveman, 2014/11/04 06:20:46]

Thanks. Fixed in latest patch.

   gfx::ScopedJavaSurface surface(
-      content::GetSurfaceTextureSurface(surface_texture_id, client_id));
+      content::GetSurfaceTextureSurface(surface_texture_id, kBrowserClientId));
   if (surface.j_surface().is_null())
     return NULL;