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>

1 files changed, 3 insertions, 3 deletions

diff --git a/core/fxcrt/xml/cfx_xmlinstruction.cpp b/core/fxcrt/xml/cfx_xmlinstruction.cpp
index d0f5cbb68b..b7a87b588a 100644
--- a/core/fxcrt/xml/cfx_xmlinstruction.cpp
+++ b/core/fxcrt/xml/cfx_xmlinstruction.cpp
@@ -49,12 +49,12 @@ void CFX_XMLInstruction::Save(
 
   pXMLStream->WriteString("<?");
   pXMLStream->WriteString(name_.UTF8Encode().AsStringView());
+  pXMLStream->WriteString(" ");
 
   for (const WideString& target : m_TargetData) {
-    pXMLStream->WriteString("\"");
     pXMLStream->WriteString(target.UTF8Encode().AsStringView());
-    pXMLStream->WriteString("\"");
+    pXMLStream->WriteString(" ");
   }
 
-  pXMLStream->WriteString("?>");
+  pXMLStream->WriteString("?>\n");
 }