Block "]]>" in createCDataSection
Created attachment 211134 [details] Fixes the bug
Created attachment 211135 [details] Added the forgotten test
Comment on attachment 211135 [details] Added the forgotten test View in context: https://bugs.webkit.org/attachment.cgi?id=211135&action=review > Source/WebCore/ChangeLog:11 > + It's not possible to serialize a CData section with ]]> in it so disallow that already in xmldoc.createCDataSection. > + New behavior matches that of Trident and Gecko. What about appendData, insertData, replaceData and replaceWholeText? This change as made in Blink is pretty much useless alone.
Comment on attachment 211135 [details] Added the forgotten test Attachment 211135 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1736487 New failing tests: dom/xhtml/level3/core/documentnormalizedocument07.xhtml dom/xhtml/level3/core/documentnormalizedocument08.xhtml dom/xhtml/level3/core/handleerror01.xhtml dom/xhtml/level3/core/splitcdatasections01.xhtml
Created attachment 211139 [details] Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-10 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.4
> dom/xhtml/level3/core/splitcdatasections01.xhtml So this documents correct behavior.
Comment on attachment 211135 [details] Added the forgotten test Attachment 211135 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1726773 New failing tests: dom/xhtml/level3/core/documentnormalizedocument07.xhtml dom/xhtml/level3/core/documentnormalizedocument08.xhtml dom/xhtml/level3/core/handleerror01.xhtml dom/xhtml/level3/core/splitcdatasections01.xhtml
Created attachment 211144 [details] Archive of layout-test-results from webkit-ews-05 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-05 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Comment on attachment 211135 [details] Added the forgotten test You're right. We should probably update other functions.
I'm not quite sure what DOM4 authors are trying to achieve here. Are all ways to create non-serializable documents being banned?
createCDataSection() and replaceWholeText() have been dropped from the latest DOM living standard. appendData, insertData, replaceData are still there and do not say anything about blocking "]]>". WebKit still supports createCDataSection() and replaceWholeText() though.
Okay.