Added a debug info UI for Blimp

blimp/net/blimp_connection.h

Index: blimp/net/blimp_connection.h
diff --git a/blimp/net/blimp_connection.h b/blimp/net/blimp_connection.h
index a662c4e10ef5d488ef5352ae3b2d1c555f66c030..0e7ef546a1e59da3fd7d823a9142c7d15a197711 100644
--- a/blimp/net/blimp_connection.h
+++ b/blimp/net/blimp_connection.h
@@ -14,6 +14,7 @@
 
 namespace blimp {
 
+class BlimpConnectionStatistics;
 class BlimpMessageProcessor;
 class BlimpMessagePump;
 class PacketReader;
@@ -42,6 +43,8 @@ class BLIMP_NET_EXPORT BlimpConnection : public ConnectionErrorObserver {
   // Gets a processor for BrowserSession->BlimpConnection message routing.
   virtual BlimpMessageProcessor* GetOutgoingMessageProcessor();
 
+  void SetBlimpConnectionStatistics(BlimpConnectionStatistics* statistics);

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

Does it make sense to *not* have connection statistics? Can this be moved into
the constructor?

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

No. The BlimpConnection is built at too deep in the call stack where I don't
have access. I have to pass this all the way from BlimpClientSession to specific
PacketReader/PacketWriter implementation, which is pretty terrible. The other
alternative is to make this class a singleton but singletons are not good
patterns in general.

+
  protected:
   BlimpConnection();