[webkit-reviews] review requested: [Bug 97299] [Forms] Multiple fields month input UI : [Attachment 166174] Patch 5

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 28 01:58:05 PDT 2012


yosin at chromium.org has asked  for review:
Bug 97299: [Forms] Multiple fields month input UI
https://bugs.webkit.org/show_bug.cgi?id=97299

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

------- Additional Comments from yosin at chromium.org
Could you review this patch?
Thanks in advance.

= Changes since the last review =
* Not to use reference in DateTimeEditBuilder member variables.
* Use "%04u-%02u" for MonthInputType::formatDateTimeFieldsState
* Added comment for month specifier in DateTimeEditBuilder::visitField


More information about the webkit-reviews mailing list