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

Side by Side Diff: xfa/fde/css/fde_cssstylesheet.h

Issue 2208423002: Use smart pointers for class owned pointers under xfa/fde (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: one more change Created 4 years, 4 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 | « xfa/fde/css/fde_cssstyleselector.cpp ('k') | xfa/fde/css/fde_cssstylesheet.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 XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_ 7 #ifndef XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_
8 #define XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_ 8 #define XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_
9 9
10 #include <memory>
10 #include <unordered_map> 11 #include <unordered_map>
11 12
12 #include "core/fxcrt/include/fx_ext.h" 13 #include "core/fxcrt/include/fx_ext.h"
13 #include "xfa/fde/css/fde_cssdeclaration.h" 14 #include "xfa/fde/css/fde_cssdeclaration.h"
14 15
15 class CFDE_CSSSyntaxParser; 16 class CFDE_CSSSyntaxParser;
16 17
17 class CFDE_CSSSelector : public CFX_Target { 18 class CFDE_CSSSelector : public CFX_Target {
18 public: 19 public:
19 CFDE_CSSSelector(FDE_CSSSELECTORTYPE eType, 20 CFDE_CSSSelector(FDE_CSSSELECTORTYPE eType,
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 FDE_CSSSYNTAXSTATUS LoadImportRule(CFDE_CSSSyntaxParser* pSyntax); 127 FDE_CSSSYNTAXSTATUS LoadImportRule(CFDE_CSSSyntaxParser* pSyntax);
127 FDE_CSSSYNTAXSTATUS LoadPageRule(CFDE_CSSSyntaxParser* pSyntax); 128 FDE_CSSSYNTAXSTATUS LoadPageRule(CFDE_CSSSyntaxParser* pSyntax);
128 FDE_CSSSYNTAXSTATUS LoadMediaRule(CFDE_CSSSyntaxParser* pSyntax); 129 FDE_CSSSYNTAXSTATUS LoadMediaRule(CFDE_CSSSyntaxParser* pSyntax);
129 FDE_CSSSYNTAXSTATUS LoadFontFaceRule( 130 FDE_CSSSYNTAXSTATUS LoadFontFaceRule(
130 CFDE_CSSSyntaxParser* pSyntax, 131 CFDE_CSSSyntaxParser* pSyntax,
131 CFX_MassArrayTemplate<IFDE_CSSRule*>& ruleArray); 132 CFX_MassArrayTemplate<IFDE_CSSRule*>& ruleArray);
132 FDE_CSSSYNTAXSTATUS SkipRuleSet(CFDE_CSSSyntaxParser* pSyntax); 133 FDE_CSSSYNTAXSTATUS SkipRuleSet(CFDE_CSSSyntaxParser* pSyntax);
133 uint16_t m_wCodePage; 134 uint16_t m_wCodePage;
134 uint16_t m_wRefCount; 135 uint16_t m_wRefCount;
135 uint32_t m_dwMediaList; 136 uint32_t m_dwMediaList;
136 IFX_MemoryAllocator* m_pAllocator; 137 std::unique_ptr<IFX_MemoryAllocator> m_pAllocator;
137 CFX_MassArrayTemplate<IFDE_CSSRule*> m_RuleArray; 138 CFX_MassArrayTemplate<IFDE_CSSRule*> m_RuleArray;
138 CFX_WideString m_szUrl; 139 CFX_WideString m_szUrl;
139 CFX_ArrayTemplate<CFDE_CSSSelector*> m_Selectors; 140 CFX_ArrayTemplate<CFDE_CSSSelector*> m_Selectors;
140 std::unordered_map<uint32_t, FX_WCHAR*> m_StringCache; 141 std::unordered_map<uint32_t, FX_WCHAR*> m_StringCache;
141 }; 142 };
142 143
143 #endif // XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_ 144 #endif // XFA_FDE_CSS_FDE_CSSSTYLESHEET_H_
OLDNEW
« no previous file with comments | « xfa/fde/css/fde_cssstyleselector.cpp ('k') | xfa/fde/css/fde_cssstylesheet.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698