Mojo JS Bindings: fix the demos

services/js/echo_apptest.cc

Index: services/js/echo_apptest.cc
diff --git a/services/js/echo_apptest.cc b/services/js/echo_apptest.cc
index cccd9c6c200c6f6dc8c65686e6158d4347f72413..e3f078994af770c000dc5d5c71481d018949d081 100644
--- a/services/js/echo_apptest.cc
+++ b/services/js/echo_apptest.cc
@@ -39,16 +39,8 @@ class JSServiceProviderEchoTest : public test::JSApplicationTestBase {
   void SetUp() override {
     ApplicationTestBase::SetUp();
     const std::string& url = JSAppURL("echo.js");
-    // TODO(hansmuller): We should be able to pass a null ServiceProviderPtr
-    // here as the third parameter to ConnectToApplication since the parameter
-    // is annotated as nullable in the mojom, but the JS bindings currently
-    // don't correctly handle a null handle. So instead we allocate (and
-    // immediately close) a MessagePipe.
-    mojo::MessagePipe throwaway_pipe;
-    mojo::ServiceProviderPtr exposed_services;
-    exposed_services.Bind(throwaway_pipe.handle0.Pass());
     application_impl()->shell()->ConnectToApplication(
-        url, GetProxy(&echo_service_provider_), exposed_services.Pass());
+        url, GetProxy(&echo_service_provider_), nullptr);
   }
 
   mojo::ServiceProviderPtr echo_service_provider_;