1653232 - implement a public GetInnerHTML() for DocumentFragment (copied from ShadowRoot)

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P5)

Comment 1

[Frederik Braun [:freddy]]

Attachment: Bug 1653232 implement GetInnerHTML() for DocFragments

Comment 2

[Frederik Braun [:freddy]]

try https://treeherder.mozilla.org/#/jobs?repo=try&revision=0ac63ee82729725a6e3d180daf2de5e5b2ee1f64

Comment 3

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

Curious, why? :)

Comment 4

[Frederik Braun [:freddy]]

taskcluster did not like it and fails restarting the decision task. https://treeherder.mozilla.org/#/jobs?repo=try&revision=3270246fa0f9b2d84e77554fba0a4c60eb8c700b

Comment 5

[Frederik Braun [:freddy]]

(In reply to Olli Pettay [:smaug] from comment #3)

Curious, why? :)

This makes code in bug 1652481 much easiser. The preliminary API accepts a String/DocumentFragment/Document and returns a sanitized DocumentFragment. The current prototype will just use nsTreeSanitizer as it is (and we'll iterate on sanitizer flags once we have agreement on compat questions).
From looking at assertions in nsTreeSanitizer, it seems that we want a new DocumentFragment and can't just call Sanitize on the DocumentFragment that was passed in (probably due to MutationObservers and stuff?).

Maybe there is a better/easier way to clone a DocumentFragment that I should use instead?

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dce645f310a5
implement GetInnerHTML() for DocFragments r=emilio

Comment 7

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/dce645f310a5