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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2795883002: Eliminate OnConnect usage (Closed)
Patch Set: . Created 3 years, 8 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: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 40ef7bfe264b14d3160b618bf8409b197b5dae1e..bdcae644c2777605a83c9fdd4b461bbe6eb8747b 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -504,8 +504,7 @@ class ChromeServiceChromeOS
public mash::mojom::Launchable,
public service_manager::InterfaceFactory<mash::mojom::Launchable> {
public:
- ChromeServiceChromeOS()
- : interfaces_(service_manager::mojom::kServiceManager_ConnectorSpec) {
+ ChromeServiceChromeOS() {
interfaces_.AddInterface<mash::mojom::Launchable>(this);
}
~ChromeServiceChromeOS() override {}
@@ -525,7 +524,7 @@ class ChromeServiceChromeOS
void OnBindInterface(const service_manager::ServiceInfo& remote_info,
const std::string& name,
mojo::ScopedMessagePipeHandle handle) override {
- interfaces_.BindInterface(name, std::move(handle));
+ interfaces_.BindInterface(remote_info.identity, name, std::move(handle));
}
// mash::mojom::Launchable:
@@ -552,7 +551,7 @@ class ChromeServiceChromeOS
bindings_.AddBinding(this, std::move(request));
}
- service_manager::InterfaceRegistry interfaces_;
+ service_manager::BinderRegistry interfaces_;
mojo::BindingSet<mash::mojom::Launchable> bindings_;
DISALLOW_COPY_AND_ASSIGN(ChromeServiceChromeOS);

Powered by Google App Engine
This is Rietveld 408576698