Reverts VM thread cleanup

runtime/vm/service_isolate.cc

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/service_isolate.h"
 
 #include "vm/compiler.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_entry.h"
 #include "vm/isolate.h"
@@ skipping 731 matching lines @@
   ASSERT(monitor_ == NULL);
   monitor_ = new Monitor();
   ASSERT(monitor_ != NULL);
   // Grab the isolate create callback here to avoid race conditions with tests
   // that change this after Dart_Initialize returns.
   create_callback_ = Isolate::CreateCallback();
   Dart::thread_pool()->Run(new RunServiceTask());
 }
 
 
-void ServiceIsolate::KillServiceIsolate() {
-  {
-    MonitorLocker ml(monitor_);
-    shutting_down_ = true;
-  }
-  Isolate::KillIsolate(isolate_);
-  {
-    MonitorLocker ml(monitor_);
-    while (shutting_down_) {
-      ml.Wait();
-    }
-  }
-}
-
-
 void ServiceIsolate::Shutdown() {
   if (!IsRunning()) {
-    if (isolate_ != NULL) {
-      // TODO(johnmccutchan,turnidge) When it is possible to properly create
-      // the VMService object and set up its shutdown handler in the service
-      // isolate's main() function, this case will no longer be possible and
-      // can be removed.
-      KillServiceIsolate();
-    }
     return;
   }
   {
     MonitorLocker ml(monitor_);
     shutting_down_ = true;
   }
   SendServiceExitMessage();
   {
     MonitorLocker ml(monitor_);
     while (shutting_down_ && (port_ != ILLEGAL_PORT)) {
@@ skipping 43 matching lines @@
     return result;
   }
   Dart_Handle source = GetSource(url_string);
   if (Dart_IsError(source)) {
     return source;
   }
   return Dart_LoadSource(library, url, source, 0, 0);
 }
 
 }  // namespace dart