BlobChannelService is modified to be Mojo Service.

blimp/engine/mojo/blob_channel_service.h

 // Copyright 2016 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_MOJO_BLOB_CHANNEL_SERVICE_H_
 #define BLIMP_ENGINE_MOJO_BLOB_CHANNEL_SERVICE_H_
 
 #include <string>
 
 #include "blimp/engine/mojo/blob_channel.mojom.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
+#include "mojo/public/cpp/bindings/binding_set.h"
 
 namespace blimp {
 
 class BlobChannelSender;
 
 namespace engine {
 
 // Service for processing BlobChannel requests from the renderer.
 // Runs on the browser process.
 class BlobChannelService : public mojom::BlobChannel {
  public:
+  // |blob_channel_sender| must outlive the Mojo connection.
+  explicit BlobChannelService(BlobChannelSender* blob_channel_sender);
+
   ~BlobChannelService() override;
 
   // Factory method called by Mojo.
-  static void Create(BlobChannelSender* blob_channel_sender,
-                     mojo::InterfaceRequest<mojom::BlobChannel> request);
+  // Binds |this| to the connection specified by |request|.
+  void BindRequest(mojo::InterfaceRequest<mojom::BlobChannel> request);
 
  private:
-  // Creates a BlobChannel bound to the connection specified by |request|.
-  // |blob_channel_sender| must outlive the Mojo connection.
-  BlobChannelService(BlobChannelSender* blob_channel_sender,
-                     mojom::BlobChannelRequest request);
-
   // BlobChannel implementation.
   void GetCachedBlobIds(
       const GetCachedBlobIdsCallback& response_callback) override;
   void PutBlob(const std::string& id,
                mojo::ScopedSharedBufferHandle data,
                uint32_t size) override;
   void DeliverBlob(const std::string& id) override;
 
-  // Binds |this| and its object lifetime to a Mojo connection.
-  mojo::StrongBinding<mojom::BlobChannel> binding_;
+  // Binds |this| and its object lifetime to a set of Mojo connections.

[Kevin M, 2016/07/28 20:59:17]

The lifetime is not bound to BindingSet at all?

[CJ, 2016/07/28 22:47:05]

Done.

+  mojo::BindingSet<mojom::BlobChannel> bindings_;
 
   // Sender object which will receive the blobs passed over the Mojo service.
   BlobChannelSender* blob_channel_sender_;
 
   DISALLOW_COPY_AND_ASSIGN(BlobChannelService);
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_MOJO_BLOB_CHANNEL_SERVICE_H_