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

Unified Diff: core/fxge/ge/cfx_font.cpp

Issue 2368693002: Remove FX_BOOL from cpdf_font (Closed)
Patch Set: And again Created 4 years, 3 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
Index: core/fxge/ge/cfx_font.cpp
diff --git a/core/fxge/ge/cfx_font.cpp b/core/fxge/ge/cfx_font.cpp
index b1847112116651ffd1bdbf3322cff2675bd4bd3a..ce77cbf66152640dd791adaaba3458fdbe785bc3 100644
--- a/core/fxge/ge/cfx_font.cpp
+++ b/core/fxge/ge/cfx_font.cpp
@@ -233,8 +233,8 @@ CFX_Font::CFX_Font()
#if _FXM_PLATFORM_ == _FXM_PLATFORM_APPLE_
m_pPlatformFont(nullptr),
#endif
- m_bEmbedded(FALSE),
- m_bVertical(FALSE) {
+ m_bEmbedded(false),
+ m_bVertical(false) {
}
#ifdef PDF_ENABLE_XFA
@@ -316,8 +316,8 @@ void CFX_Font::LoadSubst(const CFX_ByteString& face_name,
int weight,
int italic_angle,
int CharsetCP,
- FX_BOOL bVertical) {
- m_bEmbedded = FALSE;
+ bool bVertical) {
+ m_bEmbedded = false;
m_bVertical = bVertical;
m_pSubstFont.reset(new CFX_SubstFont);
m_Face = CFX_GEModule::Get()->GetFontMgr()->FindSubstFont(
@@ -371,14 +371,14 @@ FX_BOOL CFX_Font::LoadEmbedded(const uint8_t* data, uint32_t size) {
m_pFontDataAllocation.swap(temp);
m_Face = FT_LoadFont(m_pFontDataAllocation.data(), size);
m_pFontData = m_pFontDataAllocation.data();
- m_bEmbedded = TRUE;
+ m_bEmbedded = true;
m_dwSize = size;
return !!m_Face;
}
-FX_BOOL CFX_Font::IsTTFont() const {
+bool CFX_Font::IsTTFont() const {
if (!m_Face)
- return FALSE;
+ return false;
return FXFT_Is_Face_TT_OT(m_Face) == FXFT_FACE_FLAG_SFNT;
}

Powered by Google App Engine
This is Rietveld 408576698