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

Unified Diff: services/service_manager/public/cpp/interface_registry.h

Issue 2456493003: Add frame-specific InterfaceProviderSpec. (Closed)
Patch Set: . Created 4 years, 2 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
Index: services/service_manager/public/cpp/interface_registry.h
diff --git a/services/service_manager/public/cpp/interface_registry.h b/services/service_manager/public/cpp/interface_registry.h
index f3fc799424b4d38a38d54cfefb0031618bf9ee79..e63f16834bd4605982edbad2b1b7225189c494e7 100644
--- a/services/service_manager/public/cpp/interface_registry.h
+++ b/services/service_manager/public/cpp/interface_registry.h
@@ -76,11 +76,9 @@ class InterfaceRegistry : public mojom::InterfaceProvider {
DISALLOW_COPY_AND_ASSIGN(TestApi);
};
- // Construct an unbound InterfaceRegistry for the service |identity| that
- // exposes interfaces in accordance to |interface_provider_spec|. This object
- // will not bind requests for interfaces until Bind() is called.
- InterfaceRegistry(const Identity& identity,
- const InterfaceProviderSpec& interface_provider_spec);
+ // Construct an unbound InterfaceRegistry. This object will not bind requests
+ // for interfaces until Bind() is called.
+ explicit InterfaceRegistry(const std::string& name);
~InterfaceRegistry() override;
// Sets a default handler for incoming interface requests which are allowed by
@@ -92,6 +90,8 @@ class InterfaceRegistry : public mojom::InterfaceProvider {
// InterfaceProviderSpec, which will be intersected with the local's exports
// to determine what interfaces may be bound.
void Bind(mojom::InterfaceProviderRequest request,
+ const Identity& local_identity,
+ const InterfaceProviderSpec& local_interface_provider_spec,
const Identity& remote_identity,
const InterfaceProviderSpec& remote_interface_provider_spec);
@@ -166,8 +166,9 @@ class InterfaceRegistry : public mojom::InterfaceProvider {
mojom::InterfaceProviderRequest pending_request_;
mojo::Binding<mojom::InterfaceProvider> binding_;
- const Identity identity_;
- const InterfaceProviderSpec interface_provider_spec_;
+ Identity identity_;
+ InterfaceProviderSpec interface_provider_spec_;
+ std::string name_;
// Metadata computed when Bind() is called:
Identity remote_identity_;

Powered by Google App Engine
This is Rietveld 408576698