Set debug break slot at init of loop variable in a for loop.

Set debug break slot at init of loop variable in a for loop.


BUG=102153
TEST=regress-102153.js

Committed: https://code.google.com/p/v8/source/detail?r=10963

[Yang, 2012-03-07 18:28:35]

PTAL.

http://codereview.chromium.org/9625011/diff/6/src/full-codegen.cc
File src/full-codegen.cc (right):

http://codereview.chromium.org/9625011/diff/6/src/full-codegen.cc#newcode1178
src/full-codegen.cc:1178: SetStatementPosition(stmt);
Only set a debug break slot if there is actually a statement.

http://codereview.chromium.org/9625011/diff/6/test/cctest/test-debug.cc
File test/cctest/test-debug.cc (right):

http://codereview.chromium.org/9625011/diff/6/test/cctest/test-debug.cc#newco...
test/cctest/test-debug.cc:2748: CHECK_EQ(34, break_point_hit_count);
We have exactly one debug break more whenever a for loop is involved. Same for
all changes in this file.

http://codereview.chromium.org/9625011/diff/6/test/mjsunit/debug-stepin-acces...
File test/mjsunit/debug-stepin-accessor.js (right):

http://codereview.chromium.org/9625011/diff/6/test/mjsunit/debug-stepin-acces...
test/mjsunit/debug-stepin-accessor.js:117: var x = c['getter' + i];
We want to break before accessing c, not before entering the loop.

[Jakob Kummerow, 2012-03-08 08:08:45]

LGTM

http://codereview.chromium.org/9625011/diff/6/test/mjsunit/debug-stepin-acces...
File test/mjsunit/debug-stepin-accessor.js (right):

http://codereview.chromium.org/9625011/diff/6/test/mjsunit/debug-stepin-acces...
test/mjsunit/debug-stepin-accessor.js:1: // Copyright 2008 the V8 project
authors. All rights reserved.
2012