Fix shutdown crash by ensuring valid MessageLoop

services/service_manager/public/cpp/lib/service_runner.cc

 // Copyright 2014 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 "services/service_manager/public/cpp/service_runner.h"
 
 #include "base/at_exit.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
@@ skipping 41 matching lines @@
     std::unique_ptr<base::MessageLoop> loop;
     loop.reset(new base::MessageLoop(message_loop_type_));
 
     context_.reset(new ServiceContext(
         std::move(service_),
         mojo::MakeRequest<mojom::Service>(mojo::MakeScopedHandle(
             mojo::MessagePipeHandle(service_request_handle)))));
     base::RunLoop run_loop;
     context_->SetConnectionLostClosure(run_loop.QuitClosure());
     run_loop.Run();
-    // It's very common for the service to cache the app and terminate on
-    // errors. If we don't delete the service before the app we run the risk of
-    // the service having a stale reference to the app and trying to use it.
-    // Note that we destruct the message loop first because that might trigger
-    // connection error handlers and they might access objects created by the
-    // service.
-    loop.reset();
     context_.reset();
   }
   return MOJO_RESULT_OK;
 }
 
 MojoResult ServiceRunner::Run(MojoHandle service_request_handle) {
   return Run(service_request_handle, false);
 }
 
 void ServiceRunner::Quit() {
   base::MessageLoop::current()->QuitWhenIdle();
 }
 
 }  // namespace service_manager