Create new class "CastTransport", which encapsulates the message read and write event loops.

extensions/browser/api/cast_channel/cast_channel_api.h

 // Copyright 2014 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 EXTENSIONS_BROWSER_API_CAST_CHANNEL_CAST_CHANNEL_API_H_
 #define EXTENSIONS_BROWSER_API_CAST_CHANNEL_CAST_CHANNEL_API_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 30 matching lines @@
 class CastChannelAPI : public BrowserContextKeyedAPI,
                        public cast_channel::CastSocket::Delegate {
  public:
   explicit CastChannelAPI(content::BrowserContext* context);
 
   static CastChannelAPI* Get(content::BrowserContext* context);
 
   // BrowserContextKeyedAPI implementation.
   static BrowserContextKeyedAPIFactory<CastChannelAPI>* GetFactoryInstance();
 
-  // Returns a new CastSocket that connects to |ip_endpoint| with authentication
+  // Returns a new CastSocket that connects to |ip_endpoint| with
+  // authentication

[Wez, 2014/09/25 02:09:15]

nit: This can be un-wrapped now that CastSocket is shorter!

[Kevin M, 2014/09/25 17:53:04]

Done.

   // |channel_auth| and is to be owned by |extension_id|.
   scoped_ptr<cast_channel::CastSocket> CreateCastSocket(
       const std::string& extension_id,
       const net::IPEndPoint& ip_endpoint,
       cast_channel::ChannelAuthType channel_auth,
       const base::TimeDelta& timeout);
 
   // Returns a pointer to the Logger member variable.
   // TODO(imcheng): Consider whether it is possible for this class to own the
-  // CastSockets and make this class the sole owner of Logger. Alternatively,
+  // CastSockets and make this class the sole owner of Logger.
+  // Alternatively,

[Wez, 2014/09/25 02:09:15]

nit: And this!

[Kevin M, 2014/09/25 17:53:04]

This function's still useful, the Open and GetLogs functions rely on it to get
the logger instance.

   // consider making Logger not ref-counted by passing a weak
   // reference of Logger to the CastSockets instead.
   scoped_refptr<cast_channel::Logger> GetLogger();
 
   // Sets the CastSocket instance to be returned by CreateCastSocket for
   // testing.
   void SetSocketForTest(scoped_ptr<cast_channel::CastSocket> socket_for_test);
 
  private:
   friend class BrowserContextKeyedAPIFactory<CastChannelAPI>;
@@ skipping 166 matching lines @@
   void OnClose(int result);
 
   CastChannelAPI* api_;
 
   DISALLOW_COPY_AND_ASSIGN(CastChannelGetLogsFunction);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_CAST_CHANNEL_CAST_CHANNEL_API_H_