Fix bug in compilation-handlescope.

Fix bug in compilation-handlescope.


BUG=
TEST=

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

[Jakob Kummerow, 2012-07-06 13:30:32]

http://codereview.chromium.org/10696125/diff/1/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6474
src/api.cc:6474: deferred_iterator = deferred_iterator->previous_;
nit: {} please

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6511
src/api.cc:6511: ASSERT((last_block_limit_ >= blocks_.first()) &&
last_block_limit_ should probably be renamed to first_block_limit_ now.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6516
src/api.cc:6516: for (int i = 1; i < (blocks_.length() - 1); i++) {
This skips over the last block.

[sanjoy, 2012-07-06 13:56:57]

http://codereview.chromium.org/10696125/diff/1/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6474
src/api.cc:6474: deferred_iterator = deferred_iterator->previous_;
On 2012/07/06 13:30:32, Jakob wrote:
> nit: {} please

Done.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6511
src/api.cc:6511: ASSERT((last_block_limit_ >= blocks_.first()) &&
On 2012/07/06 13:30:32, Jakob wrote:
> last_block_limit_ should probably be renamed to first_block_limit_ now.

Done.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6516
src/api.cc:6516: for (int i = 1; i < (blocks_.length() - 1); i++) {
On 2012/07/06 13:30:32, Jakob wrote:
> This skips over the last block.

Done.

[Jakob Kummerow, 2012-07-06 14:04:10]

lgtm