Added a debug info UI for Blimp

blimp/client/feature/compositor/blimp_compositor_manager.h

Index: blimp/client/feature/compositor/blimp_compositor_manager.h
diff --git a/blimp/client/feature/compositor/blimp_compositor_manager.h b/blimp/client/feature/compositor/blimp_compositor_manager.h
index 9a1879637cf179cabfd8c70f87f934883a674efd..d4e1ec2177253864e8aa900f307892ecba823f47 100644
--- a/blimp/client/feature/compositor/blimp_compositor_manager.h
+++ b/blimp/client/feature/compositor/blimp_compositor_manager.h
@@ -19,6 +19,7 @@ class ClientImageSerializationProcessor;
 class BlimpCompositorManagerClient {

[Kevin M, 2016/05/20 01:02:03]

This should probably be called a Delegate, not a Client.

[shaktisahu, 2016/05/22 22:36:56]

Hmm... Sounds okay to me. Anyway this is not my changes and don't want to change
it.

  public:
   virtual void OnSwapBuffersCompleted() = 0;
+  virtual void DidCommitAndDrawFrame() = 0;

[Kevin M, 2016/05/20 01:02:03]

Can we standardize our naming here, so it's all OnXYZ or DidXYZ, but not both?

[shaktisahu, 2016/05/22 22:36:56]

This is the same callback propagated all the way from cc layer. May be sometimes
one convention makes more sense than the other? e.g. when there are more than
one verb in the method name :)

 };
 
 // The BlimpCompositorManager manages multiple BlimpCompositor instances, each
@@ -70,6 +71,7 @@ class BlimpCompositorManager
 
   // BlimpCompositorClient implementation.
   void DidCompleteSwapBuffers() override;
+  void DidCommitAndDrawFrame() override;
   cc::LayerTreeSettings* GetLayerTreeSettings() override;
   scoped_refptr<base::SingleThreadTaskRunner>
   GetCompositorTaskRunner() override;