Revert "Avoid to run dart code during paused_on_exit"

runtime/vm/message_handler.cc

 // Copyright (c) 2011, 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/message_handler.h"
 
 #include "vm/dart.h"
 #include "vm/lockers.h"
 #include "vm/object.h"
 #include "vm/object_store.h"
@@ skipping 328 matching lines @@
       status = HandleMessages(&ml, false, false);
       if (ShouldPauseOnStart(status)) {
         // Still paused.
         ASSERT(oob_queue_->IsEmpty());
         task_ = NULL;  // No task in queue.
         return;
       } else {
         PausedOnStartLocked(&ml, false);
       }
     }
-    if (is_paused_on_exit()) {
-      status = HandleMessages(&ml, false, false);
-      if (ShouldPauseOnExit(status)) {
-        // Still paused.
-        ASSERT(oob_queue_->IsEmpty());
-        task_ = NULL;  // No task in queue.
-        return;
-      } else {
-        PausedOnExitLocked(&ml, false);
-      }
-    }
-#endif  // !defined(PRODUCT)
+#endif
 
     if (status == kOK) {
       if (start_callback_) {
         // Initialize the message handler by running its start function,
         // if we have one.  For an isolate, this will run the isolate's
         // main() function.
         //
         // Release the monitor_ temporarily while we call the start callback.
         ml.Exit();
         status = start_callback_(callback_data_);
         ASSERT(Isolate::Current() == NULL);
         start_callback_ = NULL;
         ml.Enter();
       }
 
       // Handle any pending messages for this message handler.
       if (status != kShutdown) {
         status = HandleMessages(&ml, (status == kOK), true);
       }
     }
 
     // The isolate exits when it encounters an error or when it no
     // longer has live ports.
     if (status != kOK || !HasLivePorts()) {
 #if !defined(PRODUCT)
       if (ShouldPauseOnExit(status)) {
-        if (FLAG_trace_service_pause_events) {
-          OS::PrintErr(
-              "Isolate %s paused before exiting. "
-              "Use the Observatory to release it.\n",
-              name());
+        if (!is_paused_on_exit()) {
+          if (FLAG_trace_service_pause_events) {
+            OS::PrintErr(
+                "Isolate %s paused before exiting. "
+                "Use the Observatory to release it.\n",
+                name());
+          }
+          PausedOnExitLocked(&ml, true);
+          // More messages may have come in while we released the monitor.
+          status = HandleMessages(&ml, false, false);
         }
-        PausedOnExitLocked(&ml, true);
-        // More messages may have come in while we released the monitor.
-        status = HandleMessages(&ml, false, false);
         if (ShouldPauseOnExit(status)) {
           // Still paused.
           ASSERT(oob_queue_->IsEmpty());
           task_ = NULL;  // No task in queue.
           return;
         } else {
           PausedOnExitLocked(&ml, false);
         }
       }
 #endif  // !defined(PRODUCT)
@@ skipping 173 matching lines @@
   ASSERT(handler != NULL);
   handler_->oob_message_handling_allowed_ = false;
 }
 
 MessageHandler::AcquiredQueues::~AcquiredQueues() {
   ASSERT(handler_ != NULL);
   handler_->oob_message_handling_allowed_ = true;
 }
 
 }  // namespace dart