diff options
| author | tsepez <tsepez@chromium.org> | 2016-11-16 14:45:42 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-11-16 14:45:42 -0800 |
| commit | cc4d0a44f3025821f88f3ed1ee78dfdc416487c7 (patch) | |
| tree | 505c6181f0dc078c351312f0e960cd37a89cf08b /core/fpdfapi/parser/cpdf_indirect_object_holder.cpp | |
| parent | 242b95e2a9df3b57dd9aa28adea58ad51f84a221 (diff) | |
| download | pdfium-cc4d0a44f3025821f88f3ed1ee78dfdc416487c7.tar.xz | |
Move ByteStringPool from document to indirect object holder.
Since the indirect object holder is now in the object creation
business, this will allow it to intern strings in a subsequent
CL.
Review-Url: https://codereview.chromium.org/2509773003
Diffstat (limited to 'core/fpdfapi/parser/cpdf_indirect_object_holder.cpp')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_indirect_object_holder.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/core/fpdfapi/parser/cpdf_indirect_object_holder.cpp b/core/fpdfapi/parser/cpdf_indirect_object_holder.cpp index 12b1e9f089..d1183ebaf4 100644 --- a/core/fpdfapi/parser/cpdf_indirect_object_holder.cpp +++ b/core/fpdfapi/parser/cpdf_indirect_object_holder.cpp @@ -9,9 +9,13 @@ #include "core/fpdfapi/parser/cpdf_object.h" #include "core/fpdfapi/parser/cpdf_parser.h" -CPDF_IndirectObjectHolder::CPDF_IndirectObjectHolder() : m_LastObjNum(0) {} +CPDF_IndirectObjectHolder::CPDF_IndirectObjectHolder() + : m_LastObjNum(0), + m_pByteStringPool(pdfium::MakeUnique<CFX_ByteStringPool>()) {} -CPDF_IndirectObjectHolder::~CPDF_IndirectObjectHolder() {} +CPDF_IndirectObjectHolder::~CPDF_IndirectObjectHolder() { + m_pByteStringPool.DeleteObject(); // Make weak. +} CPDF_Object* CPDF_IndirectObjectHolder::GetIndirectObject( uint32_t objnum) const { |