Added a debug info UI for Blimp

blimp/net/blimp_message_pump.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 "blimp/net/blimp_message_pump.h"
 
 #include "base/macros.h"
 #include "blimp/common/logging.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/net/blimp_message_processor.h"
 #include "blimp/net/common.h"
 #include "blimp/net/connection_error_observer.h"
 #include "blimp/net/packet_reader.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 
 namespace blimp {
 
 BlimpMessagePump::BlimpMessagePump(PacketReader* reader)
     : reader_(reader), buffer_(new net::GrowableIOBuffer), weak_factory_(this) {
   DCHECK(reader_);
   buffer_->SetCapacity(kMaxPacketPayloadSizeBytes);
 }
 
 BlimpMessagePump::~BlimpMessagePump() {}
 
+void BlimpMessagePump::SetBlimpConnectionStatistics(
+    BlimpConnectionStatistics* connection_statistics) {
+  DCHECK(reader_);

[Kevin M, 2016/05/24 01:02:01]

Not needed: constructor enforces non-nullness, and the following line will
segfault if it *does* manage to be null

[shaktisahu, 2016/05/24 21:02:45]

Removed function since we are passing in constructor

+  reader_->set_blimp_connection_statistics(connection_statistics);
+}
+
 void BlimpMessagePump::SetMessageProcessor(BlimpMessageProcessor* processor) {
   DVLOG(1) << "SetMessageProcessor, processor=" << processor;
   if (processor && !processor_) {
     processor_ = processor;
     ReadNextPacket();
   } else {
     // Don't allow |processor_| to be cleared while there's a read inflight.
     if (processor) {
       DCHECK(!processor_ || !read_inflight_);
     }
@@ skipping 40 matching lines @@
   if (result < 0) {
     error_observer_->OnConnectionError(result);
     return;
   }
 
   if (processor_)
     ReadNextPacket();
 }
 
 }  // namespace blimp