Add CPDF_Document::GetPage() unittests

core/fpdfapi/parser/cpdf_document.h

Index: core/fpdfapi/parser/cpdf_document.h
diff --git a/core/fpdfapi/parser/cpdf_document.h b/core/fpdfapi/parser/cpdf_document.h
index c557a56013f71da4dc7106ad5faa97eb154c56ae..d35375be6aeab47c06182c56c89ceab3a306b458 100644
--- a/core/fpdfapi/parser/cpdf_document.h
+++ b/core/fpdfapi/parser/cpdf_document.h
@@ -104,6 +104,7 @@ class CPDF_Document : public CPDF_IndirectObjectHolder {
 
  private:

[Tom Sepez, 2016/10/21 21:33:38]

maybe this just becomes protected:, as we have subclasses for testing.

[npm, 2016/10/21 21:46:00]

protected and no friends is preferred then?

[Tom Sepez, 2016/10/21 21:48:33]

either way.

[npm, 2016/10/21 22:12:03]

Done.

   friend class CPDF_TestDocument;
+  friend class CPDF_DocumentTest;

[Tom Sepez, 2016/10/21 21:31:40]

nit: can we call this class CPDF_TestDocumentForPages ?

[npm, 2016/10/21 22:12:03]

Done.

 
   // Retrieve page count information by getting count value from the tree nodes
   int RetrievePageCount() const;