Revert of Quit the message loop by default in ShellConnectionLost when ApplicationRunner is used

mojo/shell/tests/connect/connect_test_driver.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/at_exit.h"
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/process/process.h"
 #include "mojo/public/cpp/bindings/binding_set.h"
 #include "mojo/shell/public/cpp/connection.h"
@@ skipping 19 matching lines @@
  private:
   // mojo::ShellClient:
   void Initialize(mojo::Connector* connector, const mojo::Identity& identity,
                   uint32_t id) override {
     connector_ = connector;
   }
   bool AcceptConnection(mojo::Connection* connection) override {
     connection->AddInterface<ClientProcessTest>(this);
     return true;
   }
-  bool ShellConnectionLost() override {
-    // TODO(rockot): http://crbug.com/596621. Should be able to remove this
-    // override entirely.
+  void ShellConnectionLost() override {
+    // TODO: This should exit cleanly.
     _exit(1);
   }
 
   // mojo::InterfaceFactory<ConnectTestService>:
   void Create(mojo::Connection* connection,
               ClientProcessTestRequest request) override {
     bindings_.AddBinding(this, std::move(request));
   }
 
   // test::mojom::ClientProcessTest:
@@ skipping 26 matching lines @@
 
 int main(int argc, char** argv) {
   base::AtExitManager at_exit;
   base::CommandLine::Init(argc, argv);
 
   mojo::shell::InitializeLogging();
 
   Driver driver;
   return mojo::shell::TestNativeMain(&driver);
 }