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

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

Issue 2412523002: Cleanup env variable names (Closed)
Patch Set: Fix spelling 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
« no previous file with comments | « fpdfsdk/javascript/Document.cpp ('k') | fpdfsdk/javascript/global.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1728 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
« no previous file with comments | « fpdfsdk/javascript/Document.cpp ('k') | fpdfsdk/javascript/global.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698