BlobChannelService is modified to be Mojo Service.

blimp/engine/mojo/blob_channel_service.cc

 // 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.
 
 #include "blimp/engine/mojo/blob_channel_service.h"
 
 #include <string>
 #include <unordered_map>
 #include <utility>
 #include <vector>
 
 #include "base/memory/ptr_util.h"
 #include "blimp/net/blob_channel/blob_channel_sender.h"
 #include "mojo/public/cpp/system/buffer.h"
 
 namespace blimp {
 namespace engine {
 
-BlobChannelService::BlobChannelService(BlobChannelSender* blob_channel_sender,
-                                       mojom::BlobChannelRequest request)
-    : binding_(this, std::move(request)),
-      blob_channel_sender_(blob_channel_sender) {
+BlobChannelService::BlobChannelService(BlobChannelSender* blob_channel_sender)
+    : blob_channel_sender_(blob_channel_sender) {
   DCHECK(blob_channel_sender_);
 }
 
 BlobChannelService::~BlobChannelService() {}
 
 void BlobChannelService::GetCachedBlobIds(
     const BlobChannelService::GetCachedBlobIdsCallback& response_callback) {
   VLOG(1) << "BlobChannel::GetCachedBlobIds called.";
   std::unordered_map<std::string, bool> cache_state;
   for (const auto& next_entry : blob_channel_sender_->GetCachedBlobIds()) {
@@ skipping 21 matching lines @@
 
   scoped_refptr<BlobData> new_blob(new BlobData);
   new_blob->data.assign(reinterpret_cast<const char*>(mapping.get()), size);
   blob_channel_sender_->PutBlob(id, std::move(new_blob));
 }
 
 void BlobChannelService::DeliverBlob(const std::string& id) {
   blob_channel_sender_->DeliverBlob(id);
 }
 
-// static
-void BlobChannelService::Create(
-    BlobChannelSender* blob_channel_sender,
+void BlobChannelService::BindRequest(
     mojo::InterfaceRequest<mojom::BlobChannel> request) {
-  // Object lifetime is managed by BlobChannelService's StrongBinding
-  // |binding_|.
-  new BlobChannelService(blob_channel_sender, std::move(request));
+  // Object lifetime is managed by BlobChannelService's BindingSet

[Kevin M, 2016/07/28 18:14:47]

This isn't the case anymore?

[CJ, 2016/07/28 19:54:15]

Done.

+  // |bindings_|.
+  bindings_.AddBinding(this, std::move(request));
 }
 
 }  // namespace engine
 }  // namespace blimp