Make PacketReader/PacketWriter interfaces async-only.

blimp/net/packet_reader.h

Index: blimp/net/packet_reader.h
diff --git a/blimp/net/packet_reader.h b/blimp/net/packet_reader.h
index 407e2fdd2c1d7f3fc7343404a0e2bf41bf95a7e8..bcc4901254609c4f7f9d9afbaa3d00281d652bc0 100644
--- a/blimp/net/packet_reader.h
+++ b/blimp/net/packet_reader.h
@@ -18,14 +18,11 @@ class BLIMP_NET_EXPORT PacketReader {
   virtual ~PacketReader() {}
 
   // Reads a packet from the connection.
-  // Returns the size of the packet, in bytes, if the read operation executed
-  // successfully.
-  // Returns ERR_IO_PENDING if the operation will be executed asynchronously.
-  //     |cb| is later invoked with the packet size or an error code.
-  // All other return values indicate errors and caller should stop using this
-  // reader.
-  virtual int ReadPacket(const scoped_refptr<net::GrowableIOBuffer>& buf,
-                         const net::CompletionCallback& cb) = 0;
+  // Passes net::OK to |cb| if the read operation  executed successfully.

[Wez, 2015/11/24 23:43:18]

nit: Remove double-space

[Kevin M, 2015/11/25 01:18:37]

Done.

+  // The read size can be accessed by calling |buf.offset()|.

[Wez, 2015/11/24 23:43:18]

Suggest: Sets |buf.offset()| to the received message's size, and invokes |cb|
with net::OK result, on success.

[Kevin M, 2015/11/25 01:18:37]

Done.

+  // All other values indicate errors.
+  virtual void ReadPacket(const scoped_refptr<net::GrowableIOBuffer>& buf,
+                          const net::CompletionCallback& cb) = 0;
 };
 
 }  // namespace blimp