commit 270283669b1f53edfce8b96a579e973c5de5f205
author Henrique Nakashima <hnakashima@chromium.org> Thu Feb 01 22:00:33 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Feb 01 22:00:33 2018 +0000
tree 527846f4d936e1b878aaaf129ef2cd1223395489
parent e99ee7f3e550f7706a3e2d8105ccc55553a9badb

Change internal usages of GetPageIndexDeprecated() to new method.

The new GetDestPageIndex() method does the same thing, but has a
consistent form of returning an error (returns -1).

Bug: pdfium:938
Change-Id: I31583e1c544d9173a28582b849edd5f73c40e174
Reviewed-on: https://pdfium-review.googlesource.com/25070
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>

fpdfsdk/fsdk_actionhandler.cpp

diff --git a/fpdfsdk/fsdk_actionhandler.cpp b/fpdfsdk/fsdk_actionhandler.cpp
index 49e0041..5999763 100644
--- a/fpdfsdk/fsdk_actionhandler.cpp
+++ b/fpdfsdk/fsdk_actionhandler.cpp
@@ -362,7 +362,7 @@
   ASSERT(pPDFDocument);
 
   CPDF_Dest MyDest = action.GetDest(pPDFDocument);
-  int nPageIndex = MyDest.GetPageIndexDeprecated(pPDFDocument);
+  int nPageIndex = MyDest.GetDestPageIndex(pPDFDocument);
   int nFitType = MyDest.GetZoomMode();
   const CPDF_Array* pMyArray = ToArray(MyDest.GetObject());
   std::vector<float> posArray;

fxjs/cjs_document.cpp

diff --git a/fxjs/cjs_document.cpp b/fxjs/cjs_document.cpp
index 9af0f07..980edd9 100644
--- a/fxjs/cjs_document.cpp
+++ b/fxjs/cjs_document.cpp
@@ -1467,7 +1467,7 @@
       scrollPositionArray.push_back(arrayObject->GetFloatAt(i));
   }
   pRuntime->BeginBlock();
-  m_pFormFillEnv->DoGoToAction(dest.GetPageIndexDeprecated(pDocument),
+  m_pFormFillEnv->DoGoToAction(dest.GetDestPageIndex(pDocument),
                                dest.GetZoomMode(), scrollPositionArray.data(),
                                scrollPositionArray.size());
   pRuntime->EndBlock();