Revision: 15860
Author:   [email protected]
Date:     Wed Jul 24 05:50:53 2013
Log:      wrap gc callbacks in handlescopes

[email protected]
BUG=

Review URL: https://codereview.chromium.org/19845007
http://code.google.com/p/v8/source/detail?r=15860

Modified:
 /branches/bleeding_edge/src/global-handles.cc
 /branches/bleeding_edge/src/heap.cc

=======================================
--- /branches/bleeding_edge/src/global-handles.cc       Mon Jul  8 08:28:25 2013
+++ /branches/bleeding_edge/src/global-handles.cc       Wed Jul 24 05:50:53 2013
@@ -262,6 +262,7 @@
              ExternalTwoByteString::cast(object_)->resource() != NULL);
       // Leaving V8.
       VMState<EXTERNAL> state(isolate);
+      HandleScope handle_scope(isolate);
       weak_reference_callback_(reinterpret_cast<v8::Isolate*>(isolate),
reinterpret_cast<Persistent<Value>*>(&object),
                                par);
=======================================
--- /branches/bleeding_edge/src/heap.cc Wed Jul 24 00:45:54 2013
+++ /branches/bleeding_edge/src/heap.cc Wed Jul 24 05:50:53 2013
@@ -920,6 +920,7 @@
   {
     GCTracer::Scope scope(tracer, GCTracer::Scope::EXTERNAL);
     VMState<EXTERNAL> state(isolate_);
+    HandleScope handle_scope(isolate_);
     CallGCPrologueCallbacks(gc_type, kNoGCCallbackFlags);
   }

@@ -1025,6 +1026,7 @@
   {
     GCTracer::Scope scope(tracer, GCTracer::Scope::EXTERNAL);
     VMState<EXTERNAL> state(isolate_);
+    HandleScope handle_scope(isolate_);
     CallGCEpilogueCallbacks(gc_type);
   }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to