Incorporate BlobChannel into Blimp image encode/decode pipeline.

blimp/client/session/blimp_client_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_CLIENT_SESSION_BLIMP_CLIENT_SESSION_H_
 #define BLIMP_CLIENT_SESSION_BLIMP_CLIENT_SESSION_H_
 
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/threading/thread.h"
 #include "blimp/client/session/assignment_source.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/net/blimp_message_processor.h"
 
 namespace net {
 class IPEndPoint;
 }
 
 namespace blimp {
 
 class BlimpMessageProcessor;
 class BlimpMessageThreadPipe;
+class BlobCache;

[Wez, 2016/05/21 01:08:04]

You don't seem to use this below?

[Kevin M, 2016/05/27 22:35:30]

Done.

+class BlobChannelReceiver;
 class BrowserConnectionHandler;
 class ClientConnectionManager;
+class HeliumBlobReceiverDelegate;
 class ThreadPipeManager;
 
 namespace client {
 
 class ClientNetworkComponents;
 class NavigationFeature;
 class ImeFeature;
 class RenderWidgetFeature;
 class SettingsFeature;
 class TabControlFeature;
@@ skipping 46 matching lines @@
                                                const Assignment& assignment);
 
  private:
   void RegisterFeatures();
 
   // NetworkEventObserver implementation.
   void OnConnected() override;
   void OnDisconnected(int result) override;
 
   base::Thread io_thread_;
+  HeliumBlobReceiverDelegate* blob_delegate_ = nullptr;

[Wez, 2016/05/21 01:08:04]

nit: Need a comment to explain this member, and in particular why it is bare.

[Kevin M, 2016/05/27 22:35:30]

Done.

+  std::unique_ptr<BlobChannelReceiver> blob_receiver_;
   std::unique_ptr<TabControlFeature> tab_control_feature_;
   std::unique_ptr<NavigationFeature> navigation_feature_;
   std::unique_ptr<ImeFeature> ime_feature_;
   std::unique_ptr<RenderWidgetFeature> render_widget_feature_;
   std::unique_ptr<SettingsFeature> settings_feature_;
 
   // The AssignmentSource is used when the user of BlimpClientSession calls
   // Connect() to get a valid assignment and later connect to the engine.
   std::unique_ptr<AssignmentSource> assignment_source_;
 
   // Container struct for network components.
   // Must be deleted on the IO thread.
   std::unique_ptr<ClientNetworkComponents> net_components_;
 
   std::unique_ptr<ThreadPipeManager> thread_pipe_manager_;
 
   base::WeakPtrFactory<BlimpClientSession> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpClientSession);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_SESSION_BLIMP_CLIENT_SESSION_H_