diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-05-01 17:01:54 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-01 17:01:54 +0000 |
commit | 048afc6aba4848d5296affb4335500f960262580 (patch) | |
tree | 1cef231f4c264542412b8ad67db4f622f8eea81d /core/fxcrt/xml/cfx_xmlparser.cpp | |
parent | b5902c78075141d9d569a463486d20ccabd78a2d (diff) | |
download | pdfium-048afc6aba4848d5296affb4335500f960262580.tar.xz |
Fix CFX_XML and add unit tests
This CL fixes several issues in the CFX_XML class and
adds unit tests.
Change-Id: I05270690de8f3c45dceb866e17ef899ae6d23389
Reviewed-on: https://pdfium-review.googlesource.com/31753
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'core/fxcrt/xml/cfx_xmlparser.cpp')
-rw-r--r-- | core/fxcrt/xml/cfx_xmlparser.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fxcrt/xml/cfx_xmlparser.cpp b/core/fxcrt/xml/cfx_xmlparser.cpp index eb79637095..cbfb949705 100644 --- a/core/fxcrt/xml/cfx_xmlparser.cpp +++ b/core/fxcrt/xml/cfx_xmlparser.cpp @@ -175,8 +175,8 @@ bool CFX_XMLParser::Parse() { auto* instruction = static_cast<CFX_XMLInstruction*>(m_pChild); if (!target_data.IsEmpty()) instruction->AppendData(target_data); - - instruction->AppendData(GetTextData()); + if (!GetTextData().IsEmpty()) + instruction->AppendData(GetTextData()); } break; } |