Fix the ScopeIterator reimplemantation.

test/mjsunit/debug-stepout-scope.js

 // Copyright 2011 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 148 matching lines @@
 eval(pre + "with ({}) {}");
 eval(pre + "with ({x:1}) x");
 eval(pre + "with ({x:1}) x = 1");
 eval(pre + "try { throw 'stuff' } catch (e) { e = 1; }");
 
 pre = "'use strict'; debugger; "
 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; }