Revision: 9994
Author: [email protected]
Date: Tue Nov 15 02:10:35 2011
Log: Fix missing HandleScope to prevent local handles to DOMWindow
leaking.
[email protected]
BUG=102895
TEST=none
Review URL: http://codereview.chromium.org/8528024
http://code.google.com/p/v8/source/detail?r=9994
Modified:
/branches/bleeding_edge/src/profile-generator.cc
=======================================
--- /branches/bleeding_edge/src/profile-generator.cc Mon Nov 14 03:13:29
2011
+++ /branches/bleeding_edge/src/profile-generator.cc Tue Nov 15 02:10:35
2011
@@ -2473,6 +2473,7 @@
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 =
@@ -2480,6 +2481,7 @@
const char** urls = NewArray<const char*>(enumerator.count());
for (int i = 0, l = enumerator.count(); i < l; ++i) {
urls[i] = NULL;
+ HandleScope scope;
Handle<JSGlobalObject> global_obj = enumerator.at(i);
Object* obj_document;
if (global_obj->GetProperty(*document_string)->ToObject(&obj_document)
&&
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
