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

Unified Diff: core/include/fxge/fx_font.h

Issue 1410823005: Take FOXIT_FACE_COUNT and MM_FACE_COUNT constants from xfa (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Comments Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | core/src/fxge/ge/fx_ge_fontmap.cpp » ('j') | core/src/fxge/ge/fx_ge_fontmap.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/include/fxge/fx_font.h
diff --git a/core/include/fxge/fx_font.h b/core/include/fxge/fx_font.h
index 36a739f737a6dee34c1143d4d3cf780df7d7a92a..0ab529ef4e1814b7710c2ff24216cc788168e798 100644
--- a/core/include/fxge/fx_font.h
+++ b/core/include/fxge/fx_font.h
@@ -262,6 +262,9 @@ class CFX_FontMapper {
CFX_SubstFont* pSubstFont);
private:
+ static const size_t MM_FACE_COUNT = 2;
+ static const size_t FOXIT_FACE_COUNT = 14;
+
CFX_ByteString GetPSNameFromTT(void* hFont);
CFX_ByteString MatchInstalledFonts(const CFX_ByteString& norm_name);
FXFT_Face UseInternalSubst(CFX_SubstFont* pSubstFont,
@@ -271,12 +274,12 @@ class CFX_FontMapper {
int picthfamily);
FX_BOOL m_bListLoaded;
- FXFT_Face m_MMFaces[2];
+ FXFT_Face m_MMFaces[MM_FACE_COUNT];
CFX_ByteString m_LastFamily;
CFX_DWordArray m_CharsetArray;
CFX_ByteStringArray m_FaceArray;
IFX_SystemFontInfo* m_pFontInfo;
- FXFT_Face m_FoxitFaces[14];
+ FXFT_Face m_FoxitFaces[FOXIT_FACE_COUNT];
IFX_FontEnumerator* m_pFontEnumerator;
CFX_FontMgr* const m_pFontMgr;
};
« no previous file with comments | « no previous file | core/src/fxge/ge/fx_ge_fontmap.cpp » ('j') | core/src/fxge/ge/fx_ge_fontmap.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698