tree 4384dc625041c81203c34a3f4fc9fce0bd374b17
parent 8532ef05056ce00eda2d73d2a8a35bac4b2973ad
author Tom Sepez <tsepez@chromium.org> 1604597698 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1604597698 +0000

Remove use of CFXJSE_Value in CFXJSE_Engine::m_mapObjectToValue.

Just use v8::Global<v8::Object> directly, and avoid dynamic
allocation of another object.

-- Remove NDEBUG section that can reach an ASSERT() against
   document-controlled values that can't be constrained.

-- Return locals rather than reference to global, which required
   introducing a new scope in one place, since you can't have a
   local without a scope already in effect.

Bug: pdfium:1610
Change-Id: I8fb9982eb65ed2c01a1ac844f04eb722120b6331
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/75933
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Daniel Hosseinian <dhoss@chromium.org>