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

Unified Diff: mojo/examples/wm_flow/app/app.cc

Issue 463523003: Fix an issue with view manager init service where you had to retain a connection to the init servic… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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: mojo/examples/wm_flow/app/app.cc
diff --git a/mojo/examples/wm_flow/app/app.cc b/mojo/examples/wm_flow/app/app.cc
index 9edd47e84913860196b0d5fb1c323ba7391837db..44fbd6481cc1dfdcdc95bcb71751c742d51575ce 100644
--- a/mojo/examples/wm_flow/app/app.cc
+++ b/mojo/examples/wm_flow/app/app.cc
@@ -56,9 +56,10 @@ class WMFlowApp : public mojo::ApplicationDelegate,
// Overridden from Application:
virtual void Initialize(mojo::ApplicationImpl* app) MOJO_OVERRIDE {
mojo::ServiceProviderPtr sp;
- app->ConnectToService("mojo:mojo_view_manager", &init_svc_);
- init_svc_->Embed("mojo:mojo_wm_flow_app", sp.Pass(),
- base::Bind(&ConnectCallback));
+ mojo::ViewManagerInitServicePtr init_svc;
+ app->ConnectToService("mojo:mojo_view_manager", &init_svc);
+ init_svc->Embed("mojo:mojo_wm_flow_app", sp.Pass(),
+ base::Bind(&ConnectCallback));
}
virtual bool ConfigureIncomingConnection(
mojo::ApplicationConnection* connection) MOJO_OVERRIDE {
@@ -103,7 +104,6 @@ class WMFlowApp : public mojo::ApplicationDelegate,
mojo::ViewManagerClientFactory view_manager_client_factory_;
mojo::InterfaceFactoryImpl<EmbedderImpl> embedder_factory_;
EmbeddeePtr embeddee_;
- mojo::ViewManagerInitServicePtr init_svc_;
DISALLOW_COPY_AND_ASSIGN(WMFlowApp);
};

Powered by Google App Engine
This is Rietveld 408576698