Do not trigger ExceptionEvents for another forwarding case

Do not trigger ExceptionEvents for another forwarding case

This patch fixes up one last case of redundant ExceptionEvents being
triggered in the debugger for Promises--it makes the default reject
handler for Promises (e.g., if the second argument for
Promise.prototype.then is missing) appear to the debugger as a
rethrow.

R=adamk@chromium.org,jgruber@chromium.org
BUG=v8:5167
Committed: https://crrev.com/aae17eb31b5bafb3c95b000ce351b9c690ce6791
Cr-Commit-Position: refs/heads/master@{#38876}

[Dan Ehrenberg, 2016-08-24 18:36:52]

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

[commit-bot: I haz the power, 2016-08-24 18:36:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Dan Ehrenberg, 2016-08-24 19:33:00]

littledan@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-08-24 19:35:54]

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

[commit-bot: I haz the power, 2016-08-24 19:35:55]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-08-24 20:28:22]

Thus intuitively seems right to me and lgtm from code side, but want to make
sure kozyatinskiy agrees on the logic.

[kozy, 2016-08-24 20:31:38]

lgtm

[Dan Ehrenberg, 2016-08-24 20:35:35]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-08-24 20:35:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-24 20:38:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 20:38:24]

Description was changed from

==========
Do not trigger ExceptionEvents for another forwarding case

This patch fixes up one last case of redundant ExceptionEvents being
triggered in the debugger for Promises--it makes the default reject
handler for Promises (e.g., if the second argument for
Promise.prototype.then is missing) appear to the debugger as a
rethrow.

R=adamk@chromium.org,jgruber@chromium.org
BUG=v8:5167
==========

to

==========
Do not trigger ExceptionEvents for another forwarding case

This patch fixes up one last case of redundant ExceptionEvents being
triggered in the debugger for Promises--it makes the default reject
handler for Promises (e.g., if the second argument for
Promise.prototype.then is missing) appear to the debugger as a
rethrow.

R=adamk@chromium.org,jgruber@chromium.org
BUG=v8:5167

Committed: https://crrev.com/aae17eb31b5bafb3c95b000ce351b9c690ce6791
Cr-Commit-Position: refs/heads/master@{#38876}
==========

[commit-bot: I haz the power, 2016-08-24 20:38:25]

Patchset 1 (id:??) landed as
https://crrev.com/aae17eb31b5bafb3c95b000ce351b9c690ce6791
Cr-Commit-Position: refs/heads/master@{#38876}