[webkit-reviews] review requested: [Bug 35937] Support for HTMLProgressElement : [Attachment 50423] Patch v4

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 10 12:33:36 PST 2010


Yael <yael.aharon at nokia.com> has asked	for review:
Bug 35937: Support for HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=35937

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

------- Additional Comments from Yael <yael.aharon at nokia.com>
Address most of the comments from Darin.

1. There should be no impact from the addition to html.css on platforms that
don't support the progress tag. That's because the element would not be
created.

2. I don't think "value" and "max" qualify as [reflect] attributes. If they are
set with invalid values, those values are not accepted and some defaults are
used instead.


More information about the webkit-reviews mailing list