Added a debug info UI for Blimp

cc/trees/remote_channel_unittest.cc

Index: cc/trees/remote_channel_unittest.cc
diff --git a/cc/trees/remote_channel_unittest.cc b/cc/trees/remote_channel_unittest.cc
index e7a0ef0e91c6e6c8953c2b4a51019a47c5c7fe84..1974c23b6e4b3e0152e210b6a291ad7bc8f533d8 100644
--- a/cc/trees/remote_channel_unittest.cc
+++ b/cc/trees/remote_channel_unittest.cc
@@ -143,10 +143,12 @@ class RemoteChannelTestCommit : public RemoteChannelTest {
 
   void StartCommitOnImpl() override { EXPECT_EQ(2, calls_received_++); }
 
-  void DidCommitAndDrawFrame() override { EXPECT_EQ(3, calls_received_++); }
+  void DidCommitAndDrawFrame() override {
+    calls_received_on_both_server_and_client_++;
+  }
 
   void DidCompleteSwapBuffers() override {
-    if (++calls_received_on_both_server_and_client_ == 2)
+    if (++calls_received_on_both_server_and_client_ == 4)

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

I know the previous code was written this way, but we shouldn't be mixing
increment operations with conditionals - plz put them on separate lines.

[vmpstr, 2016/05/20 23:26:14]

(here and in the above lines please. increment in a macro is typically a bad
thing)

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

Done.

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

Done.

       EndTest();
   }
 
@@ -157,8 +159,8 @@ class RemoteChannelTestCommit : public RemoteChannelTest {
   }
 
   void AfterTest() override {
-    EXPECT_EQ(4, calls_received_);
-    EXPECT_EQ(2, calls_received_on_both_server_and_client_);
+    EXPECT_EQ(3, calls_received_);
+    EXPECT_EQ(4, calls_received_on_both_server_and_client_);
   }
 
   const gfx::Size viewport_size_ = gfx::Size(5, 3);