Add interfaces for most major Blimp net components.

blimp/net/stream_socket_connection.h

 // 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.
 
 #ifndef BLIMP_NET_STREAM_SOCKET_CONNECTION_H_
 #define BLIMP_NET_STREAM_SOCKET_CONNECTION_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "blimp/net/blimp_connection.h"
 #include "net/socket/stream_socket.h"
 
 namespace net {
 class StreamSocket;
 }  // namespace net
 
 namespace blimp {
 
 // BlimpConnection specialization for StreamSocket-based connections.
 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_;
 };
 
 }  // namespace blimp
 
 #endif  // BLIMP_NET_STREAM_SOCKET_CONNECTION_H_