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

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

[Vitaly Repeshko, 2011/11/15 20:06:01]

I'm a bit worried that by allocating handles we're creating new GC roots while
traversing GC roots. It's probably better to save raw pointers with
AssertNoAllocation first and then promote them to handles.

[Vitaly Repeshko, 2011/11/15 20:07:19]

Ah, never mind. We only traverse global handles here.

[mnaganov (inactive), 2011/11/17 17:46:55]

Added a test.

-  HandleScope scope;
   Handle<String> document_string =
       isolate->factory()->NewStringFromAscii(CStrVector("document"));
   Handle<String> url_string =