Cleanup CPDF_DocPageData release methods and callers.

core/fpdfapi/fpdf_font/cpdf_font.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/fpdf_font/include/cpdf_font.h"
 
+#include <memory>
+
 #include "core/fpdfapi/fpdf_font/cpdf_truetypefont.h"
 #include "core/fpdfapi/fpdf_font/cpdf_type1font.h"
 #include "core/fpdfapi/fpdf_font/cpdf_type3font.h"
 #include "core/fpdfapi/fpdf_font/font_int.h"
 #include "core/fpdfapi/fpdf_font/include/cpdf_fontencoding.h"
 #include "core/fpdfapi/fpdf_page/cpdf_pagemodule.h"
 #include "core/fpdfapi/fpdf_page/pageint.h"
 #include "core/fpdfapi/fpdf_parser/include/cpdf_array.h"
 #include "core/fpdfapi/fpdf_parser/include/cpdf_dictionary.h"
 #include "core/fpdfapi/fpdf_parser/include/cpdf_document.h"
 #include "core/fpdfapi/fpdf_parser/include/cpdf_name.h"
 #include "core/fpdfapi/fpdf_parser/include/cpdf_stream_acc.h"
 #include "core/fpdfapi/include/cpdf_modulemgr.h"
 #include "core/fxge/include/fx_freetype.h"
 
 namespace {
 
-const uint8_t ChineseFontNames[][5] = {{0xCB, 0xCE, 0xCC, 0xE5, 0x00},
-                                       {0xBF, 0xAC, 0xCC, 0xE5, 0x00},
-                                       {0xBA, 0xDA, 0xCC, 0xE5, 0x00},
-                                       {0xB7, 0xC2, 0xCB, 0xCE, 0x00},
-                                       {0xD0, 0xC2, 0xCB, 0xCE, 0x00}};
+const uint8_t kChineseFontNames[][5] = {{0xCB, 0xCE, 0xCC, 0xE5, 0x00},
+                                        {0xBF, 0xAC, 0xCC, 0xE5, 0x00},
+                                        {0xBA, 0xDA, 0xCC, 0xE5, 0x00},
+                                        {0xB7, 0xC2, 0xCB, 0xCE, 0x00},
+                                        {0xD0, 0xC2, 0xCB, 0xCE, 0x00}};
 
-FX_BOOL GetPredefinedEncoding(int& basemap, const CFX_ByteString& value) {
+void GetPredefinedEncoding(int& basemap, const CFX_ByteString& value) {
   if (value == "WinAnsiEncoding")
     basemap = PDFFONT_ENCODING_WINANSI;
   else if (value == "MacRomanEncoding")
     basemap = PDFFONT_ENCODING_MACROMAN;
   else if (value == "MacExpertEncoding")
     basemap = PDFFONT_ENCODING_MACEXPERT;
   else if (value == "PDFDocEncoding")
     basemap = PDFFONT_ENCODING_PDFDOC;
-  else
-    return FALSE;
-  return TRUE;
 }
 
 }  // namespace
 
 CPDF_Font::CPDF_Font()
     : m_pFontFile(nullptr),
       m_pFontDict(nullptr),
       m_bToUnicodeLoaded(FALSE),
       m_Flags(0),
       m_StemV(0),
       m_Ascent(0),
       m_Descent(0),
       m_ItalicAngle(0) {}
 
 CPDF_Font::~CPDF_Font() {
   if (m_pFontFile) {
     m_pDocument->GetPageData()->ReleaseFontFileStreamAcc(
-        const_cast<CPDF_Stream*>(m_pFontFile->GetStream()->AsStream()));
+        m_pFontFile->GetStream()->AsStream());
   }
 }
 
 bool CPDF_Font::IsType1Font() const {
   return false;
 }
 
 bool CPDF_Font::IsTrueTypeFont() const {
   return false;
 }
@@ skipping 143 matching lines @@
     return;
 
   m_pFontFile = m_pDocument->LoadFontFile(pFontFile);
   if (!m_pFontFile)
     return;
 
   const uint8_t* pFontData = m_pFontFile->GetData();
   uint32_t dwFontSize = m_pFontFile->GetSize();
   if (!m_Font.LoadEmbedded(pFontData, dwFontSize)) {
     m_pDocument->GetPageData()->ReleaseFontFileStreamAcc(
-        const_cast<CPDF_Stream*>(m_pFontFile->GetStream()->AsStream()));
+        m_pFontFile->GetStream()->AsStream());
     m_pFontFile = nullptr;
   }
 }
 
 void CPDF_Font::CheckFontMetrics() {
   if (m_FontBBox.top == 0 && m_FontBBox.bottom == 0 && m_FontBBox.left == 0 &&
       m_FontBBox.right == 0) {
     FXFT_Face face = m_Font.GetFace();
     if (face) {
       m_FontBBox.left = TT2PDF(FXFT_Get_Face_xMin(face), face);
@@ skipping 79 matching lines @@
   pFontGlobals->Set(pDoc, font_id, pFont);
   return pFont;
 }
 
 CPDF_Font* CPDF_Font::CreateFontF(CPDF_Document* pDoc,
                                   CPDF_Dictionary* pFontDict) {
   CFX_ByteString type = pFontDict->GetStringBy("Subtype");
   std::unique_ptr<CPDF_Font> pFont;
   if (type == "TrueType") {
     CFX_ByteString tag = pFontDict->GetStringBy("BaseFont").Left(4);
-    for (size_t i = 0; i < FX_ArraySize(ChineseFontNames); ++i) {
-      if (tag == CFX_ByteString(ChineseFontNames[i], 4)) {
+    for (size_t i = 0; i < FX_ArraySize(kChineseFontNames); ++i) {
+      if (tag == CFX_ByteString(kChineseFontNames[i], 4)) {
         CPDF_Dictionary* pFontDesc = pFontDict->GetDictBy("FontDescriptor");
         if (!pFontDesc || !pFontDesc->KeyExist("FontFile2"))
           pFont.reset(new CPDF_CIDFont);
         break;
       }
     }
     if (!pFont)
       pFont.reset(new CPDF_TrueTypeFont);
   } else if (type == "Type3") {
     pFont.reset(new CPDF_Type3Font);
@@ skipping 106 matching lines @@
   }
 
   if (pCharNames && !pCharNames[charcode].IsEmpty())
     return pCharNames[charcode].c_str();
 
   const FX_CHAR* name = nullptr;
   if (iBaseEncoding)
     name = PDF_CharNameFromPredefinedCharSet(iBaseEncoding, charcode);
   return name && name[0] ? name : nullptr;
 }