Changes to pass the current isolate to all runtime and native calls.

runtime/vm/handles.h

 // 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.
 
 #ifndef VM_HANDLES_H_
 #define VM_HANDLES_H_
 
 #include "vm/allocation.h"
 
 namespace dart {
 
 // Handles are used in the Dart Virtual Machine to ensure that access
 // to dart objects in the virtual machine code is done in a
 // Garbage Collection safe manner.
 //
 // The class Handles is the basic type that implements creation of handles and
 // manages their life cycle (allocated either in the current zone or
 // current handle scope).
 // The two forms of handle allocation are:
 // - allocation of handles in the current zone (Handle::AllocateZoneHandle).
 //   Handles allocated in this manner are destroyed when the zone is destroyed.
 // - allocation of handles in a scoped manner (Handle::AllocateHandle).
 //   A new scope can be started using HANDLESCOPE().
 //   Handles allocated in this manner are destroyed when the HandleScope
 //   object is destroyed.
 // Code that uses scoped handles typically looks as follows:
 //   {
-//     HANDLESCOPE();
+//     HANDLESCOPE(isolate);
 //     const String& str = String::Handle(String::New("abc"));
 //     .....
 //     .....
 //   }
 // Code that uses zone handles typically looks as follows:
 //   const String& str = String::ZoneHandle(String::New("abc"));
 //   .....
 //   .....
 //
 //   The Handle function for each object type internally uses the
 //   Handles::AllocateHandle() function for creating handles. The Handle
 //   function of the object type is the only way to create scoped handles
 //   in the dart VM.
 //   The ZoneHandle function for each object type internally uses the
 //   Handles::AllocateZoneHandle() function for creating zone handles.
 //   The ZoneHandle function of the object type is the only way to create
 //   zone handles in the dart VM.
 //
 // There are some critical regions of the Dart VM were we may need to manipulate
 // raw dart objects directly. We use NOHANDLESCOPE to assert that we do not
 // add code that will allocate new handles during this critical area.
 // {
-//   NOHANDLESCOPE();
+//   NOHANDLESCOPE(isolate);
 //   ....
 //   ....
 // }
 
 
 // Forward declarations.
 class ObjectPointerVisitor;
 
 
 template <int kHandleSizeInWords, int kHandlesPerChunk, int kOffsetOfRawPtr>
@@ skipping 174 matching lines @@
 
   // Returns number of handles, these functions are used for testing purposes.
   static int ScopedHandleCount();
   static int ZoneHandleCount();
 };
 
 
 // The class HandleScope is used to start a new handles scope in the code.
 // It is used as follows:
 // {
-//   HANDLESCOPE();
+//   HANDLESCOPE(isolate);
 //   ....
 //   .....
 //   code that creates some scoped handles.
 //   ....
 // }
 class HandleScope : public StackResource {
  public:
-  HandleScope();
+  explicit HandleScope(Isolate* isolate);
   ~HandleScope();
 
  private:
   VMHandles::HandlesBlock* saved_handle_block_;  // Handle block at prev scope.
   uword saved_handle_slot_;  // Next available handle slot at previous scope.
 #if defined(DEBUG)
   HandleScope* link_;  // Link to previous scope.
 #endif
   DISALLOW_COPY_AND_ASSIGN(HandleScope);
 };
 
 // Macro to start a new Handle scope.
-#define HANDLESCOPE() dart::HandleScope vm_internal_handles_scope_;
+#define HANDLESCOPE(isolate)                                                  \
+    dart::HandleScope vm_internal_handles_scope_(isolate);
 
 
 // The class NoHandleScope is used in critical regions of the virtual machine
 // code where raw dart object pointers are directly manipulated.
 // This class asserts that we do not add code that will allocate new handles
 // during this critical area.
 // It is used as follows:
 // {
-//   NOHANDLESCOPE();
+//   NOHANDLESCOPE(isolate);
 //   ....
 //   .....
 //   critical code that manipulates dart objects directly.
 //   ....
 // }
 #if defined(DEBUG)
 class NoHandleScope : public StackResource {
  public:
-  NoHandleScope();
+  explicit NoHandleScope(Isolate* isolate);
   ~NoHandleScope();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NoHandleScope);
 };
 #else  // defined(DEBUG)
 class NoHandleScope : public ValueObject {
  public:
-  NoHandleScope() { }
+  explicit NoHandleScope(Isolate* isolate) { }
   ~NoHandleScope() { }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NoHandleScope);
 };
 #endif  // defined(DEBUG)
 
 // Macro to start a no handles scope in the code.
-#define NOHANDLESCOPE() dart::NoHandleScope no_vm_internal_handles_scope_;
+#define NOHANDLESCOPE(isolate)                                                \
+    dart::NoHandleScope no_vm_internal_handles_scope_(isolate);
 
 }  // namespace dart
 
 #endif  // VM_HANDLES_H_