Added a unique command buffer ID for command buffers.

components/mus/gles2/command_buffer_local.cc

 // Copyright 2015 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 "components/mus/gles2/command_buffer_local.h"
 
 #include "base/bind.h"
 #include "components/mus/gles2/command_buffer_local_client.h"
 #include "components/mus/gles2/gpu_memory_tracker.h"
 #include "components/mus/gles2/mojo_gpu_memory_buffer.h"
@@ skipping 194 matching lines @@
 
 void CommandBufferLocal::SetLock(base::Lock* lock) {
   NOTIMPLEMENTED();
 }
 
 bool CommandBufferLocal::IsGpuChannelLost() {
   // This is only possible for out-of-process command buffers.
   return false;
 }
 
+gpu::CommandBufferNamespace CommandBufferLocal::GetNamespaceID() const {
+  NOTIMPLEMENTED();

[sky, 2015/09/17 17:03:06]

Are these going to be spammy? What is the consequence of not implementing this?

[David Yen, 2015/09/17 17:07:04]

These are going to be used for a new sync point implementation, so sync points
will not work. However, since the old sync points were already NOTIMPLEMENTED()
it should be safe to assume they aren't used.

+  return gpu::kCommandBufferNamespace_Invalid;
+}
+
+uint64_t CommandBufferLocal::GetCommandBufferID() const {
+  NOTIMPLEMENTED();
+  return 0;
+}
+
 void CommandBufferLocal::PumpCommands() {
   if (!decoder_->MakeCurrent()) {
     command_buffer_->SetContextLostReason(decoder_->GetContextLostReason());
     command_buffer_->SetParseError(::gpu::error::kLostContext);
     return;
   }
   scheduler_->PutChanged();
 }
 
 void CommandBufferLocal::OnResize(gfx::Size size, float scale_factor) {
@@ skipping 28 matching lines @@
 void CommandBufferLocal::OnContextLost(uint32_t reason) {
   if (client_)
     client_->DidLoseContext();
 }
 
 void CommandBufferLocal::OnSyncPointRetired() {
   scheduler_->SetScheduled(true);
 }
 
 }  // namespace gles2