commit c25a4219431c90a95233a08f25eecc921abbf3ed
author Tom Sepez <tsepez@chromium.org> Fri Oct 14 17:45:56 2016 -0700
committer Tom Sepez <tsepez@chromium.org> Fri Oct 14 17:45:56 2016 -0700
tree bd2ad923d527440785c19623fe2d19cc1a70a92d
parent 62f367348ff8ff1e64c6f52c1ee5f77c3a89edc1

Revert "Update CPDF_IndirectObjectHolder APIs for unique objects."

This reverts commit 3ba098595ae56b64eacc0c25ab76b89a4d78d920.

TBR=thestig@chromium.org,weili@chromium.org

Review URL: https://codereview.chromium.org/2424533003 .

fpdfsdk/cpdfsdk_baannot.cpp

diff --git a/fpdfsdk/cpdfsdk_baannot.cpp b/fpdfsdk/cpdfsdk_baannot.cpp
index e704822..72468e1 100644
--- a/fpdfsdk/cpdfsdk_baannot.cpp
+++ b/fpdfsdk/cpdfsdk_baannot.cpp
@@ -316,9 +316,10 @@
   }
 
   if (!pStream) {
+    pStream = new CPDF_Stream;
     CPDF_Document* pDoc = m_pPageView->GetPDFDocument();
-    pStream = pDoc->AddIndirectStream();
-    pParentDict->SetReferenceFor(sAPType, pDoc, pStream);
+    pParentDict->SetReferenceFor(sAPType, pDoc,
+                                 pDoc->AddIndirectObject(pStream));
   }
 
   CPDF_Dictionary* pStreamDict = pStream->GetDict();
@@ -353,10 +354,8 @@
   CPDF_Dictionary* pDict = action.GetDict();
   if (pDict != m_pAnnot->GetAnnotDict()->GetDictFor("A")) {
     CPDF_Document* pDoc = m_pPageView->GetPDFDocument();
-
-    // TODO(tsepez): check |pDict| ownership.
-    m_pAnnot->GetAnnotDict()->SetReferenceFor(
-        "A", pDoc, pDoc->AddIndirectObject(UniqueDictionary(pDict)));
+    m_pAnnot->GetAnnotDict()->SetReferenceFor("A", pDoc,
+                                              pDoc->AddIndirectObject(pDict));
   }
 }