Change return types for some rendering Start/Continue methods.

Either return a bool or a CPDF_DIBSource::LoadState. Explain what the
bool return value means.

Change-Id: I6346d0cd61d5faee5290df181cfff3b9e4d5d84e
Reviewed-on: https://pdfium-review.googlesource.com/27851
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>

1 files changed, 13 insertions, 8 deletions

diff --git a/core/fpdfapi/render/cpdf_imagecacheentry.h b/core/fpdfapi/render/cpdf_imagecacheentry.h
index 2bede23e62..dfeeddf408 100644
--- a/core/fpdfapi/render/cpdf_imagecacheentry.h
+++ b/core/fpdfapi/render/cpdf_imagecacheentry.h
@@ -9,11 +9,11 @@
 
 #include <memory>
 
+#include "core/fpdfapi/render/cpdf_dibsource.h"
 #include "core/fxcrt/fx_system.h"
 #include "core/fxcrt/retain_ptr.h"
 #include "core/fxcrt/unowned_ptr.h"
 
-class CFX_DIBSource;
 class CFX_DIBitmap;
 class CPDF_Dictionary;
 class CPDF_Document;
@@ -31,13 +31,18 @@ class CPDF_ImageCacheEntry {
   uint32_t EstimateSize() const { return m_dwCacheSize; }
   uint32_t GetTimeCount() const { return m_dwTimeCount; }
   CPDF_Image* GetImage() const { return m_pImage.Get(); }
-  int StartGetCachedBitmap(CPDF_Dictionary* pFormResources,
-                           CPDF_Dictionary* pPageResources,
-                           bool bStdCS,
-                           uint32_t GroupFamily,
-                           bool bLoadMask,
-                           CPDF_RenderStatus* pRenderStatus);
-  int Continue(IFX_PauseIndicator* pPause, CPDF_RenderStatus* pRenderStatus);
+
+  CPDF_DIBSource::LoadState StartGetCachedBitmap(
+      CPDF_Dictionary* pFormResources,
+      CPDF_Dictionary* pPageResources,
+      bool bStdCS,
+      uint32_t GroupFamily,
+      bool bLoadMask,
+      CPDF_RenderStatus* pRenderStatus);
+
+  // Returns whether to Continue() or not.
+  bool Continue(IFX_PauseIndicator* pPause, CPDF_RenderStatus* pRenderStatus);
+
   RetainPtr<CFX_DIBSource> DetachBitmap();
   RetainPtr<CFX_DIBSource> DetachMask();