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

Unified Diff: services/window_manager/main.cc

Issue 858103002: Remove [Client=] annotation from ServiceProvider (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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/window_manager/main.cc
diff --git a/services/window_manager/main.cc b/services/window_manager/main.cc
index 0a6a0241c3a101ae1d444ecfe96789257f83e609..acb02a8384f798c587cdebab67bed17dc706ad9a 100644
--- a/services/window_manager/main.cc
+++ b/services/window_manager/main.cc
@@ -45,16 +45,16 @@ class DefaultWindowManager : public mojo::ApplicationDelegate,
// Overridden from ViewManagerDelegate:
void OnEmbed(View* root,
- mojo::ServiceProviderImpl* exported_services,
- scoped_ptr<mojo::ServiceProvider> imported_services) override {
+ mojo::InterfaceRequest<mojo::ServiceProvider> services,
+ mojo::ServiceProviderPtr exposed_services) override {
root_ = root;
}
void OnViewManagerDisconnected(ViewManager* view_manager) override {}
// Overridden from WindowManagerDelegate:
- void Embed(
- const mojo::String& url,
- mojo::InterfaceRequest<mojo::ServiceProvider> service_provider) override {
+ void Embed(const mojo::String& url,
+ mojo::InterfaceRequest<mojo::ServiceProvider> services,
+ mojo::ServiceProviderPtr exposed_services) override {
DCHECK(root_);
View* view = root_->view_manager()->CreateView();
root_->AddChild(view);
@@ -66,7 +66,7 @@ class DefaultWindowManager : public mojo::ApplicationDelegate,
window_offset_ += 10;
view->SetVisible(true);
- view->Embed(url, service_provider.Pass());
+ view->Embed(url, services.Pass(), exposed_services.Pass());
}
scoped_ptr<WindowManagerApp> window_manager_app_;

Powered by Google App Engine
This is Rietveld 408576698