Added a debug info UI for Blimp

blimp/net/blimp_message_pump.h

 // 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.
 
 #ifndef BLIMP_NET_BLIMP_MESSAGE_PUMP_H_
 #define BLIMP_NET_BLIMP_MESSAGE_PUMP_H_
 
+#include <blimp/net/blimp_connection_statistics.h>

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

Angle brackets.

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

Now that I think about it, I suspect angle bracket was automatically added when
I refactored the class name in Eclipse. Coz, in the previous commit, I din't
have any :)

 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "blimp/net/blimp_net_export.h"
+#include "blimp/net/packet_reader.h"
 #include "net/base/completion_callback.h"
 
 namespace net {
 class GrowableIOBuffer;
 }
 
 namespace blimp {
 
 class BlimpMessageProcessor;
 class ConnectionErrorObserver;
@@ skipping 13 matching lines @@
   // Caller retains the ownership of |processor|.
   // The processor can be unset by passing a null pointer when no reads are
   // inflight, ie. after |processor_|->ProcessMessage() is called, but before
   // ProcessMessage() invokes its completion callback.
   void SetMessageProcessor(BlimpMessageProcessor* processor);
 
   void set_error_observer(ConnectionErrorObserver* observer) {
     error_observer_ = observer;
   }
 
+  void SetBlimpConnectionStatistics(

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

IIRC we can only inline simple setters and getters - if we have to pull in a
header so we can access a class member, then it should go in the .cc file.

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

Done.

+      BlimpConnectionStatistics* connection_statistics) {
+    DCHECK(reader_);
+    reader_->set_blimp_connection_statistics(connection_statistics);
+  }
+
  private:
   // Read next packet from |reader_|.
   void ReadNextPacket();
 
   // Callback when next packet is ready in |buffer_|.
   void OnReadPacketComplete(int result);
 
   // Callback when |processor_| finishes processing a BlimpMessage.
   // Any values other than net::OK indicate that |processor_| has encountered an
   // error that should be handled. Currently all errors will cause the
@@ skipping 10 matching lines @@
 
   // Used to abandon ProcessMessage completion callbacks if |this| is deleted.
   base::WeakPtrFactory<BlimpMessagePump> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpMessagePump);
 };
 
 }  // namespace blimp
 
 #endif  // BLIMP_NET_BLIMP_MESSAGE_PUMP_H_