[webkit-reviews] review canceled: [Bug 22678] HTML serialization doesn't handle boolean properties properly : [Attachment 59072] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has canceled Darin Adler <darin at apple.com>'s
request for review:
Bug 22678: HTML serialization doesn't handle boolean properties properly
https://bugs.webkit.org/show_bug.cgi?id=22678

Attachment 59072: Patch
https://bugs.webkit.org/attachment.cgi?id=59072&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
It’s true, there are boolean IDL properties in the DOM. However, content
attributes don’t have types, to the best of my knowledge. They are all strings.
So it would not makes sense to serialize content attributes that control
boolean values differently.

Reading the HTML fragment serialization algorithm in current HTML5 draft, I see
that I am wrong about this change, though, so I won’t land it.