diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-03-01 14:24:03 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-03-01 19:52:34 +0000 |
| commit | fed39cf4a23341cf9cb5a5b432248b4247022282 (patch) | |
| tree | fecfa77b0371357516521f4c3b1cc3f9d247ecb9 | |
| parent | ef81390393ef5fed1ba168cff081d459eed9f260 (diff) | |
| download | pdfium-fed39cf4a23341cf9cb5a5b432248b4247022282.tar.xz | |
Fix font manager negation issue
In https://codereview.chromium.org/2559903002 we refactored the font manager
in XFA but got part of the negation wrong. Fixup logic.
Change-Id: Idd51ff846fe64a56908ec26bf2798c1df8433611
Reviewed-on: https://pdfium-review.googlesource.com/2891
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
| -rw-r--r-- | xfa/fgas/font/cfgas_fontmgr.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xfa/fgas/font/cfgas_fontmgr.cpp b/xfa/fgas/font/cfgas_fontmgr.cpp index bf243a49c6..57b499fb10 100644 --- a/xfa/fgas/font/cfgas_fontmgr.cpp +++ b/xfa/fgas/font/cfgas_fontmgr.cpp @@ -516,7 +516,7 @@ CFX_ByteString CFX_FontSourceEnum_File::GetNextFile() { FX_CloseFolder(pCurHandle); if (!m_FolderQueue.empty()) m_FolderQueue.pop_back(); - if (!m_FolderQueue.empty()) { + if (m_FolderQueue.empty()) { if (!m_FolderPaths.empty()) m_FolderPaths.pop_back(); return !m_FolderPaths.empty() ? GetNextFile() : ""; |