Monitor the ShellClient binding for pipe closure as a signal to shut down the ShellConnection, rath…

mojo/shell/tests/connect/connect_test_package.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 130 matching lines @@
                   uint32_t id, uint32_t user_id) override {
     bindings_.set_connection_error_handler(
         base::Bind(&ConnectTestShellClient::OnConnectionError,
                    base::Unretained(this)));
   }
   bool AcceptConnection(Connection* connection) override {
     connection->AddInterface<ShellClientFactory>(this);
     connection->AddInterface<test::mojom::ConnectTestService>(this);
     return true;
   }
-  bool ShellConnectionLost() override {
+  void ShellConnectionLost() override {
     if (base::MessageLoop::current() &&
         base::MessageLoop::current()->is_running()) {
       base::MessageLoop::current()->QuitWhenIdle();
     }
-    return true;
   }
 
   // InterfaceFactory<mojom::ShellClientFactory>:
   void Create(Connection* connection,
               mojom::ShellClientFactoryRequest request) override {
     shell_client_factory_bindings_.AddBinding(this, std::move(request));
   }
 
   // InterfaceFactory<test::mojom::ConnectTestService>:
   void Create(Connection* connection,
@@ skipping 29 matching lines @@
 
 }  // namespace shell
 }  // namespace mojo
 
 
 MojoResult MojoMain(MojoHandle shell_handle) {
   MojoResult rv = mojo::ApplicationRunner(
       new mojo::shell::ConnectTestShellClient).Run(shell_handle);
   return rv;
 }