Fix the ScopeIterator reimplemantation.

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index fc92adfb5439ea723dd447612dfcaa1caa5520d5..84bc36af33311f170cf520aecf955f4a1f91ac8d 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -492,14 +492,7 @@ Handle<SharedFunctionInfo> Compiler::Compile(Handle<String> source,
     }
     if (pre_data == NULL
         && source_length >= FLAG_min_preparse_length) {
-      if (source->IsExternalTwoByteString()) {
-        ExternalTwoByteStringUC16CharacterStream stream(
-            Handle<ExternalTwoByteString>::cast(source), 0, source->length());
-        pre_data = ParserApi::PartialPreParse(&stream, extension, flags);
-      } else {
-        GenericStringUC16CharacterStream stream(source, 0, source->length());
-        pre_data = ParserApi::PartialPreParse(&stream, extension, flags);
-      }
+      pre_data = ParserApi::PartialPreParse(source, extension, flags);

[Lasse Reichstein, 2011/11/28 11:51:36]

You can just skip creating preparser data now.
The parser will handle lazy functions effectively in the absence of pre_data now
(using the preparser's function-body-parser instead of its own). It'll be faster
than preparsing the entire script first, so preparser data is only an advantage
if it's been created offline (so you can skip the function body entirely).

     }
 
     // Create a script object describing the script to be compiled.