[d8] Fix Shell::Stringify when exceptions are thrown.

[d8] Fix Shell::Stringify when exceptions are thrown.

BUG=v8:5105
Committed: https://crrev.com/e0348dc78338aa4b0ae60771cba3017d23787641
Cr-Commit-Position: refs/heads/master@{#37553}

[Jakob Kummerow, 2016-07-06 13:37:20]

jkummerow@chromium.org changed reviewers:
+ yangguo@chromium.org

[Jakob Kummerow, 2016-07-06 13:37:20]

PTAL. Clearly the "result.IsEmpty()" check should happen before
.ToLocalChecked().

Verified in an interactive d8 session; I don't know how to trigger this with a
testcase.

[Yang, 2016-07-06 13:38:06]

On 2016/07/06 13:37:20, Jakob wrote:
> PTAL. Clearly the "result.IsEmpty()" check should happen before
> .ToLocalChecked().
> 
> Verified in an interactive d8 session; I don't know how to trigger this with a
> testcase.

lgtm

[Jakob Kummerow, 2016-07-06 13:38:22]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2016-07-06 13:38:29]

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

[Yang, 2016-07-06 13:38:33]

On 2016/07/06 13:38:06, Yang wrote:
> On 2016/07/06 13:37:20, Jakob wrote:
> > PTAL. Clearly the "result.IsEmpty()" check should happen before
> > .ToLocalChecked().
> > 
> > Verified in an interactive d8 session; I don't know how to trigger this with
a
> > testcase.
> 
> lgtm

This can't be reproduced outside of an interactive session afaict.

[commit-bot: I haz the power, 2016-07-06 14:00:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-06 14:00:55]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-06 14:02:35]

Description was changed from

==========
[d8] Fix Shell::Stringify when exceptions are thrown.

BUG=v8:5105
==========

to

==========
[d8] Fix Shell::Stringify when exceptions are thrown.

BUG=v8:5105

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

[commit-bot: I haz the power, 2016-07-06 14:02:35]

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