Add initial GpuMemoryBufferSurfaceTexture implementation.

content/common/gpu/gpu_command_buffer_stub.cc

Index: content/common/gpu/gpu_command_buffer_stub.cc
diff --git a/content/common/gpu/gpu_command_buffer_stub.cc b/content/common/gpu/gpu_command_buffer_stub.cc
index 497858a38784b8559f4d27649b5762f795548cf3..c436fa74429a441b132a5f12ab7430809a2cd52a 100644
--- a/content/common/gpu/gpu_command_buffer_stub.cc
+++ b/content/common/gpu/gpu_command_buffer_stub.cc
@@ -870,6 +870,15 @@ void GpuCommandBufferStub::OnRegisterGpuMemoryBuffer(
     uint32 height,
     uint32 internalformat) {
   TRACE_EVENT0("gpu", "GpuCommandBufferStub::OnRegisterGpuMemoryBuffer");
+#if defined(OS_ANDROID)
+  // Verify that renderer is not trying to use a surface texture it doesn't own.
+  if (gpu_memory_buffer.type == gfx::SURFACE_TEXTURE_BUFFER &&
+      gpu_memory_buffer.surface_texture_id.secondary_id !=
+          channel()->renderer_pid()) {

[piman, 2014/03/27 01:07:25]

nit: rather than the PID, use the channel's client_id_ (which is the renderer
process id, render_process_id_ in RenderMessageFilter or GpuMessageFilter), it's
more portable and keeps the separation independent of how renderer processes are
used by various compositors.
Also, PIDs get reused (small namespace), whereas render_process_id_ don't.

[reveman, 2014/03/27 14:48:11]

Oh, I didn't know that we shared this id the GPU service. That's great! Cleans
up a lot of the Java code and will make out-of-process GPU support easier to
implement as we don't have to worry about id reuse.

+    LOG(ERROR) << "Illegal surface texture ID for renderer.";
+    return;
+  }
+#endif
   if (gpu_control_) {
     gpu_control_->RegisterGpuMemoryBuffer(id,
                                           gpu_memory_buffer,