Fixed a couple of proxies-related unhandled exceptions.

Fixed a couple of proxies-related unhandled exceptions.

BUG=chromium:506956, chromium:505907
LOG=N

Committed: https://crrev.com/52b3e41799ef80cf47b84db15a3ade4f613a6ce1
Cr-Commit-Position: refs/heads/master@{#29530}

[Igor Sheludko, 2015-07-07 10:33:22]

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

[Igor Sheludko, 2015-07-07 10:33:22]

PTAL

[Yang, 2015-07-08 07:55:18]

LGTM with comment.

https://codereview.chromium.org/1215463012/diff/1/src/runtime/runtime-scopes.cc
File src/runtime/runtime-scopes.cc (right):

https://codereview.chromium.org/1215463012/diff/1/src/runtime/runtime-scopes....
src/runtime/runtime-scopes.cc:234: if (isolate->has_pending_exception()) return
isolate->heap()->exception();
This looks hacky. I.e. we should have returned a MaybeHandle.

But then again this is proxy, so I guess I'm fine with this. Can we first check
for holder.is_null() first like we do below?

[Igor Sheludko, 2015-07-08 10:37:37]

Addressed comments

[Igor Sheludko, 2015-07-08 11:17:06]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2015-07-08 11:17:06]

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

[commit-bot: I haz the power, 2015-07-08 11:17:08]

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

[commit-bot: I haz the power, 2015-07-08 11:46:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-08 11:46:28]

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