Encapsulate CPDF_StreamAcc::LoadAllData().

Make it a private method, and add public methods so only limited
combinations of LoadAllData() arguments are possible.

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

1 files changed, 14 insertions, 0 deletions

diff --git a/core/fpdfapi/parser/cpdf_stream_acc.cpp b/core/fpdfapi/parser/cpdf_stream_acc.cpp
index 1083e3177e..78bcb57e0b 100644
--- a/core/fpdfapi/parser/cpdf_stream_acc.cpp
+++ b/core/fpdfapi/parser/cpdf_stream_acc.cpp
@@ -20,6 +20,11 @@ CPDF_StreamAcc::~CPDF_StreamAcc() {
 void CPDF_StreamAcc::LoadAllData(bool bRawAccess,
                                  uint32_t estimated_size,
                                  bool bImageAcc) {
+  if (bRawAccess) {
+    ASSERT(!estimated_size);
+    ASSERT(!bImageAcc);
+  }
+
   if (!m_pStream)
     return;
 
@@ -63,6 +68,15 @@ void CPDF_StreamAcc::LoadAllDataFiltered() {
   LoadAllData(false, 0, false);
 }
 
+void CPDF_StreamAcc::LoadAllDataFilteredWithEstimatedSize(
+    uint32_t estimated_size) {
+  LoadAllData(false, estimated_size, false);
+}
+
+void CPDF_StreamAcc::LoadAllDataImageAcc(uint32_t estimated_size) {
+  LoadAllData(false, estimated_size, true);
+}
+
 void CPDF_StreamAcc::LoadAllDataRaw() {
   LoadAllData(true, 0, false);
 }