summaryrefslogtreecommitdiff
path: root/core/fpdfdoc
diff options
context:
space:
mode:
authorthestig <thestig@chromium.org>2016-10-15 18:31:04 -0700
committerCommit bot <commit-bot@chromium.org>2016-10-15 18:31:04 -0700
commit4dc112664e9d87c0d450ee3349a5091c624a3363 (patch)
tree151cbfb71dc8cda4b31a982bb2caf1985f55f5de /core/fpdfdoc
parentc25a4219431c90a95233a08f25eecc921abbf3ed (diff)
downloadpdfium-4dc112664e9d87c0d450ee3349a5091c624a3363.tar.xz
Fix some nits in cpdf_annotlist.cpp.
Review-Url: https://codereview.chromium.org/2395693002
Diffstat (limited to 'core/fpdfdoc')
-rw-r--r--core/fpdfdoc/cpdf_annotlist.cpp13
1 files changed, 9 insertions, 4 deletions
diff --git a/core/fpdfdoc/cpdf_annotlist.cpp b/core/fpdfdoc/cpdf_annotlist.cpp
index f73ace9791..b626115de1 100644
--- a/core/fpdfdoc/cpdf_annotlist.cpp
+++ b/core/fpdfdoc/cpdf_annotlist.cpp
@@ -6,6 +6,9 @@
#include "core/fpdfdoc/cpdf_annotlist.h"
+#include <memory>
+#include <utility>
+
#include "core/fpdfapi/page/cpdf_page.h"
#include "core/fpdfapi/parser/cpdf_document.h"
#include "core/fpdfapi/parser/cpdf_reference.h"
@@ -64,11 +67,13 @@ CPDF_AnnotList::CPDF_AnnotList(CPDF_Page* pPage)
CPDF_Dictionary* pRoot = m_pDocument->GetRoot();
CPDF_Dictionary* pAcroForm = pRoot->GetDictFor("AcroForm");
- FX_BOOL bRegenerateAP =
- pAcroForm && pAcroForm->GetBooleanFor("NeedAppearances");
+ bool bRegenerateAP = pAcroForm && pAcroForm->GetBooleanFor("NeedAppearances");
for (size_t i = 0; i < pAnnots->GetCount(); ++i) {
CPDF_Dictionary* pDict = ToDictionary(pAnnots->GetDirectObjectAt(i));
- if (!pDict || pDict->GetStringFor("Subtype") == "Popup") {
+ if (!pDict)
+ continue;
+ const CFX_ByteString subtype = pDict->GetStringFor("Subtype");
+ if (subtype == "Popup") {
// Skip creating Popup annotations in the PDF document since PDFium
// provides its own Popup annotations.
continue;
@@ -76,7 +81,7 @@ CPDF_AnnotList::CPDF_AnnotList(CPDF_Page* pPage)
pAnnots->ConvertToIndirectObjectAt(i, m_pDocument);
m_AnnotList.push_back(
std::unique_ptr<CPDF_Annot>(new CPDF_Annot(pDict, m_pDocument, false)));
- if (bRegenerateAP && pDict->GetStringFor("Subtype") == "Widget" &&
+ if (bRegenerateAP && subtype == "Widget" &&
CPDF_InterForm::IsUpdateAPEnabled()) {
FPDF_GenerateAP(m_pDocument, pDict);
}