Remove CPDF_Reference::Create(), make logic match master
authorTom Sepez <tsepez@chromium.org>
Mon, 26 Oct 2015 22:09:33 +0000 (15:09 -0700)
committerTom Sepez <tsepez@chromium.org>
Mon, 26 Oct 2015 22:09:33 +0000 (15:09 -0700)
Makes the files in this CL identical to master, since there are
no XFA specifics.

R=thestig@chromium.org

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

core/include/fpdfapi/fpdf_objects.h
core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp
core/src/fpdfapi/fpdf_parser/fpdf_parser_objects.cpp
core/src/fpdfdoc/doc_annot.cpp

index 7287814..419d5fc 100644 (file)
@@ -585,10 +585,6 @@ class CPDF_Null : public CPDF_Object {
 };
 class CPDF_Reference : public CPDF_Object {
  public:
-  static CPDF_Reference* Create(CPDF_IndirectObjects* pDoc, int objnum) {
-    return new CPDF_Reference(pDoc, objnum);
-  }
-
   CPDF_Reference(CPDF_IndirectObjects* pDoc, int objnum)
       : CPDF_Object(PDFOBJ_REFERENCE), m_pObjList(pDoc), m_RefObjNum(objnum) {}
 
index 0f6d973..9c1c292 100644 (file)
@@ -1026,8 +1026,7 @@ static int InsertDeletePDFPage(CPDF_Document* pDoc,
     if (pKid->GetString("Type") == FX_BSTRC("Page")) {
       if (nPagesToGo == 0) {
         if (bInsert) {
-          pKidList->InsertAt(i,
-                             CPDF_Reference::Create(pDoc, pPage->GetObjNum()));
+          pKidList->InsertAt(i, new CPDF_Reference(pDoc, pPage->GetObjNum()));
           pPage->SetAtReference("Parent", pDoc, pPages->GetObjNum());
         } else {
           pKidList->RemoveAt(i);
index 2bf529a..07b1a20 100644 (file)
@@ -521,7 +521,7 @@ void CPDF_Array::SetAt(FX_DWORD i,
     pOld->Release();
   if (pObj->GetObjNum()) {
     ASSERT(pObjs != NULL);
-    pObj = CPDF_Reference::Create(pObjs, pObj->GetObjNum());
+    pObj = new CPDF_Reference(pObjs, pObj->GetObjNum());
   }
   m_Objects.SetAt(i, pObj);
 }
@@ -531,7 +531,7 @@ void CPDF_Array::InsertAt(FX_DWORD index,
   ASSERT(pObj != NULL);
   if (pObj->GetObjNum()) {
     ASSERT(pObjs != NULL);
-    pObj = CPDF_Reference::Create(pObjs, pObj->GetObjNum());
+    pObj = new CPDF_Reference(pObjs, pObj->GetObjNum());
   }
   m_Objects.InsertAt(index, pObj);
 }
@@ -539,7 +539,7 @@ void CPDF_Array::Add(CPDF_Object* pObj, CPDF_IndirectObjects* pObjs) {
   ASSERT(pObj != NULL);
   if (pObj->GetObjNum()) {
     ASSERT(pObjs != NULL);
-    pObj = CPDF_Reference::Create(pObjs, pObj->GetObjNum());
+    pObj = new CPDF_Reference(pObjs, pObj->GetObjNum());
   }
   m_Objects.Add(pObj);
 }
@@ -731,7 +731,7 @@ void CPDF_Dictionary::SetAt(const CFX_ByteStringC& key,
   if (pObj) {
     if (pObj->GetObjNum()) {
       ASSERT(pObjs != NULL);
-      pObj = CPDF_Reference::Create(pObjs, pObj->GetObjNum());
+      pObj = new CPDF_Reference(pObjs, pObj->GetObjNum());
     }
     m_Map.SetAt(key, pObj);
   } else {
@@ -967,15 +967,14 @@ FX_BOOL CPDF_Stream::Identical(CPDF_Stream* pOther) const {
     IFX_FileRead* pFile = NULL;
     uint8_t* pBuf = NULL;
     FX_DWORD offset = 0;
-    if (m_GenNum != (FX_DWORD)-1) {
-      pFile = m_pFile;
-      pBuf = pOther->m_pDataBuf;
-      offset = m_FileOffset;
-    }
     if (pOther->m_GenNum != (FX_DWORD)-1) {
       pFile = pOther->m_pFile;
       pBuf = m_pDataBuf;
       offset = pOther->m_FileOffset;
+    } else if (m_GenNum != (FX_DWORD)-1) {
+      pFile = m_pFile;
+      pBuf = pOther->m_pDataBuf;
+      offset = m_FileOffset;
     }
     if (NULL == pBuf) {
       return FALSE;
index dc5ccb8..d37b1aa 100644 (file)
@@ -30,10 +30,7 @@ CPDF_AnnotList::CPDF_AnnotList(CPDF_Page* pPage) {
     FX_DWORD dwObjNum = pDict->GetObjNum();
     if (dwObjNum == 0) {
       dwObjNum = m_pDocument->AddIndirectObject(pDict);
-      CPDF_Reference* pAction = CPDF_Reference::Create(m_pDocument, dwObjNum);
-      if (pAction == NULL) {
-        break;
-      }
+      CPDF_Reference* pAction = new CPDF_Reference(m_pDocument, dwObjNum);
       pAnnots->InsertAt(i, pAction);
       pAnnots->RemoveAt(i + 1);
       pDict = pAnnots->GetDict(i);