Update view_manager and window_manager to make use of content handling.

mojo/examples/wm_flow/app/app.cc

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 d2b3971efd647a58a4cbede44de1366c31866b92..5b6dcf4d2366a36b222c98b81b54e56c1a77b22a 100644
--- a/mojo/examples/wm_flow/app/app.cc
+++ b/mojo/examples/wm_flow/app/app.cc
@@ -53,13 +53,14 @@ class WMFlowApp : public mojo::ApplicationDelegate,
  public:
   WMFlowApp()
       : embed_count_(0),
-        view_manager_client_factory_(this),
         app_(NULL) {}
   virtual ~WMFlowApp() {}
 
  private:
   // Overridden from Application:
   virtual void Initialize(mojo::ApplicationImpl* app) MOJO_OVERRIDE {
+    view_manager_client_factory_.reset(

[darin (slow to review), 2014/09/04 21:33:51]

it seems a bit unfortunate that we have to lazily allocate
ViewManagerClientFactory. i suppose an alternative would have been a setter for
the app->shell()?

  virtual void Initialize(mojo::ApplicationImpl* app) MOJO_OVERRIDE {
    view_manager_client_factory_.Initialize(app);
  }

had you considered something like that?

[Aaron Boodman, 2014/09/04 22:31:43]

I can do that if you like. I assumed it was a design goal that the factories
were never partially init'd. ViewManagerClientFactory is totally useless without
a |shell|.

+        new mojo::ViewManagerClientFactory(app->shell(), this));
     app_ = app;
     OpenNewWindow();
     OpenNewWindow();
@@ -67,7 +68,7 @@ class WMFlowApp : public mojo::ApplicationDelegate,
   }
   virtual bool ConfigureIncomingConnection(
       mojo::ApplicationConnection* connection) MOJO_OVERRIDE {
-    connection->AddService(&view_manager_client_factory_);
+    connection->AddService(view_manager_client_factory_.get());
     return true;
   }
 
@@ -127,7 +128,7 @@ class WMFlowApp : public mojo::ApplicationDelegate,
   }
 
   int embed_count_;
-  mojo::ViewManagerClientFactory view_manager_client_factory_;
+  scoped_ptr<mojo::ViewManagerClientFactory> view_manager_client_factory_;
   mojo::InterfaceFactoryImpl<EmbedderImpl> embedder_factory_;
   EmbeddeePtr embeddee_;
   mojo::ApplicationImpl* app_;