Implement StreamTexture::BindTexImage.

content/common/gpu/client/command_buffer_proxy_impl.cc

Index: content/common/gpu/client/command_buffer_proxy_impl.cc
diff --git a/content/common/gpu/client/command_buffer_proxy_impl.cc b/content/common/gpu/client/command_buffer_proxy_impl.cc
index 60cc94af803d22b15e21457b8b42f1d75c55b25c..3243f1b219394228b9ad618dbaaa0fb88f9e2240 100644
--- a/content/common/gpu/client/command_buffer_proxy_impl.cc
+++ b/content/common/gpu/client/command_buffer_proxy_impl.cc
@@ -452,6 +452,23 @@ uint32 CommandBufferProxyImpl::CreateStreamTexture(uint32 texture_id) {
   return stream_id;
 }
 
+uint32 CommandBufferProxyImpl::CreateStreamTextureImage(int32 image_id) {

[no sievers, 2015/07/20 23:33:08]

It would be nice if we could return the image_id here instead of passing it in,
so that it works the same as CreateImage().

Maybe we can make |stream_id| internal to StreamTextureHost somehow, otherwise
it's confusing to have two ids in the API.

We can probably do it by just shuffling code around a bit in between
StreamTextureFactory/Proxy and StreamTextureHost (EstablishPeer and SetSize
would go through StreamTextureProxy instead, so outside of proxy/host nobody
needs to know the stream/route id).

+  CheckLock();
+  if (last_state_.error != gpu::error::kNoError)
+    return 0;
+
+  int32 stream_id = channel_->GenerateRouteID();
+  bool succeeded = false;
+  Send(new GpuCommandBufferMsg_CreateStreamTextureImage(route_id_, image_id,
+                                                        stream_id, &succeeded));
+  if (!succeeded) {
+    DLOG(ERROR)
+        << "GpuCommandBufferMsg_CreateStreamTextureImage returned failure";
+    return 0;
+  }
+  return stream_id;
+}
+
 void CommandBufferProxyImpl::SetLock(base::Lock* lock) {
   lock_ = lock;
 }