[webkit-reviews] review granted: [Bug 73911] <li value="0"> behaves like <li> (the same for negative numbers) : [Attachment 118228] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Florin Malita <fmalita at google.com>'s
request for review:
Bug 73911: <li value="0"> behaves like <li> (the same for negative numbers)
https://bugs.webkit.org/show_bug.cgi?id=73911

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

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=118228&action=review


> Source/WebCore/html/HTMLLIElement.h:43
> +    void setValue(const AtomicString&);

This is great refactoring. I don’t think the name of the function is good,
though, because it doesn’t set the value on the element. Maybe
setValueInRenderer or parseValue.