summaryrefslogtreecommitdiff
path: root/fxjs/JS_Define.cpp
diff options
context:
space:
mode:
authorAdam Klein <adamk@chromium.org>2018-01-22 17:53:56 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-22 17:53:56 +0000
commit434e28dd41e37dc829440aab1eb4cb4ec561962b (patch)
tree3fe7c26d9e4e9d34b0883c11214ab3fb2f93efd8 /fxjs/JS_Define.cpp
parent1fa1b1f771dd077b0fb4056c7b84f701ba733a8c (diff)
downloadpdfium-434e28dd41e37dc829440aab1eb4cb4ec561962b.tar.xz
Use MaybeLocal versions of v8::Script APIs
The non-Maybe versions will soon be deprecated. Where the code was already handling failures, I did the same; where the code was not handling failures, I inserted ToLocalChecked() calls. Thus the behavior before and after this patch should be the same. For consistency, also updated the use of v8::Function::Call in CFXJSE_Context::ExecuteScript() to the Maybe version, so that all code in that function now branches on the non-emptyness of MaybeLocals, rather than TryCatch::HasCaught(). ASSERTs were inserted to sanity-check the API usage. Bug: v8:7269, v8:7273, v8:7274 Change-Id: I59696e63a64a233a9f9c2b72c621a05b5e349a31 Reviewed-on: https://pdfium-review.googlesource.com/23270 Commit-Queue: Jochen Eisinger <jochen@chromium.org> Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Diffstat (limited to 'fxjs/JS_Define.cpp')
0 files changed, 0 insertions, 0 deletions