Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(785)

Unified Diff: services/catalog/catalog.cc

Issue 2182643003: Remove shell::Connection* parameter to InterfaceFactory<T>::Create() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « services/catalog/catalog.h ('k') | services/navigation/navigation.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/catalog/catalog.cc
diff --git a/services/catalog/catalog.cc b/services/catalog/catalog.cc
index 8f55425d722cf3d97feb912a8844fb4a413af428..53efd082b01008aa933ead0428d6cc4740485692 100644
--- a/services/catalog/catalog.cc
+++ b/services/catalog/catalog.cc
@@ -111,26 +111,24 @@ bool Catalog::OnConnect(shell::Connection* connection) {
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_);
« no previous file with comments | « services/catalog/catalog.h ('k') | services/navigation/navigation.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698