diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-08-29 23:53:53 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-29 23:53:53 +0000 |
| commit | 20c41a5a47b2460277897a1059789fc9e184e3ab (patch) | |
| tree | a60b9695de7242be833c4119bb9815bd39b297ca /core/fpdfapi | |
| parent | 8020fd856805ed5c5f48a7a5f3a346ea6948355f (diff) | |
| download | pdfium-20c41a5a47b2460277897a1059789fc9e184e3ab.tar.xz | |
Use pdfium::span<>/UnownedPtr in CFX_Font.
Move some platform-specific ifdefs so the come last in each
section as it is easier to read, perhaps.
Change-Id: Ic1c2652c46ecebc63b66213735ed6d94737a7f32
Reviewed-on: https://pdfium-review.googlesource.com/41630
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfapi')
| -rw-r--r-- | core/fpdfapi/font/cpdf_font.cpp | 4 | ||||
| -rw-r--r-- | core/fpdfapi/font/cpdf_type1font.cpp | 4 |
2 files changed, 3 insertions, 5 deletions
diff --git a/core/fpdfapi/font/cpdf_font.cpp b/core/fpdfapi/font/cpdf_font.cpp index 9b83733eea..298911e2cd 100644 --- a/core/fpdfapi/font/cpdf_font.cpp +++ b/core/fpdfapi/font/cpdf_font.cpp @@ -208,9 +208,7 @@ void CPDF_Font::LoadFontDescriptor(const CPDF_Dictionary* pFontDesc) { if (!m_pFontFile) return; - const uint8_t* pFontData = m_pFontFile->GetData(); - uint32_t dwFontSize = m_pFontFile->GetSize(); - if (!m_Font.LoadEmbedded(pFontData, dwFontSize)) { + if (!m_Font.LoadEmbedded(m_pFontFile->GetSpan())) { m_pDocument->GetPageData()->MaybePurgeFontFileStreamAcc( m_pFontFile->GetStream()->AsStream()); m_pFontFile = nullptr; diff --git a/core/fpdfapi/font/cpdf_type1font.cpp b/core/fpdfapi/font/cpdf_type1font.cpp index 44e6b3e23c..7fb7ce6b54 100644 --- a/core/fpdfapi/font/cpdf_type1font.cpp +++ b/core/fpdfapi/font/cpdf_type1font.cpp @@ -129,8 +129,8 @@ void CPDF_Type1Font::LoadGlyphMap() { if (m_Font.GetPsName() == "DFHeiStd-W5") bCoreText = false; - m_Font.SetPlatformFont( - quartz2d.CreateFont(m_Font.GetFontData(), m_Font.GetSize())); + pdfium::span<const uint8_t> span = m_Font.GetFontSpan(); + m_Font.SetPlatformFont(quartz2d.CreateFont(span.data(), span.size())); if (!m_Font.GetPlatformFont()) bCoreText = false; } |