Refactor BlimpConnection to TCPConnection

blimp/net/stream_packet_reader.cc

Index: blimp/net/stream_packet_reader.cc
diff --git a/blimp/net/stream_packet_reader.cc b/blimp/net/stream_packet_reader.cc
index 08b652e4019e8db38be50bdf0f36ecf4cc81e3a6..af5cae9c7cdbb785800be3fb92cae87901468b97 100644
--- a/blimp/net/stream_packet_reader.cc
+++ b/blimp/net/stream_packet_reader.cc
@@ -38,7 +38,10 @@ std::ostream& operator<<(std::ostream& out,
 }
 
 StreamPacketReader::StreamPacketReader(net::StreamSocket* socket)
-    : read_state_(ReadState::IDLE), socket_(socket), weak_factory_(this) {
+    : read_state_(ReadState::IDLE),
+      socket_(socket),
+      payload_size_(0),

[Kevin M, 2016/10/25 18:49:23]

Curious - why add this now?

[perumaal, 2016/10/25 23:02:39]

Oh Eclipse flagged this as uninitialized variable.

It might *just* work without having to initialize given the call ordering but
still...

+      weak_factory_(this) {
   DCHECK(socket_);
   header_buffer_ = new net::GrowableIOBuffer;
   header_buffer_->SetCapacity(kPacketHeaderSizeBytes);
@@ -153,7 +156,7 @@ void StreamPacketReader::OnReadComplete(int result) {
     return;
   }
 
-  // If the read was succesful, then process the result.
+  // If the read was successful, then process the result.
   if (result > 0) {
     result = DoReadLoop(result);
   }