[webkit-reviews] review requested: [Bug 97992] [Forms] Adding DateTimeWeekFieldElement for multiple fields "week" input UI : [Attachment 166430] Patch 3
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Oct 1 02:13:52 PDT 2012
yosin at chromium.org has asked for review:
Bug 97992: [Forms] Adding DateTimeWeekFieldElement for multiple fields "week"
input UI
https://bugs.webkit.org/show_bug.cgi?id=97992
Attachment 166430: Patch 3
https://bugs.webkit.org/attachment.cgi?id=166430&action=review
------- Additional Comments from yosin at chromium.org
Could you review this patch?
Thanks in advance.
= Changes since the last review =
* Rename maximumWeekOfYearNumber to maximumWeekNumber.
* Rename minimumWeekOfYearNumber to minimumWeekNumber.
* Use maximumWeekNumber in DateComponents::maxWeekNumberInYear()
* Use minimumWeekNumber in parseWeek()
More information about the webkit-reviews
mailing list