1259723 - innerHTML needs a smarter serializer for <pre>

Status: RESOLVED DUPLICATE of bug 838954

(Core :: DOM: Core & HTML, defect)

Description

[Anne (:annevk)]

See http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3504. Per the HTML Standard innerHTML needs to insert an additonal LF if the contents of <pre> start with LF. Otherwise each time you roundtrip you lose an LF since the HTML parser eats them.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

We explicitly decided to not do this at some point.
http://mxr.mozilla.org/mozilla-central/source/dom/base/nsContentUtils.cpp?rev=604a180b6cc0#8606

Is any browser doing this, or is this actually a spec bug?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

See https://bugzilla.mozilla.org/show_bug.cgi?id=744830#c24
https://bugzilla.mozilla.org/show_bug.cgi?id=744830#c28
https://bugzilla.mozilla.org/show_bug.cgi?id=744830#c34 (this is generic comment about other issues too)
But bad me for not filing spec bugs.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Filed https://github.com/whatwg/html/issues/944

Comment 4

[Simon Pieters [:zcorpan]]

Presto does this FWIW. I don't think it caused any Web compat problems. I have some tests I can submit to web-platform-tests too (note to self: CORE-23623).