Unify some code

core/fpdfapi/parser/cpdf_hint_tables.cpp

 // Copyright 2016 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "core/fpdfapi/parser/cpdf_hint_tables.h"
 
 #include <limits>
 
 #include "core/fpdfapi/parser/cpdf_array.h"
 #include "core/fpdfapi/parser/cpdf_data_avail.h"
 #include "core/fpdfapi/parser/cpdf_dictionary.h"
 #include "core/fpdfapi/parser/cpdf_document.h"
+#include "core/fpdfapi/parser/cpdf_linearized.h"
 #include "core/fpdfapi/parser/cpdf_stream.h"
 #include "core/fpdfapi/parser/cpdf_stream_acc.h"
 #include "core/fxcrt/fx_safe_types.h"
 #include "third_party/base/numerics/safe_conversions.h"
 
 namespace {
 
 bool CanReadFromBitStream(const CFX_BitStream* hStream,
                           const FX_SAFE_UINT32& bits) {
   return bits.IsValid() && hStream->BitsRemaining() >= bits.ValueOrDie();
 }
 
 // Sanity check values from the page table header. The note in the PDF 1.7
 // reference for Table F.3 says the valid range is only 0 through 32. Though 0
 // is not useful either.
 bool IsValidPageOffsetHintTableBitCount(uint32_t bits) {
   return bits > 0 && bits <= 32;
 }
 
 }  // namespace
 
 CPDF_HintTables::CPDF_HintTables(CPDF_DataAvail* pDataAvail,
-                                 CPDF_Dictionary* pLinearized)
+                                 CPDF_Linearized* pLinearized)
     : m_pDataAvail(pDataAvail),
-      m_pLinearizedDict(pLinearized),
+      m_pLinearized(pLinearized),
       m_nFirstPageSharedObjs(0),
       m_szFirstPageObjOffset(0) {
-  ASSERT(m_pLinearizedDict);
+  ASSERT(m_pLinearized);
 }
 
 CPDF_HintTables::~CPDF_HintTables() {}
 
 uint32_t CPDF_HintTables::GetItemLength(
     uint32_t index,
     const std::vector<FX_FILESIZE>& szArray) {
   if (szArray.size() < 2 || index > szArray.size() - 2 ||
       szArray[index] > szArray[index + 1]) {
     return 0;
@@ skipping 427 matching lines @@
     return false;
   }
 
   CFX_BitStream bs;
   bs.Init(acc.GetData(), size);
   return ReadPageHintTable(&bs) &&
          ReadSharedObjHintTable(&bs, shared_hint_table_offset);
 }
 
 int CPDF_HintTables::GetEndOfFirstPageOffset() const {
-  CPDF_Object* pOffsetE = m_pLinearizedDict->GetDirectObjectFor("E");
-  return pOffsetE ? pOffsetE->GetInteger() : -1;
+  return static_cast<int>(m_pLinearized->GetFirstPageEndOffset());
 }
 
 int CPDF_HintTables::GetNumberOfPages() const {
-  CPDF_Object* pPageNum = m_pLinearizedDict->GetDirectObjectFor("N");
-  return pPageNum ? pPageNum->GetInteger() : 0;
+  return static_cast<int>(m_pLinearized->GetPageCount());
 }
 
 int CPDF_HintTables::GetFirstPageObjectNumber() const {
-  CPDF_Object* pFirstPageObj = m_pLinearizedDict->GetDirectObjectFor("O");
-  return pFirstPageObj ? pFirstPageObj->GetInteger() : -1;
+  return static_cast<int>(m_pLinearized->GetFirstPageObjNum());
 }
 
 int CPDF_HintTables::GetFirstPageNumber() const {
-  CPDF_Object* pFirstPageNum = m_pLinearizedDict->GetDirectObjectFor("P");
-  return pFirstPageNum ? pFirstPageNum->GetInteger() : 0;
+  return static_cast<int>(m_pLinearized->GetFirstPageNo());
 }
 
 int CPDF_HintTables::ReadPrimaryHintStreamOffset() const {
-  return ReadPrimaryHintStream(0);
+  return static_cast<int>(m_pLinearized->GetHintStart());
 }
 
 int CPDF_HintTables::ReadPrimaryHintStreamLength() const {
-  return ReadPrimaryHintStream(1);
+  return static_cast<int>(m_pLinearized->GetHintLength());
 }
-
-int CPDF_HintTables::ReadPrimaryHintStream(int index) const {
-  CPDF_Array* pRange = m_pLinearizedDict->GetArrayFor("H");
-  if (!pRange)
-    return -1;
-
-  CPDF_Object* pStreamLen = pRange->GetDirectObjectAt(index);
-  return pStreamLen ? pStreamLen->GetInteger() : -1;
-}