Add a second kind of HandleScope that ties the lifetime of Handles created in its scope to the life…

src/handles.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 77 matching lines @@
 
   // Closes the given scope, but lets this handle escape. See
   // implementation in api.h.
   inline Handle<T> EscapeFrom(v8::HandleScope* scope);
 
  private:
   T** location_;
 };
 
 
+class HandleScopeImplementer;
+
+
 // A stack-allocated class that governs a number of local handles.
 // After a handle scope has been created, all local handles will be
 // allocated within that handle scope until either the handle scope is
 // deleted or another handle scope is created.  If there is already a
 // handle scope and a new one is created, all allocations will take
 // place in the new handle scope until it is deleted.  After that,
 // new handles will again be allocated in the original handle scope.
 //
 // After the handle scope of a local handle has been deleted the
 // garbage collector will no longer track the object stored in the
@@ skipping 42 matching lines @@
   Object** prev_next_;
   Object** prev_limit_;
 
   // Extend the handle scope making room for more handles.
   static internal::Object** Extend();
 
   // Zaps the handles in the half-open interval [start, end).
   static void ZapRange(internal::Object** start, internal::Object** end);
 
   friend class v8::HandleScope;
+  friend class v8::internal::HandleScopeImplementer;
   friend class v8::ImplementationUtilities;
 };
 
 
+class CompilationInfo;
+
+// Like a regular stack-allocated HandleScope, but does not delete the
+// handles allocated in its scope on destruction.  Instead, it ties
+// the Handles created in its scope with the CompilatationInfo passed
+// to its constructor.  The Handles are reclaimed when the
+// CompilationInfo destructs.
+class GatheringHandleScope {

[danno, 2012/06/24 11:07:03]

Why not just call this CompilationHandleScope?

[danno, 2012/06/24 11:07:03]

This guy should be responsible for creating the "PersistentExtension", not the
CompilationInfo itself. It should hand off the PersistentExtension to the
CompilationInfo in a "detach" operation.

+ public:
+  explicit GatheringHandleScope(CompilationInfo* info);
+  ~GatheringHandleScope();
+
+ private:
+  CompilationInfo* info_;
+  Object** prev_next_;
+  Object** prev_limit_;
+};
+
+
 // ----------------------------------------------------------------------------
 // Handle operations.
 // They might invoke garbage collection. The result is an handle to
 // an object of expected type, or the handle is an error if running out
 // of space or encountering an internal error.
 
 // Flattens a string.
 void FlattenString(Handle<String> str);
 
 // Flattens a string and returns the underlying external or sequential
@@ skipping 119 matching lines @@
   inline NoHandleAllocation();
   inline ~NoHandleAllocation();
  private:
   int level_;
 #endif
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_HANDLES_H_