Factor stuff from ApplicationImpl out to a new class, ApplicationImplBase.

mojo/public/cpp/application/lib/application_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 "mojo/public/cpp/application/application_runner.h"
 
 #include "mojo/public/cpp/application/application_delegate.h"
 #include "mojo/public/cpp/application/application_impl.h"
+#include "mojo/public/cpp/application/application_impl_base.h"
 #include "mojo/public/cpp/environment/environment.h"
 #include "mojo/public/cpp/environment/logging.h"
 #include "mojo/public/cpp/utility/run_loop.h"
 
 namespace mojo {
 namespace {
 bool g_running = false;
 }  // namespace
 
 // static
-void ApplicationImpl::Terminate() {
+void ApplicationImplBase::Terminate() {

[vardhan, 2016/05/17 23:36:52]

feels weird that this is defined here.  is there a better way?

(i'm assuming we define it here because it uses ApplicationRunner's "managed"
RunLoop?)

[viettrungluu, 2016/05/17 23:42:50]

Yeah, but it'll require a bunch of work.

   RunLoop::current()->Quit();
 }
 
 ApplicationRunner::ApplicationRunner(
     std::unique_ptr<ApplicationDelegate> delegate)
     : delegate_(std::move(delegate)) {}
 
 ApplicationRunner::~ApplicationRunner() {
   assert(!delegate_);
 }
@@ skipping 25 matching lines @@
   }
 
   delegate_.reset();
 
   g_running = false;
 
   return MOJO_RESULT_OK;
 }
 
 }  // namespace mojo