Cleanup CPDFXFA and CPDF document methods

This CL renames and cleans up some methods that are similar between
CPDF_Document and CPDFXFA_Document.

Review-Url: https://codereview.chromium.org/2351673004

3 files changed, 16 insertions, 16 deletions

diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp
index 3bf116e64a..7ee3cbb859 100644
--- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp
+++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp
@@ -61,7 +61,7 @@ void CPDFXFA_DocEnvironment::InvalidateRect(CXFA_FFPageView* pPageView,
   if (m_pDocument->GetDocType() != DOCTYPE_DYNAMIC_XFA)
     return;
 
-  CPDFXFA_Page* pPage = m_pDocument->GetPage(pPageView);
+  CPDFXFA_Page* pPage = m_pDocument->GetXFAPage(pPageView);
   if (!pPage)
     return;
 
@@ -93,7 +93,7 @@ void CPDFXFA_DocEnvironment::DisplayCaret(CXFA_FFWidget* hWidget,
   if (!pPageView)
     return;
 
-  CPDFXFA_Page* pPage = m_pDocument->GetPage(pPageView);
+  CPDFXFA_Page* pPage = m_pDocument->GetXFAPage(pPageView);
   if (!pPage)
     return;
 
@@ -118,7 +118,7 @@ FX_BOOL CPDFXFA_DocEnvironment::GetPopupPos(CXFA_FFWidget* hWidget,
   if (!pXFAPageView)
     return FALSE;
 
-  CPDFXFA_Page* pPage = m_pDocument->GetPage(pXFAPageView);
+  CPDFXFA_Page* pPage = m_pDocument->GetXFAPage(pXFAPageView);
   if (!pPage)
     return FALSE;
 
@@ -231,7 +231,7 @@ FX_BOOL CPDFXFA_DocEnvironment::PopupMenu(CXFA_FFWidget* hWidget,
   if (!pXFAPageView)
     return FALSE;
 
-  CPDFXFA_Page* pPage = m_pDocument->GetPage(pXFAPageView);
+  CPDFXFA_Page* pPage = m_pDocument->GetXFAPage(pXFAPageView);
   if (!pPage)
     return FALSE;
 
@@ -302,7 +302,7 @@ void CPDFXFA_DocEnvironment::WidgetPostAdd(CXFA_FFWidget* hWidget,
   if (!pPageView)
     return;
 
-  CPDFXFA_Page* pXFAPage = m_pDocument->GetPage(pPageView);
+  CPDFXFA_Page* pXFAPage = m_pDocument->GetXFAPage(pPageView);
   if (!pXFAPage)
     return;
 
@@ -318,7 +318,7 @@ void CPDFXFA_DocEnvironment::WidgetPreRemove(CXFA_FFWidget* hWidget,
   if (!pPageView)
     return;
 
-  CPDFXFA_Page* pXFAPage = m_pDocument->GetPage(pPageView);
+  CPDFXFA_Page* pXFAPage = m_pDocument->GetXFAPage(pPageView);
   if (!pXFAPage)
     return;
 
diff --git a/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp b/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp
index ed9407858b..8db16bf4e9 100644
--- a/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp
+++ b/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp
@@ -108,7 +108,7 @@ FX_BOOL CPDFXFA_Document::LoadXFADoc() {
   return TRUE;
 }
 
-int CPDFXFA_Document::GetPageCount() {
+int CPDFXFA_Document::GetPageCount() const {
   if (!m_pPDFDoc && !m_pXFADoc)
     return 0;
 
@@ -125,7 +125,7 @@ int CPDFXFA_Document::GetPageCount() {
   }
 }
 
-CPDFXFA_Page* CPDFXFA_Document::GetPage(int page_index) {
+CPDFXFA_Page* CPDFXFA_Document::GetXFAPage(int page_index) {
   if (page_index < 0)
     return nullptr;
 
@@ -151,7 +151,7 @@ CPDFXFA_Page* CPDFXFA_Document::GetPage(int page_index) {
   return pPage;
 }
 
-CPDFXFA_Page* CPDFXFA_Document::GetPage(CXFA_FFPageView* pPage) {
+CPDFXFA_Page* CPDFXFA_Document::GetXFAPage(CXFA_FFPageView* pPage) const {
   if (!pPage)
     return nullptr;
 
diff --git a/fpdfsdk/fpdfxfa/include/fpdfxfa_doc.h b/fpdfsdk/fpdfxfa/include/fpdfxfa_doc.h
index 77597d47c2..ed788eac55 100644
--- a/fpdfsdk/fpdfxfa/include/fpdfxfa_doc.h
+++ b/fpdfsdk/fpdfxfa/include/fpdfxfa_doc.h
@@ -38,16 +38,16 @@ class CPDFXFA_Document {
   CPDF_Document* GetPDFDoc() { return m_pPDFDoc.get(); }
   CXFA_FFDoc* GetXFADoc() { return m_pXFADoc.get(); }
   CXFA_FFDocView* GetXFADocView() { return m_pXFADocView; }
-
-  int GetPageCount();
-  CPDFXFA_Page* GetPage(int page_index);
-  CPDFXFA_Page* GetPage(CXFA_FFPageView* pPage);
+  CPDFSDK_Document* GetSDKDocument(CPDFSDK_Environment* pFormFillEnv);
+  int GetDocType() const { return m_iDocType; }
 
   void DeletePage(int page_index);
-  void RemovePage(CPDFXFA_Page* page);
-  int GetDocType() { return m_iDocType; }
+  int GetPageCount() const;
 
-  CPDFSDK_Document* GetSDKDocument(CPDFSDK_Environment* pFormFillEnv);
+  CPDFXFA_Page* GetXFAPage(int page_index);
+  CPDFXFA_Page* GetXFAPage(CXFA_FFPageView* pPage) const;
+
+  void RemovePage(CPDFXFA_Page* page);
 
   void ClearChangeMark();