Bug 247096 – XMLSerializer.serializeToString() should support Attr node as parameter

• Status: RESOLVED
• Resolution: FIXED
• Component: XML

[Ahmad Saleem]

Hi Team,

I just came across another bug / testcase, which is failing in Safari 16 / Safari TP 156:

Test Case - https://jsfiddle.net/20no3p91/show

Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/271184ec08ca181a7d93eed532f3ce9efb9c0bcc

WPT Cover - Nope

Gecko - Failing similar to Webkit as of right now.

_________

Webkit Source - https://github.com/WebKit/WebKit/blob/f6f2c95eb900fce4967fbfb6586416363eb4fe7e/Source/WebCore/editing/MarkupAccumulator.cpp#L602

To do:

result.appendAttribute(downcast<Attr>(node).value());

or

appendAttribute(downcast<Attr>(node).value());

or something else totally.

NOTE - Add "Attr.h" header as well.

_____

Just wanted to create bug, so if I end up doing PR then I have something or else someone else take it up.

Thanks!

[EWS]

Committed 256126@main (38ce5b803b34): <https://commits.webkit.org/256126@main>

Reviewed commits have been landed. Closing PR #5835 and removing active labels.

[Radar WebKit Bug Importer]

<rdar://problem/101705913>

[Anne van Kesteren]

*** Bug 20250 has been marked as a duplicate of this bug. ***