Simplify pending message object handling.

Simplify pending message object handling.

This moves the decision whether to report a message or not to when
the pending exception is propagated instead of trying to preserve the
decision in a ThreadLocalTop field.

R=titzer@chromium.org

Committed: https://crrev.com/d4696c484142c601cbd691ae80164043785a3af7
Cr-Commit-Position: refs/heads/master@{#27150}

[titzer, 2015-03-11 18:17:10]

LGTM love red

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1407
src/isolate.cc:1407: // propagation was unsuccessful, then we will get another
change at reporting
s/change/chance

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1408
src/isolate.cc:1408: // the pending message once the exception is re-thrown (if
it ever is).
s/once the exception re-thrown (if it ever is)/if the exception is re-thrown/

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1412
src/isolate.cc:1412: // Clear the pending message object early to avoid endless
recurison.
s/recurison/recursion/

[Michael Starzinger, 2015-03-11 18:35:53]

Thanks.

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1407
src/isolate.cc:1407: // propagation was unsuccessful, then we will get another
change at reporting
On 2015/03/11 18:17:10, titzer wrote:
> s/change/chance

Done.

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1408
src/isolate.cc:1408: // the pending message once the exception is re-thrown (if
it ever is).
On 2015/03/11 18:17:10, titzer wrote:
> s/once the exception re-thrown (if it ever is)/if the exception is re-thrown/

Done.

https://codereview.chromium.org/998943003/diff/1/src/isolate.cc#newcode1412
src/isolate.cc:1412: // Clear the pending message object early to avoid endless
recurison.
On 2015/03/11 18:17:10, titzer wrote:
> s/recurison/recursion/

Done.

[Michael Starzinger, 2015-03-12 09:29:54]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2015-03-12 09:29:55]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org
Link to the patchset: https://codereview.chromium.org/998943003/#ps20001 (title:
"Addressed comments.")

[commit-bot: I haz the power, 2015-03-12 09:30:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/998943003/20001

[commit-bot: I haz the power, 2015-03-12 09:50:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-12 09:50:53]

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