Refactoring XSLT

Source/core/xml/parser/XMLDocumentParser.cpp

Index: Source/core/xml/parser/XMLDocumentParser.cpp
diff --git a/Source/core/xml/parser/XMLDocumentParser.cpp b/Source/core/xml/parser/XMLDocumentParser.cpp
index e06c6998513dd3b5ecbc799b41adb80f3fd2f28f..86e30b2da341bd6f131039ac006695a2bb0f989a 100644
--- a/Source/core/xml/parser/XMLDocumentParser.cpp
+++ b/Source/core/xml/parser/XMLDocumentParser.cpp
@@ -53,6 +53,7 @@
 #include "core/loader/FrameLoader.h"
 #include "core/loader/ImageLoader.h"
 #include "core/svg/graphics/SVGImage.h"
+#include "core/xml/DocumentXSLT.h"
 #include "core/xml/parser/SharedBufferReader.h"
 #include "core/xml/parser/XMLDocumentParserScope.h"
 #include "core/xml/parser/XMLParserInput.h"
@@ -102,7 +103,7 @@ static inline AtomicString toAtomicString(const xmlChar* string)
 
 static inline bool hasNoStyleInformation(Document* document)
 {
-    if (document->sawElementsInKnownNamespaces() || document->transformSourceDocument())
+    if (document->sawElementsInKnownNamespaces() || DocumentXSLT::hasTransformSourceDocument(*document))

[haraken, 2014/11/17 14:18:32]

We want to write this as:

  DocumentXSLT::from(*document).hasTransformSourceDocument()

[tasak, 2014/11/19 08:23:17]

We don't need to create any DocumentXSLT instance here.

         return false;
 
     if (!document->frame() || !document->frame()->page())
@@ -1165,7 +1166,7 @@ void XMLDocumentParser::processingInstruction(const String& target, const String
         return;
 
     m_sawXSLTransform = !m_sawFirstElement && pi->isXSL();
-    if (m_sawXSLTransform && !document()->transformSourceDocument()) {
+    if (m_sawXSLTransform && !DocumentXSLT::hasTransformSourceDocument(*document())) {

[haraken, 2014/11/17 14:18:32]

Ditto.

[tasak, 2014/11/19 08:23:17]

Ditto.

         // This behavior is very tricky. We call stopParsing() here because we
         // want to stop processing the document until we're ready to apply the
         // transform, but we actually still want to be fed decoded string pieces