commit 6edd938572566bd707cbcbd90af12a32e525b2fc
author Lei Zhang <thestig@chromium.org> Thu Jan 10 19:38:44 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Jan 10 19:38:44 2019 +0000
tree e548f12719782d46369b053a8ae2f925416f76ee
parent cb13a088dd164c92e522d40394a23f8882eec643

Fix default CPDFSDK_FormFillEnvironment::SetTimer() return value.

On failure, it should return CFX_SystemHandler::kInvalidTimerID, which
is 0, not -1.

Change-Id: I955a97e2faa4a5cf0176ed49bbb45af8964e3c42
Reviewed-on: https://pdfium-review.googlesource.com/c/48033
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

fpdfsdk/cpdfsdk_formfillenvironment.cpp

diff --git a/fpdfsdk/cpdfsdk_formfillenvironment.cpp b/fpdfsdk/cpdfsdk_formfillenvironment.cpp
index 6d3aab3..f016ff4 100644
--- a/fpdfsdk/cpdfsdk_formfillenvironment.cpp
+++ b/fpdfsdk/cpdfsdk_formfillenvironment.cpp
@@ -278,7 +278,7 @@
                                           TimerCallback lpTimerFunc) {
   if (m_pInfo && m_pInfo->FFI_SetTimer)
     return m_pInfo->FFI_SetTimer(m_pInfo, uElapse, lpTimerFunc);
-  return -1;
+  return CFX_SystemHandler::kInvalidTimerID;
 }
 
 void CPDFSDK_FormFillEnvironment::KillTimer(int nTimerID) {