Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

services/catalog/catalog.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 "services/catalog/catalog.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/ptr_util.h"
@@ skipping 93 matching lines @@
                                   weak_factory_.GetWeakPtr()));
 }
 
 bool Catalog::OnConnect(shell::Connection* connection) {
   connection->AddInterface<mojom::Catalog>(this);
   connection->AddInterface<filesystem::mojom::Directory>(this);
   connection->AddInterface<shell::mojom::Resolver>(this);
   return true;
 }
 
-void Catalog::Create(shell::Connection* connection,
+void Catalog::Create(const shell::Identity& remote_identity,
                      shell::mojom::ResolverRequest request) {
-  Instance* instance =
-      GetInstanceForUserId(connection->GetRemoteIdentity().user_id());
+  Instance* instance = GetInstanceForUserId(remote_identity.user_id());
   instance->BindResolver(std::move(request));
 }
 
-void Catalog::Create(shell::Connection* connection,
+void Catalog::Create(const shell::Identity& remote_identity,
                      mojom::CatalogRequest request) {
-  Instance* instance =
-      GetInstanceForUserId(connection->GetRemoteIdentity().user_id());
+  Instance* instance = GetInstanceForUserId(remote_identity.user_id());
   instance->BindCatalog(std::move(request));
 }
 
-void Catalog::Create(shell::Connection* connection,
+void Catalog::Create(const shell::Identity& remote_identity,
                      filesystem::mojom::DirectoryRequest request) {
   if (!lock_table_)
     lock_table_ = new filesystem::LockTable;
   base::FilePath resources_path =
-      GetPathForApplicationName(connection->GetRemoteIdentity().name());
+      GetPathForApplicationName(remote_identity.name());
   new filesystem::DirectoryImpl(std::move(request), resources_path,
                                 scoped_refptr<filesystem::SharedTempDir>(),
                                 lock_table_);
 }
 
 Instance* Catalog::GetInstanceForUserId(const std::string& user_id) {
   auto it = instances_.find(user_id);
   if (it != instances_.end())
     return it->second.get();
 
   // TODO(beng): There needs to be a way to load the store from different users.
   Instance* instance = new Instance(std::move(store_), system_reader_.get());
   instances_[user_id] = base::WrapUnique(instance);
   if (loaded_)
     instance->CacheReady(&system_cache_);
 
   return instance;
 }
 
 void Catalog::SystemPackageDirScanned() {
   loaded_ = true;
   for (auto& instance : instances_)
     instance.second->CacheReady(&system_cache_);
 }
 
 }  // namespace catalog