Simple multi-url support for mojo apps

examples/forwarding_content_handler/forwarding_content_handler.cc

Index: examples/forwarding_content_handler/forwarding_content_handler.cc
diff --git a/examples/forwarding_content_handler/forwarding_content_handler.cc b/examples/forwarding_content_handler/forwarding_content_handler.cc
index 6f4275bba75401f849956743c814c8482fb40e0b..5c75320e3a726bac49ae76a567a7034f5cf24f1e 100644
--- a/examples/forwarding_content_handler/forwarding_content_handler.cc
+++ b/examples/forwarding_content_handler/forwarding_content_handler.cc
@@ -23,9 +23,7 @@ class ForwardingApplicationImpl : public Application {
  public:
   ForwardingApplicationImpl(InterfaceRequest<Application> request,
                             std::string target_url)

[qsr, 2015/02/27 15:01:40]

I don't understand this change.

This content handler expect to read a txt file that contains the URL of the app
to connect to.

You seems to have just ignore this parameter altogether.

[Aaron Boodman, 2015/02/27 16:54:44]

Whoopsie. fixed.

-      : binding_(this, request.Pass()),
-        target_url_(target_url) {
-  }
+      : binding_(this, request.Pass()) {}
 
  private:
   // Application:
@@ -36,16 +34,15 @@ class ForwardingApplicationImpl : public Application {
   }
   void AcceptConnection(const String& requestor_url,
                         InterfaceRequest<ServiceProvider> services,
-                        ServiceProviderPtr exposed_services) override {
-    shell_->ConnectToApplication(target_url_, services.Pass(),
-                                  exposed_services.Pass());
+                        ServiceProviderPtr exposed_services,
+                        const String& url) override {
+    shell_->ConnectToApplication(url, services.Pass(), exposed_services.Pass());
   }
   void RequestQuit() override {
     RunLoop::current()->Quit();
   }
 
   Binding<Application> binding_;
-  std::string target_url_;
   ShellPtr shell_;
 };