XMLSerializer escapes < > & correctly inside <script> and <style> tags.

Source/core/editing/MarkupAccumulator.cpp

Index: Source/core/editing/MarkupAccumulator.cpp
diff --git a/Source/core/editing/MarkupAccumulator.cpp b/Source/core/editing/MarkupAccumulator.cpp
index 1ccbc83f7694b02468b203c8a766b579e1184058..51a8d0a620a8faf8b296af871b543ce173053161 100644
--- a/Source/core/editing/MarkupAccumulator.cpp
+++ b/Source/core/editing/MarkupAccumulator.cpp
@@ -308,7 +308,7 @@ EntityMask MarkupAccumulator::entityMaskForText(Text* text) const
         parentName = &(text->parentElement())->tagQName();
 
     if (parentName && (*parentName == scriptTag || *parentName == styleTag || *parentName == xmpTag))
-        return EntityMaskInCDATA;
+        return text->document().isHTMLDocument() ? EntityMaskInCDATA : EntityMaskInPCDATA;
 
     return text->document().isHTMLDocument() ? EntityMaskInHTMLPCDATA : EntityMaskInPCDATA;

[jochen (gone - plz use gerrit), 2013/11/06 16:17:49]

can you simplilfy this method a bit?

if (!text->document().isHTMLDocument())
  return EntitytMaskInPCDATA;

// rest of method here with ?:

[pwnall-personal, 2013/11/06 17:34:30]

Done.

Did you mean I should use a ? : in a return clause?

I thought the condition was rather long, so I removed the ? : from the two if
branches and turned them into simple returns. If you think I should use ? :, I
can make that change right away.

 }