Reviewers: Vitaly Repeshko,

Description:
Really fix missing HandleScope to prevent local handles to DOMWindow leaking.

A follow-up to r9994.

[email protected]
BUG=102895
TEST=none


Please review this at http://codereview.chromium.org/8468016/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/profile-generator.cc


Index: src/profile-generator.cc
diff --git a/src/profile-generator.cc b/src/profile-generator.cc
index 0284b98d72a5016e17c5a356cfe43ad095459de7..5626acaba4bf7fc37c75921767c12c77a0cff172 100644
--- a/src/profile-generator.cc
+++ b/src/profile-generator.cc
@@ -2470,10 +2470,10 @@ class GlobalObjectsEnumerator : public ObjectVisitor {

 // Modifies heap. Must not be run during heap traversal.
 void V8HeapExplorer::TagGlobalObjects() {
+  HandleScope scope;
   Isolate* isolate = Isolate::Current();
   GlobalObjectsEnumerator enumerator;
   isolate->global_handles()->IterateAllRoots(&enumerator);
-  HandleScope scope;
   Handle<String> document_string =
       isolate->factory()->NewStringFromAscii(CStrVector("document"));
   Handle<String> url_string =


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to