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

mojo/shell/tests/connect/connect_test_app.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 "base/bind.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "mojo/public/c/system/main.h"
 #include "mojo/public/cpp/bindings/binding_set.h"
@@ skipping 48 matching lines @@
     state->connection_remote_userid = connection->GetRemoteUserID();
     state->connection_remote_id = remote_id;
     state->initialize_local_name = name_;
     state->initialize_id = id_;
     state->initialize_userid = userid_;
     connection->GetInterface(&caller_);
     caller_->ConnectionAccepted(std::move(state));
 
     return true;
   }
-  bool ShellConnectionLost() override {
+  void ShellConnectionLost() override {
     if (base::MessageLoop::current() &&
         base::MessageLoop::current()->is_running()) {
       base::MessageLoop::current()->QuitWhenIdle();
     }
-    return true;
   }
 
   // InterfaceFactory<test::mojom::ConnectTestService>:
   void Create(Connection* connection,
               test::mojom::ConnectTestServiceRequest request) override {
     bindings_.AddBinding(this, std::move(request));
   }
 
   // InterfaceFactory<test::mojom::StandaloneApp>:
   void Create(Connection* connection,
@@ skipping 68 matching lines @@
 
 }  // namespace shell
 }  // namespace mojo
 
 
 MojoResult MojoMain(MojoHandle shell_handle) {
   MojoResult rv = mojo::ApplicationRunner(
       new mojo::shell::ConnectTestApp).Run(shell_handle);
   return rv;
 }