Revision: 13818
Author:   [email protected]
Date:     Mon Mar  4 10:24:37 2013
Log:      Enable zapping of disposed global handles in release mode.

[email protected]
BUG=chromium:176056

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

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

=======================================
--- /branches/bleeding_edge/src/global-handles.cc       Mon Feb 18 02:20:58 2013
+++ /branches/bleeding_edge/src/global-handles.cc       Mon Mar  4 10:24:37 2013
@@ -113,10 +113,10 @@
   void Release(GlobalHandles* global_handles) {
     ASSERT(state() != FREE);
     set_state(FREE);
-    // TODO(176056): Enable as soon as WebKit bindings are fixed.
-#ifdef DEBUG_TODO
-    // Zap the values for eager trapping.
+ // TODO(mstarzinger): Put behind debug flag once embedders are stabilized.
     object_ = reinterpret_cast<Object*>(kGlobalHandleZapValue);
+#ifdef DEBUG
+    // Zap the values for eager trapping.
     class_id_ = v8::HeapProfiler::kPersistentHandleNoClassId;
     set_independent(false);
     set_partially_dependent(false);

--
--
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