Fast path for strict closure creation.

src/bootstrapper.cc

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 546 matching lines @@
     Builtins::Name builtin) {
   // Create the ThrowTypeError function.
   Handle<String> name = Factory::LookupAsciiSymbol("ThrowTypeError");
   Handle<JSFunction> pill = Factory::NewFunctionWithoutPrototypeStrict(name);
   Handle<Code> code = Handle<Code>(Builtins::builtin(builtin));
   pill->set_map(global_context()->function_map_strict());
   pill->set_code(*code);
   pill->shared()->set_code(*code);
   pill->shared()->DontAdaptArguments();
 
+  PreventExtensions(pill);
+

[Martin Maly, 2011/03/15 03:08:57]

For some reason this felt cleaner, to freeze first and then install...

[Mads Ager (chromium), 2011/03/15 08:49:54]

I agree. :)

   // Install the poison pills into the calbacks array.
   callbacks->set(0, *pill);
   callbacks->set(1, *pill);
-
-  PreventExtensions(pill);
 }
 
 
 // ECMAScript 5th Edition, 13.2.3
 void Genesis::CreateThrowTypeError(Handle<JSFunction> empty) {
   // Create the pill callbacks arrays. The get/set callacks are installed
   // after the maps get created below.
   Handle<FixedArray> arguments = Factory::NewFixedArray(2, TENURED);
   Handle<FixedArray> caller = Factory::NewFixedArray(2, TENURED);
 
@@ skipping 1451 matching lines @@
 }
 
 
 // Restore statics that are thread local.
 char* BootstrapperActive::RestoreState(char* from) {
   nesting_ = *reinterpret_cast<int*>(from);
   return from + sizeof(nesting_);
 }
 
 } }  // namespace v8::internal