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

Side by Side Diff: fpdfsdk/include/javascript/global.h

Issue 1252613002: FX_BOOL considered harmful. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Manual edits. Created 5 years, 5 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/include/javascript/event.h ('k') | fpdfsdk/include/javascript/report.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 #ifndef FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_ 7 #ifndef FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_
8 #define FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_ 8 #define FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_
9 9
10 #include "JS_Define.h" 10 #include "JS_Define.h"
11 11
12 class CJS_GlobalData; 12 class CJS_GlobalData;
13 class CJS_GlobalVariableArray; 13 class CJS_GlobalVariableArray;
14 class CJS_KeyValue; 14 class CJS_KeyValue;
15 15
16 struct js_global_data 16 struct js_global_data
17 { 17 {
18 js_global_data() 18 js_global_data()
19 { 19 {
20 nType = 0; 20 nType = 0;
21 dData = 0; 21 dData = 0;
22 » » bData = FALSE; 22 » » bData = false;
23 sData = ""; 23 sData = "";
24 » » bPersistent = FALSE; 24 » » bPersistent = false;
25 » » bDeleted = FALSE; 25 » » bDeleted = false;
26 } 26 }
27 27
28 ~js_global_data() 28 ~js_global_data()
29 { 29 {
30 pData.Reset(); 30 pData.Reset();
31 } 31 }
32 int nType; //0:int 1:bool 2:string 3 :obj 32 int nType; //0:int 1:bool 2:string 3 :obj
33 double dData; 33 double dData;
34 bool bData; 34 bool bData;
35 CFX_ByteString sData; 35 CFX_ByteString sData;
36 v8::Global<v8::Object> pData; 36 v8::Global<v8::Object> pData;
37 bool bPersistent; 37 bool bPersistent;
38 bool bDeleted; 38 bool bDeleted;
39 }; 39 };
40 40
41 class global_alternate : public CJS_EmbedObj 41 class global_alternate : public CJS_EmbedObj
42 { 42 {
43 public: 43 public:
44 global_alternate(CJS_Object* pJSObject); 44 global_alternate(CJS_Object* pJSObject);
45 virtual ~global_alternate(); 45 virtual ~global_alternate();
46 46
47 public: 47 public:
48 » FX_BOOL»» » » » » setPersistent(IFXJS_Cont ext* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError); 48 » bool» » » » » » setPersistent(IFXJS_Cont ext* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError);
49 49
50 public: 50 public:
51 » FX_BOOL»» » » » » QueryProperty(const FX_W CHAR* propname); 51 » bool» » » » » » QueryProperty(const FX_W CHAR* propname);
52 » FX_BOOL»» » » » » DoProperty(IFXJS_Context * cc, const FX_WCHAR* propname, CJS_PropValue & vp, CFX_WideString & sError); 52 » bool» » » » » » DoProperty(IFXJS_Context * cc, const FX_WCHAR* propname, CJS_PropValue & vp, CFX_WideString & sError);
53 » FX_BOOL»» » » » » DelProperty(IFXJS_Contex t* cc, const FX_WCHAR* propname, CFX_WideString & sError); 53 » bool» » » » » » DelProperty(IFXJS_Contex t* cc, const FX_WCHAR* propname, CFX_WideString & sError);
54 54
55 void Initial(CPDFDoc_Environm ent* pApp); 55 void Initial(CPDFDoc_Environm ent* pApp);
56 56
57 private: 57 private:
58 void UpdateGlobalPersistentVa riables(); 58 void UpdateGlobalPersistentVa riables();
59 void CommitGlobalPersisitentV ariables(); 59 void CommitGlobalPersisitentV ariables();
60 void DestroyGlobalPersisitent Variables(); 60 void DestroyGlobalPersisitent Variables();
61 » FX_BOOL»» » » » » SetGlobalVariables(const FX_CHAR* propname, int nType, 61 » bool» » » » » » SetGlobalVariables(const FX_CHAR* propname, int nType,
62 double d Data, bool bData, const CFX_ByteString& sData, JSObject pData, bool bDefaultPers istent); 62 double d Data, bool bData, const CFX_ByteString& sData, JSObject pData, bool bDefaultPers istent);
63 63
64 void ObjectToArray(v8::Local< v8::Object> pObj, CJS_GlobalVariableArray& array); 64 void ObjectToArray(v8::Local< v8::Object> pObj, CJS_GlobalVariableArray& array);
65 void PutObjectProperty(v8::Lo cal<v8::Object> obj, CJS_KeyValue* pData); 65 void PutObjectProperty(v8::Lo cal<v8::Object> obj, CJS_KeyValue* pData);
66 66
67 private: 67 private:
68 CFX_MapByteStringToPtr m_mapGlobal; 68 CFX_MapByteStringToPtr m_mapGlobal;
69 CFX_WideString m_sFilePath; 69 CFX_WideString m_sFilePath;
70 CJS_GlobalData* m_pGlobalData; 70 CJS_GlobalData* m_pGlobalData;
71 CPDFDoc_Environment* m_pApp; 71 CPDFDoc_Environment* m_pApp;
72 }; 72 };
73 73
74 74
75 class CJS_Global : public CJS_Object 75 class CJS_Global : public CJS_Object
76 { 76 {
77 public: 77 public:
78 CJS_Global(JSFXObject pObject) : CJS_Object(pObject) {}; 78 CJS_Global(JSFXObject pObject) : CJS_Object(pObject) {};
79 virtual ~CJS_Global(void){}; 79 virtual ~CJS_Global(void){};
80 80
81 » virtual FX_BOOL»InitInstance(IFXJS_Context* cc); 81 » virtual bool» InitInstance(IFXJS_Context* cc);
82 82
83 DECLARE_SPECIAL_JS_CLASS(CJS_Global); 83 DECLARE_SPECIAL_JS_CLASS(CJS_Global);
84 84
85 JS_SPECIAL_STATIC_METHOD(setPersistent, global_alternate, global); 85 JS_SPECIAL_STATIC_METHOD(setPersistent, global_alternate, global);
86 86
87 }; 87 };
88 88
89 #endif // FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_ 89 #endif // FPDFSDK_INCLUDE_JAVASCRIPT_GLOBAL_H_
OLDNEW
« no previous file with comments | « fpdfsdk/include/javascript/event.h ('k') | fpdfsdk/include/javascript/report.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698