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

Side by Side Diff: xfa/fde/fde_visualset.h

Issue 2609423003: Properly ref-count CFGAS_GEFont with CFX_RetainPtr. (Closed)
Patch Set: comments Created 3 years, 11 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 #ifndef XFA_FDE_FDE_VISUALSET_H_ 7 #ifndef XFA_FDE_FDE_VISUALSET_H_
8 #define XFA_FDE_FDE_VISUALSET_H_ 8 #define XFA_FDE_FDE_VISUALSET_H_
9 9
10 #include "core/fxcrt/cfx_retain_ptr.h"
10 #include "core/fxcrt/fx_coordinates.h" 11 #include "core/fxcrt/fx_coordinates.h"
11 #include "core/fxcrt/fx_system.h" 12 #include "core/fxcrt/fx_system.h"
12 #include "core/fxge/fx_dib.h" 13 #include "core/fxge/fx_dib.h"
13 #include "xfa/fde/cfde_path.h" 14 #include "xfa/fde/cfde_path.h"
14 #include "xfa/fde/fde_object.h" 15 #include "xfa/fde/fde_object.h"
15 #include "xfa/fgas/crt/fgas_memory.h" 16 #include "xfa/fgas/crt/fgas_memory.h"
16 #include "xfa/fgas/font/cfgas_fontmgr.h" 17 #include "xfa/fgas/font/cfgas_fontmgr.h"
17 18
18 struct FXTEXT_CHARPOS; 19 struct FXTEXT_CHARPOS;
19 20
(...skipping 21 matching lines...) Expand all
41 public: 42 public:
42 virtual FX_POSITION GetFirstPosition() = 0; 43 virtual FX_POSITION GetFirstPosition() = 0;
43 virtual FDE_TEXTEDITPIECE* GetNext(FX_POSITION& pos, 44 virtual FDE_TEXTEDITPIECE* GetNext(FX_POSITION& pos,
44 IFDE_VisualSet*& pVisualSet) = 0; 45 IFDE_VisualSet*& pVisualSet) = 0;
45 }; 46 };
46 47
47 class IFDE_TextSet : public IFDE_VisualSet { 48 class IFDE_TextSet : public IFDE_VisualSet {
48 public: 49 public:
49 virtual int32_t GetString(FDE_TEXTEDITPIECE* hText, 50 virtual int32_t GetString(FDE_TEXTEDITPIECE* hText,
50 CFX_WideString& wsText) = 0; 51 CFX_WideString& wsText) = 0;
51 virtual CFGAS_GEFont* GetFont() = 0; 52 virtual CFX_RetainPtr<CFGAS_GEFont> GetFont() = 0;
52 virtual FX_FLOAT GetFontSize() = 0; 53 virtual FX_FLOAT GetFontSize() = 0;
53 virtual FX_ARGB GetFontColor() = 0; 54 virtual FX_ARGB GetFontColor() = 0;
54 virtual int32_t GetDisplayPos(FDE_TEXTEDITPIECE* hText, 55 virtual int32_t GetDisplayPos(FDE_TEXTEDITPIECE* hText,
55 FXTEXT_CHARPOS* pCharPos, 56 FXTEXT_CHARPOS* pCharPos,
56 bool bCharCode = false, 57 bool bCharCode = false,
57 CFX_WideString* pWSForms = nullptr) = 0; 58 CFX_WideString* pWSForms = nullptr) = 0;
58 virtual int32_t GetCharRects(const FDE_TEXTEDITPIECE* hText, 59 virtual int32_t GetCharRects(const FDE_TEXTEDITPIECE* hText,
59 CFX_RectFArray& rtArray, 60 CFX_RectFArray& rtArray,
60 bool bbox) = 0; 61 bool bbox) = 0;
61 }; 62 };
62 63
63 #endif // XFA_FDE_FDE_VISUALSET_H_ 64 #endif // XFA_FDE_FDE_VISUALSET_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698