Incorporate BlobChannel into Blimp image encode/decode pipeline.

blimp/net/browser_connection_handler.h

Index: blimp/net/browser_connection_handler.h
diff --git a/blimp/net/browser_connection_handler.h b/blimp/net/browser_connection_handler.h
index 2c972c30704e7759b8bea518bda61e7c161d2595..9827139db86d82310fc60e5e5aa958a4440466b9 100644
--- a/blimp/net/browser_connection_handler.h
+++ b/blimp/net/browser_connection_handler.h
@@ -52,9 +52,9 @@ class BLIMP_NET_EXPORT BrowserConnectionHandler
   // ConnectionErrorObserver implementation.
   void OnConnectionError(int error) override;
 
- private:
   void DropCurrentConnection();

[Wez, 2016/06/07 01:18:31]

This is only valid to call while there is a connection active, so that
limitation at least needs to be documented here.  e.g. What would happen if we
had several image-decode operations in a row fail? Wouldn't that result in
several DropCurrentConnection calls?

[Kevin M, 2016/06/08 00:21:09]

See previous comment about idempotent connection drops.

[Wez, 2016/06/10 22:29:54]

Acknowledged.

 
+ private:
   // Routes incoming messages to the relevant feature-specific handlers.
   std::unique_ptr<BlimpMessageDemultiplexer> demultiplexer_;