Update CPDF_IndirectObjectHolder APIs for unique objects

core/fpdfapi/parser/cpdf_array.h

Index: core/fpdfapi/parser/cpdf_array.h
diff --git a/core/fpdfapi/parser/cpdf_array.h b/core/fpdfapi/parser/cpdf_array.h
index f473e268005d4b0bb76e3d58e55e0f930d894d45..aee794fbc2a8f363fa38eb8fd15b388847152972 100644
--- a/core/fpdfapi/parser/cpdf_array.h
+++ b/core/fpdfapi/parser/cpdf_array.h
@@ -54,6 +54,7 @@ class CPDF_Array : public CPDF_Object {
   void AddString(const CFX_ByteString& str);
   void AddName(const CFX_ByteString& str);
   void AddReference(CPDF_IndirectObjectHolder* pDoc, uint32_t objnum);
+  void AddReference(CPDF_IndirectObjectHolder* pDoc, const CPDF_Object* objnum);

[Lei Zhang, 2016/10/14 23:18:04]

Is there any advantage to adding this, versus making the caller call
AddReference(pDoc, pObj->GetObjNum()); ? Convenience?

[Lei Zhang, 2016/10/14 23:18:04]

s/objnum/pObj/ ?

[Tom Sepez, 2016/10/14 23:42:29]

Convenience, also probably codesize as we replace N scattered calls to this with
one consolidated one.

[Tom Sepez, 2016/10/14 23:42:29]

Done.

 
   iterator begin() { return m_Objects.begin(); }
   iterator end() { return m_Objects.end(); }