Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Side by Side Diff: fpdfsdk/javascript/PublicMethods.cpp

Issue 2412523002: Cleanup env variable names (Closed)
Patch Set: Cleanup Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include "fpdfsdk/javascript/PublicMethods.h" 7 #include "fpdfsdk/javascript/PublicMethods.h"
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <vector> 10 #include <vector>
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 } 72 }
73 73
74 CFX_WideString StrTrim(const CFX_WideString& pStr) { 74 CFX_WideString StrTrim(const CFX_WideString& pStr) {
75 CFX_WideString result(pStr); 75 CFX_WideString result(pStr);
76 result.TrimLeft(' '); 76 result.TrimLeft(' ');
77 result.TrimRight(' '); 77 result.TrimRight(' ');
78 return result; 78 return result;
79 } 79 }
80 80
81 void AlertIfPossible(CJS_Context* pContext, const FX_WCHAR* swMsg) { 81 void AlertIfPossible(CJS_Context* pContext, const FX_WCHAR* swMsg) {
82 CPDFSDK_FormFillEnvironment* pEnv = pContext->GetFormFillEnv(); 82 CPDFSDK_FormFillEnvironment* pFormFillEnv = pContext->GetFormFillEnv();
83 if (pEnv) 83 if (pFormFillEnv)
84 pEnv->JS_appAlert(swMsg, nullptr, 0, 3); 84 pFormFillEnv->JS_appAlert(swMsg, nullptr, 0, 3);
85 } 85 }
86 86
87 } // namespace 87 } // namespace
88 88
89 bool CJS_PublicMethods::IsNumber(const CFX_WideString& str) { 89 bool CJS_PublicMethods::IsNumber(const CFX_WideString& str) {
90 CFX_WideString sTrim = StrTrim(str); 90 CFX_WideString sTrim = StrTrim(str);
91 const FX_WCHAR* pTrim = sTrim.c_str(); 91 const FX_WCHAR* pTrim = sTrim.c_str();
92 const FX_WCHAR* p = pTrim; 92 const FX_WCHAR* p = pTrim;
93 bool bDot = false; 93 bool bDot = false;
94 bool bKXJS = false; 94 bool bKXJS = false;
(...skipping 1557 matching lines...) Expand 10 before | Expand all | Expand 10 after
1652 1652
1653 CJS_Value params1 = params[1]; 1653 CJS_Value params1 = params[1];
1654 if (!params1.IsArrayObject() && params1.GetType() != CJS_Value::VT_string) { 1654 if (!params1.IsArrayObject() && params1.GetType() != CJS_Value::VT_string) {
1655 sError = JSGetStringFromID(IDS_STRING_JSPARAMERROR); 1655 sError = JSGetStringFromID(IDS_STRING_JSPARAMERROR);
1656 return FALSE; 1656 return FALSE;
1657 } 1657 }
1658 1658
1659 CJS_Context* pContext = static_cast<CJS_Context*>(cc); 1659 CJS_Context* pContext = static_cast<CJS_Context*>(cc);
1660 CJS_Runtime* pRuntime = CJS_Runtime::FromContext(cc); 1660 CJS_Runtime* pRuntime = CJS_Runtime::FromContext(cc);
1661 CPDFSDK_InterForm* pReaderInterForm = 1661 CPDFSDK_InterForm* pReaderInterForm =
1662 pContext->GetFormFillEnv()->GetSDKDocument()->GetInterForm(); 1662 pContext->GetFormFillEnv()->GetInterForm();
1663 CPDF_InterForm* pInterForm = pReaderInterForm->GetInterForm(); 1663 CPDF_InterForm* pInterForm = pReaderInterForm->GetInterForm();
1664 1664
1665 CFX_WideString sFunction = params[0].ToCFXWideString(pRuntime); 1665 CFX_WideString sFunction = params[0].ToCFXWideString(pRuntime);
1666 double dValue = wcscmp(sFunction.c_str(), L"PRD") == 0 ? 1.0 : 0.0; 1666 double dValue = wcscmp(sFunction.c_str(), L"PRD") == 0 ? 1.0 : 0.0;
1667 1667
1668 CJS_Array FieldNameArray = AF_MakeArrayFromList(pRuntime, params1); 1668 CJS_Array FieldNameArray = AF_MakeArrayFromList(pRuntime, params1);
1669 int nFieldsCount = 0; 1669 int nFieldsCount = 0;
1670 1670
1671 for (int i = 0, isz = FieldNameArray.GetLength(pRuntime); i < isz; i++) { 1671 for (int i = 0, isz = FieldNameArray.GetLength(pRuntime); i < isz; i++) {
1672 CJS_Value jsValue(pRuntime); 1672 CJS_Value jsValue(pRuntime);
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
1823 nums.SetElement(pRuntime, nIndex, CJS_Value(pRuntime, sPart.c_str())); 1823 nums.SetElement(pRuntime, nIndex, CJS_Value(pRuntime, sPart.c_str()));
1824 } 1824 }
1825 1825
1826 if (nums.GetLength(pRuntime) > 0) 1826 if (nums.GetLength(pRuntime) > 0)
1827 vRet = CJS_Value(pRuntime, nums); 1827 vRet = CJS_Value(pRuntime, nums);
1828 else 1828 else
1829 vRet.SetNull(pRuntime); 1829 vRet.SetNull(pRuntime);
1830 1830
1831 return TRUE; 1831 return TRUE;
1832 } 1832 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698