diff options
author | Wei Li <weili@chromium.org> | 2016-02-22 17:27:07 -0800 |
---|---|---|
committer | Wei Li <weili@chromium.org> | 2016-02-22 17:27:07 -0800 |
commit | 4f4de9a14eed86bb49340079b9bd43ed7a3caf70 (patch) | |
tree | 2a97fb41237f4058d6fc69c9695405fb4aefabc5 /core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp | |
parent | d219a247dfaa697f9761ee558b58c23680509058 (diff) | |
download | pdfium-4f4de9a14eed86bb49340079b9bd43ed7a3caf70.tar.xz |
refactor CPDF_PageObject and subclasses
Use virtual functions to replace type based logic; remove unused
functions; disable default copy constructor/assign operator.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1720003003 .
Diffstat (limited to 'core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp')
-rw-r--r-- | core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp index d49186374f..efe0f25a9f 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page_parser.cpp @@ -1389,11 +1389,8 @@ void CPDF_StreamContentParser::AddTextObject(CFX_ByteString* pStrs, m_pCurStates->m_TextHorzScale, m_Level); m_pCurStates->m_TextX += x_advance; m_pCurStates->m_TextY += y_advance; - if (textmode > 3) { - CPDF_TextObject* pCopy = new CPDF_TextObject; - pCopy->Copy(pText.get()); - m_ClipTextList.Add(pCopy); - } + if (textmode > 3) + m_ClipTextList.Add(pText->Clone()); m_pObjectHolder->GetPageObjectList()->push_back(std::move(pText)); } if (pKerning && pKerning[nsegs - 1] != 0) { |