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

Side by Side Diff: fxjs/cfxjse_class.h

Issue 2471353002: Remove FX_BOOL entirely. (Closed)
Patch Set: Fix nits now rather than later Created 4 years, 1 month 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 | « fxjs/cfxjse_arguments.cpp ('k') | fxjs/cfxjse_class.cpp » ('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 FXJS_CFXJSE_CLASS_H_ 7 #ifndef FXJS_CFXJSE_CLASS_H_
8 #define FXJS_CFXJSE_CLASS_H_ 8 #define FXJS_CFXJSE_CLASS_H_
9 9
10 #include "fxjs/cfxjse_arguments.h" 10 #include "fxjs/cfxjse_arguments.h"
11 #include "fxjs/fxjse.h" 11 #include "fxjs/fxjse.h"
12 #include "v8/include/v8.h" 12 #include "v8/include/v8.h"
13 13
14 class CFXJSE_Context; 14 class CFXJSE_Context;
15 class CFXJSE_Value; 15 class CFXJSE_Value;
16 16
17 class CFXJSE_Class { 17 class CFXJSE_Class {
18 public: 18 public:
19 static CFXJSE_Class* Create(CFXJSE_Context* pContext, 19 static CFXJSE_Class* Create(CFXJSE_Context* pContext,
20 const FXJSE_CLASS_DESCRIPTOR* lpClassDefintion, 20 const FXJSE_CLASS_DESCRIPTOR* lpClassDefintion,
21 FX_BOOL bIsJSGlobal = FALSE); 21 bool bIsJSGlobal = false);
22 static CFXJSE_Class* GetClassFromContext(CFXJSE_Context* pContext, 22 static CFXJSE_Class* GetClassFromContext(CFXJSE_Context* pContext,
23 const CFX_ByteStringC& szName); 23 const CFX_ByteStringC& szName);
24 static void SetUpNamedPropHandler( 24 static void SetUpNamedPropHandler(
25 v8::Isolate* pIsolate, 25 v8::Isolate* pIsolate,
26 v8::Local<v8::ObjectTemplate>& hObjectTemplate, 26 v8::Local<v8::ObjectTemplate>& hObjectTemplate,
27 const FXJSE_CLASS_DESCRIPTOR* lpClassDefinition); 27 const FXJSE_CLASS_DESCRIPTOR* lpClassDefinition);
28 28
29 ~CFXJSE_Class(); 29 ~CFXJSE_Class();
30 30
31 CFXJSE_Context* GetContext() { return m_pContext; } 31 CFXJSE_Context* GetContext() { return m_pContext; }
32 v8::Global<v8::FunctionTemplate>& GetTemplate() { return m_hTemplate; } 32 v8::Global<v8::FunctionTemplate>& GetTemplate() { return m_hTemplate; }
33 33
34 protected: 34 protected:
35 explicit CFXJSE_Class(CFXJSE_Context* lpContext); 35 explicit CFXJSE_Class(CFXJSE_Context* lpContext);
36 36
37 CFX_ByteString m_szClassName; 37 CFX_ByteString m_szClassName;
38 const FXJSE_CLASS_DESCRIPTOR* m_lpClassDefinition; 38 const FXJSE_CLASS_DESCRIPTOR* m_lpClassDefinition;
39 CFXJSE_Context* m_pContext; 39 CFXJSE_Context* m_pContext;
40 v8::Global<v8::FunctionTemplate> m_hTemplate; 40 v8::Global<v8::FunctionTemplate> m_hTemplate;
41 friend class CFXJSE_Context; 41 friend class CFXJSE_Context;
42 friend class CFXJSE_Value; 42 friend class CFXJSE_Value;
43 }; 43 };
44 44
45 #endif // FXJS_CFXJSE_CLASS_H_ 45 #endif // FXJS_CFXJSE_CLASS_H_
OLDNEW
« no previous file with comments | « fxjs/cfxjse_arguments.cpp ('k') | fxjs/cfxjse_class.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698