Avoid calling out to JS during stack overflow

Avoid calling out to JS during stack overflow

If an exception is thrown when there is a Promise being created, the Promise
catch prediction code would call into a part implemented in JavaScript to see if
the Promise has a catch handler. If it is not possible to call back into JS,
e.g., due to a stack overflow, then this would lead to a crash. This patch
"speculates" that, if it's impossible to call back into JavaScript, then the
error is unhandled, avoding the issue. In a future patch, the catch prediction
logic should be entirely written in C++, but this patch adds a minimal fix to
be more friendly to backports.

BUG=chromium:662935
R=jgruber

Committed: https://crrev.com/5975c47a6a0da2ed1c8b6ea3d1d5867452e0ab29
Cr-Commit-Position: refs/heads/master@{#40851}

[Dan Ehrenberg, 2016-11-08 22:35:58]

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

[commit-bot: I haz the power, 2016-11-08 22:36:08]

Dry run: 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-11-08 23:03:34]

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

[commit-bot: I haz the power, 2016-11-08 23:03:35]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-11-09 07:50:09]

lgtm, nice!

[Dan Ehrenberg, 2016-11-09 07:50:41]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-11-09 07:50:52]

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-11-09 07:52:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:26:57]

Description was changed from

==========
Avoid calling out to JS during stack overflow

If an exception is thrown when there is a Promise being created, the Promise
catch prediction code would call into a part implemented in JavaScript to see if
the Promise has a catch handler. If it is not possible to call back into JS,
e.g., due to a stack overflow, then this would lead to a crash. This patch
"speculates" that, if it's impossible to call back into JavaScript, then the
error is unhandled, avoding the issue. In a future patch, the catch prediction
logic should be entirely written in C++, but this patch adds a minimal fix to
be more friendly to backports.

BUG=chromium:662935
R=jgruber
==========

to

==========
Avoid calling out to JS during stack overflow

If an exception is thrown when there is a Promise being created, the Promise
catch prediction code would call into a part implemented in JavaScript to see if
the Promise has a catch handler. If it is not possible to call back into JS,
e.g., due to a stack overflow, then this would lead to a crash. This patch
"speculates" that, if it's impossible to call back into JavaScript, then the
error is unhandled, avoding the issue. In a future patch, the catch prediction
logic should be entirely written in C++, but this patch adds a minimal fix to
be more friendly to backports.

BUG=chromium:662935
R=jgruber

Committed: https://crrev.com/5975c47a6a0da2ed1c8b6ea3d1d5867452e0ab29
Cr-Commit-Position: refs/heads/master@{#40851}
==========

[commit-bot: I haz the power, 2016-11-17 22:26:58]

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