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 102 matching lines @@
 
   uint32_t sync_point_;
 
   mojo::Binding<mojo::CommandBufferSyncPointClient> binding_;
 };
 
 CommandBufferClientImpl::CommandBufferClientImpl(
     CommandBufferDelegate* delegate,
     const MojoAsyncWaiter* async_waiter,
     mojo::ScopedMessagePipeHandle command_buffer_handle)
-    : delegate_(delegate),
+    : CommandBuffer(gpu::kCommandBufferNamespace_Invalid, 0),

[rjkroege, 2015/09/15 22:48:41]

Is this right? Your goal with this CL is to give each command buffer a unique id
right? This would give the CommandBuffer instances uniform ids of 0. 

perhaps the ids could work like mojo ViewIds (src/components/mus/ids.h) and be a
combination of client-specific integer and the connect id? this seems more
aligned with mojo.

+      delegate_(delegate),
       observer_binding_(this),
       shared_state_(NULL),
       last_put_offset_(-1),
       next_transfer_buffer_id_(0),
       next_image_id_(0),
       async_waiter_(async_waiter) {
   command_buffer_.Bind(mojo::InterfacePtrInfo<mojo::CommandBuffer>(
                            command_buffer_handle.Pass(), 0u),
                        async_waiter);
   command_buffer_.set_connection_error_handler(
@@ skipping 253 matching lines @@
 
 void CommandBufferClientImpl::SetLock(base::Lock* lock) {
 }
 
 bool CommandBufferClientImpl::IsGpuChannelLost() {
   // This is only possible for out-of-process command buffers.
   return false;
 }
 
 }  // namespace gles2