[debugger] flood function for stepping on throw.

[debugger] flood function for stepping on throw.

We used to flood the handler when preparing for stepping,
even if we may not throw. Instead, we now flood the
handler only when we actually throw.

This also solves an issue with step-next when we throw and
leave the function unexpectedly. In combination with
microtasks, this could cause a crash.

R=mstarzinger@chromium.org
BUG=chromium:568477
LOG=N

Committed: https://crrev.com/6d8a2611c0e1ccb9d64456784a5f675534409541
Cr-Commit-Position: refs/heads/master@{#32856}

[Yang, 2015-12-14 13:52:06]

This is the fix we discussed. Please take a look. Thanks!

[Michael Starzinger, 2015-12-14 14:00:09]

LGTM. Nice, I like it!

[Yang, 2015-12-14 14:03:24]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-14 14:03:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527593002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527593002/1

[commit-bot: I haz the power, 2015-12-14 14:28:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-14 14:28:09]

Dry run: Try jobs failed on following builders:
  v8_linux_arm64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel/builds/1...)

[Yang, 2015-12-15 06:22:39]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-15 06:22:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527593002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527593002/20001

[commit-bot: I haz the power, 2015-12-15 06:55:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-15 06:55:00]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/11505)

[Yang, 2015-12-15 07:10:08]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-15 07:10:11]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527593002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527593002/20001

[commit-bot: I haz the power, 2015-12-15 07:36:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-15 07:36:39]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/11506)

[Yang, 2015-12-15 08:03:21]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-15 08:03:32]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527593002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527593002/40001

[commit-bot: I haz the power, 2015-12-15 08:25:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-15 08:25:11]

Dry run: This issue passed the CQ dry run.

[Yang, 2015-12-15 09:31:48]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2015-12-15 09:31:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1527593002/#ps40001
(title: "one more item for the black list")

[commit-bot: I haz the power, 2015-12-15 09:31:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527593002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527593002/40001

[commit-bot: I haz the power, 2015-12-15 09:33:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-15 09:33:47]

Description was changed from

==========
[debugger] flood function for stepping on throw.

We used to flood the handler when preparing for stepping,
even if we may not throw. Instead, we now flood the
handler only when we actually throw.

This also solves an issue with step-next when we throw and
leave the function unexpectedly. In combination with
microtasks, this could cause a crash.

R=mstarzinger@chromium.org
BUG=chromium:568477
LOG=N
==========

to

==========
[debugger] flood function for stepping on throw.

We used to flood the handler when preparing for stepping,
even if we may not throw. Instead, we now flood the
handler only when we actually throw.

This also solves an issue with step-next when we throw and
leave the function unexpectedly. In combination with
microtasks, this could cause a crash.

R=mstarzinger@chromium.org
BUG=chromium:568477
LOG=N

Committed: https://crrev.com/6d8a2611c0e1ccb9d64456784a5f675534409541
Cr-Commit-Position: refs/heads/master@{#32856}
==========

[commit-bot: I haz the power, 2015-12-15 09:33:48]

Patchset 3 (id:??) landed as
https://crrev.com/6d8a2611c0e1ccb9d64456784a5f675534409541
Cr-Commit-Position: refs/heads/master@{#32856}