Author: [email protected]
Date: Thu May  7 05:25:46 2009
New Revision: 1899

Modified:
    branches/bleeding_edge/src/factory.cc

Log:
Allocate proxy before the script to allow heap verification
to check that all scripts have a proxy in the wrapper field.
Review URL: http://codereview.chromium.org/115081

Modified: branches/bleeding_edge/src/factory.cc
==============================================================================
--- branches/bleeding_edge/src/factory.cc       (original)
+++ branches/bleeding_edge/src/factory.cc       Thu May  7 05:25:46 2009
@@ -167,6 +167,7 @@
    Heap::SetLastScriptId(Smi::FromInt(id));

    // Create and initialize script object.
+  Handle<Proxy> wrapper = Factory::NewProxy(0, TENURED);
    Handle<Script> script = Handle<Script>::cast(NewStruct(SCRIPT_TYPE));
    script->set_source(*source);
    script->set_name(Heap::undefined_value());
@@ -176,7 +177,7 @@
    script->set_data(Heap::undefined_value());
    script->set_context_data(Heap::undefined_value());
    script->set_type(Smi::FromInt(SCRIPT_TYPE_NORMAL));
-  script->set_wrapper(*Factory::NewProxy(0, TENURED));
+  script->set_wrapper(*wrapper);
    script->set_line_ends(Heap::undefined_value());

    return script;

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

Reply via email to