diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-09-16 10:42:08 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-09-16 10:42:08 -0700 |
commit | 506df426d5d64d68e9dc27ffebcf56f6c6a1bccf (patch) | |
tree | 3ba8d38a3bef95c0555549953bce0cce90845cdb /fpdfsdk/src/javascript/PublicMethods.cpp | |
parent | 615d7aba0f7d53a78eb05fbd4ae0e1a7e1d3b103 (diff) | |
download | pdfium-506df426d5d64d68e9dc27ffebcf56f6c6a1bccf.tar.xz |
Ensure functions in FXJS_V8 are prefixed by FXJS_.
Currently, its hard to tell which functions come from the JS_
layer at fpdfsdk/include/javascript vs. which functions come
from the FXJS_V8 layer at fpdfsdk/include/jsapi. Until we
take up the task of using namespaces, at least make the
prefix consistent.
Move objects out of FXJS_V8 that are really part of JS_.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1347833002 .
Diffstat (limited to 'fpdfsdk/src/javascript/PublicMethods.cpp')
-rw-r--r-- | fpdfsdk/src/javascript/PublicMethods.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fpdfsdk/src/javascript/PublicMethods.cpp b/fpdfsdk/src/javascript/PublicMethods.cpp index 1722d0f65d..ef745b73ee 100644 --- a/fpdfsdk/src/javascript/PublicMethods.cpp +++ b/fpdfsdk/src/javascript/PublicMethods.cpp @@ -1934,7 +1934,7 @@ FX_BOOL CJS_PublicMethods::AFSimple_Calculate(IFXJS_Context* cc, CJS_Value params1 = params[1]; - if (!params1.IsArrayObject() && params1.GetType() != VT_string) { + if (!params1.IsArrayObject() && params1.GetType() != CJS_Value::VT_string) { sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR); return FALSE; } |