commit 88040c56ba075a1fc63585d7e47807d656b4130c
author Tom Sepez <tsepez@chromium.org> Tue Apr 28 18:04:27 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Apr 28 18:04:27 2020 +0000
tree 9ac0ac623baeecb018fe21b03073e9e383721367
parent a8cfe04443e7e3c6d6c70c3c7e27aab052aff087

Null check not needed for CFX_GEModule::GetFontMgr() return value.

There's a MakeUnique<T> for CFX_GEModule::m_pFontMgr in the constructor.

Change-Id: I296ea4920f9fcfda80bc2262da3ea29f6d816736
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/69270
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

core/fpdfdoc/cba_fontmap.cpp

diff --git a/core/fpdfdoc/cba_fontmap.cpp b/core/fpdfdoc/cba_fontmap.cpp
index c71c724..705b00a 100644
--- a/core/fpdfdoc/cba_fontmap.cpp
+++ b/core/fpdfdoc/cba_fontmap.cpp
@@ -35,9 +35,6 @@
 
 bool FindNativeTrueTypeFont(ByteString sFontFaceName) {
   CFX_FontMgr* pFontMgr = CFX_GEModule::Get()->GetFontMgr();
-  if (!pFontMgr)
-    return false;
-
   CFX_FontMapper* pFontMapper = pFontMgr->GetBuiltinMapper();
   pFontMapper->LoadInstalledFonts();