Added a debug info UI for Blimp

blimp/net/tcp_engine_transport.cc

Index: blimp/net/tcp_engine_transport.cc
diff --git a/blimp/net/tcp_engine_transport.cc b/blimp/net/tcp_engine_transport.cc
index f214a69189d99ba13584c34fecc73d1620bc8acc..7acf538ad6a6088e453faecd523af896c9e84d7b 100644
--- a/blimp/net/tcp_engine_transport.cc
+++ b/blimp/net/tcp_engine_transport.cc
@@ -10,6 +10,7 @@
 #include "base/callback_helpers.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
+#include "blimp/net/blimp_connection_statistics.h"
 #include "blimp/net/stream_socket_connection.h"
 #include "net/socket/stream_socket.h"
 #include "net/socket/tcp_server_socket.h"
@@ -59,8 +60,8 @@ void TCPEngineTransport::Connect(const net::CompletionCallback& callback) {
 std::unique_ptr<BlimpConnection> TCPEngineTransport::TakeConnection() {
   DCHECK(connect_callback_.is_null());
   DCHECK(accepted_socket_);
-  return base::WrapUnique(
-      new StreamSocketConnection(std::move(accepted_socket_)));
+  return base::WrapUnique(new StreamSocketConnection(
+      std::move(accepted_socket_), new BlimpConnectionStatistics));

[Kevin M, 2016/05/24 21:49:01]

This leaks the statistics object.

Maybe consider making the statistics object refcounted. That takes care of any
potential ownership issues and gives you the ability to pass in a valid but
unused object such as the case we have here.

 }
 
 const char* TCPEngineTransport::GetName() const {