Add interfaces for most major Blimp net components.

blimp/engine/browser/blimp_engine_session.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_ENGINE_BROWSER_BLIMP_ENGINE_SESSION_H_
 #define BLIMP_ENGINE_BROWSER_BLIMP_ENGINE_SESSION_H_
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
-#include "blimp/net/blimp_message_receiver.h"
+#include "blimp/net/blimp_message_processor.h"
 #include "content/public/browser/web_contents_delegate.h"
+#include "net/base/net_errors.h"

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

Why are you including this?

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

Done.

 #include "ui/gfx/geometry/size.h"
 
 namespace aura {
 class WindowTreeHost;
 
 namespace client {
 class DefaultCaptureClient;
 class WindowTreeClient;
 }  // namespace client
 }  // namespace aura
@@ skipping 13 matching lines @@
 class BlimpMessage;
 
 namespace engine {
 
 class BlimpBrowserContext;
 class BlimpFocusClient;
 class BlimpScreen;
 class BlimpUiContextFactory;
 class BlimpWindowTreeHost;
 
-class BlimpEngineSession : public BlimpMessageReceiver,
+class BlimpEngineSession : public BlimpMessageProcessor,
                            public content::WebContentsDelegate {
  public:
   explicit BlimpEngineSession(scoped_ptr<BlimpBrowserContext> browser_context);
   ~BlimpEngineSession() override;
 
   void Initialize();
 
   BlimpBrowserContext* browser_context() { return browser_context_.get(); }
 
-  // BlimpMessageReceiver implementation.
-  // TODO(haibinlu): Remove this in favor of the BlimpMessageDispatcher.
-  net::Error OnBlimpMessage(const BlimpMessage& message) override;
+  // BlimpMessageProcessor implementation.
+  // TODO(haibinlu): Delete this and move to BlimpMessageDemultiplexer.
+  void ProcessMessage(const BlimpMessage& message,
+                      const net::CompletionCallback& callback) override;

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

include net/completion_callback for this, or forward-declare it?

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

Done.

 
  private:
   // Creates a new WebContents, which will be indexed by |target_tab_id|.
   void CreateWebContents(const int target_tab_id);
 
   // Navigates the target tab to the |url|.
   void LoadUrl(const int target_tab_id, const GURL& url);
 
   // content::WebContentsDelegate implementation.
   content::WebContents* OpenURLFromTab(
@@ skipping 37 matching lines @@
   // Currently attached client connection.
   scoped_ptr<BlimpConnection> client_connection_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpEngineSession);
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_BROWSER_BLIMP_ENGINE_SESSION_H_