Fix the ScopeIterator reimplemantation.

test/mjsunit/debug-stepout-scope.js

Index: test/mjsunit/debug-stepout-scope.js
diff --git a/test/mjsunit/debug-stepout-scope.js b/test/mjsunit/debug-stepout-scope.js
index b06ec1a83b38ae5cbbe737524b948ef66f375e0b..2e1f0de7c46287354cbbedc05d2bddb4156e471e 100644
--- a/test/mjsunit/debug-stepout-scope.js
+++ b/test/mjsunit/debug-stepout-scope.js
@@ -166,5 +166,15 @@ eval(pre + "1");
 eval(pre + "try { throw 'stuff' } catch (e) { e = 1; }");
 
 
+// The following function can be preparsed, but not parsed because
+// it contains an invalid break target. This is to make sure that
+// when a script is preparsed, the ScopeIterator will reparse it in
+// the same way.
+function lazy_parsed_but_never_called() {
+  while (true) {
+    break invalid_break_label;

[Lasse Reichstein, 2011/11/18 14:12:52]

Don't add tests like this. 
It should be an early error, so eventually the preparser will detect it and the
test will fail.
If anything, it should be in the bugs/ directory.

[Steven, 2011/11/18 16:53:19]

Moved it to the bugs/ directory.

On 2011/11/18 14:12:52, Lasse Reichstein wrote:

+  }
+}
+
 // With block as the last(!) statement in global code.
 with ({}) { debugger; }