Added a unique command buffer ID for command buffers.

mojo/gles2/command_buffer_client_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mojo/gles2/command_buffer_client_impl.h"
 
 #include <limits>
 
 #include "base/logging.h"
 #include "base/process/process_handle.h"
@@ skipping 375 matching lines @@
 }
 
 void CommandBufferClientImpl::SetLock(base::Lock* lock) {
 }
 
 bool CommandBufferClientImpl::IsGpuChannelLost() {
   // This is only possible for out-of-process command buffers.
   return false;
 }
 
+gpu::CommandBufferNamespace CommandBufferClientImpl::GetNamespaceID() const {
+  return gpu::kCommandBufferNamespace_Mojo;
+}
+
+uint64_t CommandBufferClientImpl::GetCommandBufferID() const {
+  NOTIMPLEMENTED();

[rjkroege, 2015/09/16 18:55:22]

So, there must be a point to this CL or you wouldn't have added the code. :-)
So, what's going to go wrong in mandoline because all the command buffers are id
0? If having uniform cmd buffer ids of 0 is going to  break mandoline, I'd
really like you to implement this prior to landing.


Per my previous comment -- I think that there is a reasonably obvious namespace
id to give to a command buffer in mandoline that mirrors the way that we assign
ids to views: connection id concatenated with a per-client unique int.

[piman, 2015/09/16 20:18:07]

It's not currently used, as you can see.
It will be used for memory tracing (which I don't think mandoline does), and for
the new lightweight sync points whose API can be implemented on top of existing
sync points, and will probably have to be, for mojo.
What's a connection id, what's a view id? Where do they come from?
What about offscreen context that y'all just introduced for webgl?
https://codereview.chromium.org/1338433002/

Can we get help from the mandoline/mojo team to implement the mojo side of
things? It's becoming increasingly painful to make changes to command buffers
and cc when mojo gets in the way....

+  return 0;
+}
+
 }  // namespace gles2