Index: src/bootstrapper.cc |
diff --git a/src/bootstrapper.cc b/src/bootstrapper.cc |
index a263fbce91b561cf2edf1c00375a5e7c6d8cd53a..1d1436dd87de4f2bd40dc8531a900c5d433e62e0 100644 |
--- a/src/bootstrapper.cc |
+++ b/src/bootstrapper.cc |
@@ -1052,6 +1052,15 @@ |
native_context()->set_extension(*global_object); |
native_context()->set_security_token(*global_object); |
+ // Replace outdated global objects in deserialized contexts. |
+ for (int i = 0; i < outdated_contexts->length(); ++i) { |
+ Context* context = Context::cast(outdated_contexts->get(i)); |
+ // Assert that there is only one native context. |
+ DCHECK(!context->IsNativeContext() || context == *native_context()); |
+ DCHECK_EQ(context->global_object(), *global_object_from_snapshot); |
+ context->set_global_object(*global_object); |
+ } |
+ |
TransferNamedProperties(global_object_from_snapshot, global_object); |
TransferIndexedProperties(global_object_from_snapshot, global_object); |
} |
@@ -1068,6 +1077,7 @@ |
native_context()->set_previous(NULL); |
// Set extension and global object. |
native_context()->set_extension(*global_object); |
+ native_context()->set_global_object(*global_object); |
// Security setup: Set the security token of the native context to the global |
// object. This makes the security check between two different contexts fail |
// by default even in case of global object reinitialization. |
@@ -1613,9 +1623,10 @@ |
DCHECK(context->IsNativeContext()); |
+ Handle<Context> runtime_context(context->runtime_context()); |
Handle<JSFunction> fun = |
isolate->factory()->NewFunctionFromSharedFunctionInfo(function_info, |
- context); |
+ runtime_context); |
Handle<Object> receiver = isolate->factory()->undefined_value(); |
// For non-extension scripts, run script to get the function wrapper. |
@@ -1742,6 +1753,12 @@ |
// The utils object can be removed for cases that reach this point. |
native_context()->set_natives_utils_object(heap()->undefined_value()); |
+ |
+#ifdef DEBUG |
+ JSGlobalObject* dummy = native_context()->runtime_context()->global_object(); |
+ DCHECK_EQ(0, dummy->elements()->length()); |
+ DCHECK_EQ(0, GlobalDictionary::cast(dummy->properties())->NumberOfElements()); |
+#endif |
} |
@@ -2225,6 +2242,31 @@ |
bool Genesis::InstallNatives(ContextType context_type) { |
HandleScope scope(isolate()); |
+ |
+ // Create a bridge function that has context in the native context. |
+ Handle<JSFunction> bridge = factory()->NewFunction(factory()->empty_string()); |
+ DCHECK(bridge->context() == *isolate()->native_context()); |
+ |
+ // Allocate the runtime context. |
+ { |
+ Handle<Context> context = |
+ factory()->NewFunctionContext(Context::MIN_CONTEXT_SLOTS, bridge); |
+ native_context()->set_runtime_context(*context); |
+ Handle<Code> code = isolate()->builtins()->Illegal(); |
+ Handle<JSFunction> global_fun = |
+ factory()->NewFunction(factory()->empty_string(), code, |
+ JS_GLOBAL_OBJECT_TYPE, JSGlobalObject::kSize); |
+ global_fun->initial_map()->set_dictionary_map(true); |
+ global_fun->initial_map()->set_prototype(heap()->null_value()); |
+ Handle<JSGlobalObject> dummy_global = |
+ Handle<JSGlobalObject>::cast(factory()->NewJSGlobalObject(global_fun)); |
+ dummy_global->set_native_context(*native_context()); |
+ dummy_global->set_global_proxy(native_context()->global_proxy()); |
+ context->set_global_object(*dummy_global); |
+ // Something went wrong if we actually need to write into the dummy global. |
+ dummy_global->set_properties(*GlobalDictionary::New(isolate(), 0)); |
+ dummy_global->set_elements(heap()->empty_fixed_array()); |
+ } |
// Set up the utils object as shared container between native scripts. |
Handle<JSObject> utils = factory()->NewJSObject(isolate()->object_function()); |
@@ -3149,8 +3191,6 @@ |
InitializeGlobal(global_object, empty_function, context_type); |
InitializeNormalizedMapCaches(); |
- // TODO(yangguo): Find a way to prevent accidentially installing properties |
- // on the global object. |
if (!InstallNatives(context_type)) return; |
MakeFunctionInstancePrototypeWritable(); |