Use the ThreadPool for all isolates and native ports. Previously,

runtime/vm/isolate.h

 // Copyright (c) 2012, 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.
 
 #ifndef VM_ISOLATE_H_
 #define VM_ISOLATE_H_
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "platform/thread.h"
@@ skipping 15 matching lines @@
 class MessageHandler;
 class Mutex;
 class ObjectPointerVisitor;
 class ObjectStore;
 class RawContext;
 class RawError;
 class StackResource;
 class StubCode;
 class Zone;
 
-
 class Isolate : public BaseIsolate {
  public:
   ~Isolate();
 
   static inline Isolate* Current() {
     return reinterpret_cast<Isolate*>(Thread::GetThreadLocal(isolate_key));
   }
 
   static void SetCurrent(Isolate* isolate);
 
@@ skipping 90 matching lines @@
   void SetStackLimitFromCurrentTOS(uword isolate_stack_top);
 
   uword stack_limit_address() const {
     return reinterpret_cast<uword>(&stack_limit_);
   }
 
   // The current stack limit.  This may be overwritten with a special
   // value to trigger interrupts.
   uword stack_limit() const { return stack_limit_; }
 
-  // The true stack limit for this isolate.  This does not change
-  // after isolate initialization.
+  // The true stack limit for this isolate.
   uword saved_stack_limit() const { return saved_stack_limit_; }
 
   enum {
     kApiInterrupt = 0x1,      // An interrupt from Dart_InterruptIsolate.
     kMessageInterrupt = 0x2,  // An interrupt to process an out of band message.
 
     kInterruptsMask = kApiInterrupt | kMessageInterrupt,
   };
 
   void ScheduleInterrupts(uword interrupt_bits);
   uword GetAndClearInterrupts();
 
   MessageHandler* message_handler() const { return message_handler_; }
   void set_message_handler(MessageHandler* value) { message_handler_ = value; }
 
-  // Returns null on success, a RawError on failure.
-  RawError* StandardRunLoop();
-
   intptr_t ast_node_id() const { return ast_node_id_; }
   void set_ast_node_id(int value) { ast_node_id_ = value; }
 
   Debugger* debugger() const { return debugger_; }
 
   static void SetCreateCallback(Dart_IsolateCreateCallback cback);
   static Dart_IsolateCreateCallback CreateCallback();
 
   static void SetInterruptCallback(Dart_IsolateInterruptCallback cback);
   static Dart_IsolateInterruptCallback InterruptCallback();
@@ skipping 42 matching lines @@
   MessageHandler* message_handler_;
   GcPrologueCallbacks gc_prologue_callbacks_;
   GcEpilogueCallbacks gc_epilogue_callbacks_;
 
   static Dart_IsolateCreateCallback create_callback_;
   static Dart_IsolateInterruptCallback interrupt_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(Isolate);
 };
 
+class SetIsolateScope {

[siva, 2012/04/14 00:29:53]

In two out of the three places where this is used the saved_isolate_ must be
NULL (I think there should be an assertion for that).

[turnidge, 2012/04/17 23:46:55]

Added a new StartIsolateScope for those cases.

+ public:
+  explicit SetIsolateScope(Isolate* new_isolate)
+      : new_isolate_(new_isolate), saved_isolate_(Isolate::Current()) {
+    Isolate::SetCurrent(new_isolate);
+    if (new_isolate != NULL) {
+      new_isolate->SetStackLimitFromCurrentTOS(reinterpret_cast<uword>(this));
+    }
+  }
+
+  ~SetIsolateScope() {
+    ASSERT(Isolate::Current() == new_isolate_);
+    Isolate::SetCurrent(saved_isolate_);
+  }
+
+ private:
+  Isolate* new_isolate_;
+  Isolate* saved_isolate_;
+
+  DISALLOW_COPY_AND_ASSIGN(SetIsolateScope);
+};
+
 }  // namespace dart
 
 #endif  // VM_ISOLATE_H_