Implemented XMLSerializer in PrivateScript.

Source/core/xml/XMLSerializer.cpp

Index: Source/core/xml/XMLSerializer.cpp
diff --git a/Source/core/xml/XMLSerializer.cpp b/Source/core/xml/XMLSerializer.cpp
index b39aa329b9da424bc0c871a5dec4f04912158a10..b00028fb84ec56b1f93f8506fd30b1dcbc5924ee 100644
--- a/Source/core/xml/XMLSerializer.cpp
+++ b/Source/core/xml/XMLSerializer.cpp
@@ -22,9 +22,10 @@
 #include "core/xml/XMLSerializer.h"
 
 #include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/V8XMLSerializer.h"
+#include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
-#include "core/editing/MarkupAccumulator.h"
-#include "wtf/text/WTFString.h"
+#include "wtf/text/StringBuilder.h"
 
 namespace blink {
 
@@ -35,8 +36,25 @@ String XMLSerializer::serializeToString(Node* node, ExceptionState& exceptionSta
         return String();
     }
 
-    MarkupAccumulator accumulator(0, DoNotResolveURLs, nullptr, ForcedXML);
-    return accumulator.serializeNodes(*node, IncludeNode);
+    String result;
+    String xmlDeclaration;
+    if (node->document().hasXMLDeclaration()) {
+        StringBuilder builder;
+        builder.appendLiteral("<?xml version=\"");

[Jens Widell, 2014/09/03 10:29:33]

IMO it seems a bit odd to serialize an XML declaration here, rather than just
passing the data in to the private script function and let it do all the
serializing.

Exposing the information (publicly) on the Document interface would have made
sense too (back when the DOM APIs were first designed, perhaps,) but at this
point adding XML stuff to it is probably not the direction we want it to move.

[tasak, 2014/09/03 11:40:46]

Firstly I tried to serialize an XML declaration in XMLSerializer.js.
However, I found that there is no way to find whether a given document has an
XML declaration or not.

I tried "if (document.xmlVersion) { ... }", but document.xmlVersion is always
"1.0". Since the API has been already shipped, I'm not sure whether we can
change the behavior or not.

And looking at Document.idl, it says "xmlVersion, xmlStandalone and xmlEncoding
are removed from DOM4". I think, we cannot create an XML declaration by using
only web-exposed API in the future.

+        builder.append(node->document().xmlVersion());
+        builder.append('\"');
+        if (!node->document().xmlEncoding().isEmpty()) {
+            builder.appendLiteral(" encoding=\"");
+            builder.append(node->document().xmlEncoding());
+            builder.append('\"');
+        }
+        if (node->document().xmlStandaloneStatus() != Document::StandaloneUnspecified)
+            builder.append(String::format(" standalone=\"%s\"", node->document().xmlStandalone() ? "yes" : "no"));
+        builder.append("?>");
+        xmlDeclaration = builder.toString();
+    }
+    V8XMLSerializer::PrivateScript::serializeToStringInternalMethod(m_document->frame(), this, node, xmlDeclaration, &result);
+    return result;
 }
 
 } // namespace blink