commit 2e118e8f73f939fa93d144470e7fa31228e7e74d
author Tom Sepez <tsepez@chromium.org> Tue May 01 21:24:14 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue May 01 21:24:14 2018 +0000
tree 002d889a771caded90403a50b6a16a3b6015ea02
parent bbcbc72765ea6d95d3f6ff6989c0d6477f3f929c

Add CPDF_Document::Extension::DeletePage()

Replaces one compile-time #ifdef XFA with a dynamic check and
a call through a virtual API that prevents the CPDF code from
knowing anything about the XFA code.

Change-Id: If0ff9b6918b908b3eac824fe1d525c6d4f7316e7
Reviewed-on: https://pdfium-review.googlesource.com/31890
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

fpdfsdk/fpdf_editpage.cpp

diff --git a/fpdfsdk/fpdf_editpage.cpp b/fpdfsdk/fpdf_editpage.cpp
index 19a61bc..944dbdc 100644
--- a/fpdfsdk/fpdf_editpage.cpp
+++ b/fpdfsdk/fpdf_editpage.cpp
@@ -157,14 +157,11 @@
   if (!pDoc)
     return;
 
-#ifdef PDF_ENABLE_XFA
-  CPDFXFA_Context* pContext =
-      static_cast<CPDFXFA_Context*>(pDoc->GetExtension());
-  if (pContext) {
-    pContext->DeletePage(page_index);
+  CPDF_Document::Extension* pExtension = pDoc->GetExtension();
+  if (pExtension) {
+    pExtension->DeletePage(page_index);
     return;
   }
-#endif  // PDF_ENABLE_XFA
 
   pDoc->DeletePage(page_index);
 }