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

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

Issue 2578793002: Remove unused m_OtfFontData (Closed)
Patch Set: AttachData Created 4 years 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 | « core/fxge/fx_font.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/fxge/ge/cfx_font.cpp
diff --git a/core/fxge/ge/cfx_font.cpp b/core/fxge/ge/cfx_font.cpp
index 7be300ff660f6da392e273263174296953dc1a5a..24dcb87383e1f2ba997aef9d330295716f4c9d6f 100644
--- a/core/fxge/ge/cfx_font.cpp
+++ b/core/fxge/ge/cfx_font.cpp
@@ -265,10 +265,6 @@ bool CFX_Font::LoadClone(const CFX_Font* pFont) {
m_pSubstFont->m_Family = pFont->m_pSubstFont->m_Family;
m_pSubstFont->m_ItalicAngle = pFont->m_pSubstFont->m_ItalicAngle;
}
- if (pFont->m_OtfFontData.GetSize()) {
- m_OtfFontData.AttachData(pFont->m_OtfFontData.GetBuffer(),
- pFont->m_OtfFontData.GetSize());
- }
m_Face = pFont->m_Face;
m_bEmbedded = pFont->m_bEmbedded;
m_bVertical = pFont->m_bVertical;
@@ -292,10 +288,8 @@ void CFX_Font::SetFace(FXFT_Face face) {
CFX_Font::~CFX_Font() {
#ifdef PDF_ENABLE_XFA
- if (m_bShallowCopy) {
- m_OtfFontData.DetachBuffer();
+ if (m_bShallowCopy)
return;
- }
#endif // PDF_ENABLE_XFA
if (m_Face) {
#ifndef PDF_ENABLE_XFA
« no previous file with comments | « core/fxge/fx_font.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698