Remove CPDF_DocRenderData::Clear().
The dtor will do roughly the same thing anyway.
Change-Id: Ia9c8ef3503f4a40ab4db7b767fb2f79923704f8c
Reviewed-on: https://pdfium-review.googlesource.com/c/45311
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/core/fpdfapi/render/cpdf_docrenderdata.cpp b/core/fpdfapi/render/cpdf_docrenderdata.cpp
index e713866..5f19210 100644
--- a/core/fpdfapi/render/cpdf_docrenderdata.cpp
+++ b/core/fpdfapi/render/cpdf_docrenderdata.cpp
@@ -28,24 +28,7 @@
CPDF_DocRenderData::CPDF_DocRenderData(CPDF_Document* pPDFDoc)
: m_pPDFDoc(pPDFDoc) {}
-CPDF_DocRenderData::~CPDF_DocRenderData() {
- Clear(true);
-}
-
-void CPDF_DocRenderData::Clear(bool bRelease) {
- for (auto it = m_Type3FaceMap.begin(); it != m_Type3FaceMap.end();) {
- auto curr_it = it++;
- if (bRelease || curr_it->second->HasOneRef()) {
- m_Type3FaceMap.erase(curr_it);
- }
- }
-
- for (auto it = m_TransferFuncMap.begin(); it != m_TransferFuncMap.end();) {
- auto curr_it = it++;
- if (bRelease || curr_it->second->HasOneRef())
- m_TransferFuncMap.erase(curr_it);
- }
-}
+CPDF_DocRenderData::~CPDF_DocRenderData() = default;
RetainPtr<CPDF_Type3Cache> CPDF_DocRenderData::GetCachedType3(
CPDF_Type3Font* pFont) {
diff --git a/core/fpdfapi/render/cpdf_docrenderdata.h b/core/fpdfapi/render/cpdf_docrenderdata.h
index 71ca9f4..5f78d58 100644
--- a/core/fpdfapi/render/cpdf_docrenderdata.h
+++ b/core/fpdfapi/render/cpdf_docrenderdata.h
@@ -32,8 +32,6 @@
RetainPtr<CPDF_TransferFunc> GetTransferFunc(const CPDF_Object* pObj);
void MaybePurgeTransferFunc(const CPDF_Object* pObj);
- void Clear(bool bRelease);
-
private:
UnownedPtr<CPDF_Document> m_pPDFDoc;
std::map<CPDF_Font*, RetainPtr<CPDF_Type3Cache>> m_Type3FaceMap;