diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2015-09-30 02:13:31 +0200 |
|---|---|---|
| committer | Ben Noordhuis <info@bnoordhuis.nl> | 2015-09-30 23:43:16 +0200 |
| commit | dbe48446b7981a60ac3e461056c7a888572b8e24 (patch) | |
| tree | dbeb1790127321c4ff3013b642e35226a5a0ce05 /deps | |
| parent | 9a1bc4ea7ecb15ecce643e56bb9015c790e391ea (diff) | |
| download | android-node-v8-dbe48446b7981a60ac3e461056c7a888572b8e24.tar.gz android-node-v8-dbe48446b7981a60ac3e461056c7a888572b8e24.tar.bz2 android-node-v8-dbe48446b7981a60ac3e461056c7a888572b8e24.zip | |
deps: backport f782159 from v8's upstream
Backport f78215962bf5de9d47c022e7baa3952d0bf6d17f from V8's upstream
to speed up promise introspection.
Original commit message:
Remove obsolete try/catch from ObjectIsPromise().
Review URL: https://codereview.chromium.org/1367123003
Cr-Commit-Position: refs/heads/master@{#30966}
PR-URL: https://github.com/nodejs/node/pull/3130
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Diffstat (limited to 'deps')
| -rw-r--r-- | deps/v8/src/mirror-debugger.js | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/deps/v8/src/mirror-debugger.js b/deps/v8/src/mirror-debugger.js index dbdc68e68f..d37776af38 100644 --- a/deps/v8/src/mirror-debugger.js +++ b/deps/v8/src/mirror-debugger.js @@ -19,16 +19,9 @@ function ToggleMirrorCache(value) { } -// Wrapper to check whether an object is a Promise. The call may not work -// if promises are not enabled. -// TODO(yangguo): remove try-catch once promises are enabled by default. function ObjectIsPromise(value) { - try { - return IS_SPEC_OBJECT(value) && - !IS_UNDEFINED(%DebugGetProperty(value, builtins.$promiseStatus)); - } catch (e) { - return false; - } + return IS_SPEC_OBJECT(value) && + !IS_UNDEFINED(%DebugGetProperty(value, builtins.$promiseStatus)); } |