Add interfaces for most major Blimp net components.

blimp/net/stream_socket_connection.h

Index: blimp/net/stream_socket_connection.h
diff --git a/blimp/net/stream_socket_connection.h b/blimp/net/stream_socket_connection.h
index 9739a09b482e936dc29d9a89faaa0436d1f85e57..1006f26c024d654bafd117011b2aaa1162102dd0 100644
--- a/blimp/net/stream_socket_connection.h
+++ b/blimp/net/stream_socket_connection.h
@@ -19,8 +19,7 @@ namespace blimp {
 class StreamSocketConnection : public BlimpConnection {
  public:
   explicit StreamSocketConnection(scoped_ptr<net::StreamSocket> socket);
-
-  ~StreamSocketConnection() override;
+  ~StreamSocketConnection();

[Wez, 2015/11/12 03:45:57]

This seems dubious; if BlimpConnection can be derived-from then shouldn't we
give it a virtual destructor so that code that holds a BlimpConnection can
delete it and have it delete the sub-class properly?

[Kevin M, 2015/11/12 19:03:47]

Done.

[Wez, 2015/11/12 20:12:55]

I actually meant BlimpConnection's dtor - it's still non-virtual, which means
that a caller holding a BlimpConnection* pointing to a StreamBlimpConnection
will NOT call the StreamBlimpConnection dtor.

 
  private:
   scoped_ptr<net::StreamSocket> socket_;