[webkit-reviews] review requested: [Bug 93929] [Forms] Make input type "time" to use multiple field time input UI : [Attachment 158244] Patch 3
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Aug 14 00:35:38 PDT 2012
yosin at chromium.org has asked for review:
Bug 93929: [Forms] Make input type "time" to use multiple field time input UI
https://bugs.webkit.org/show_bug.cgi?id=93929
Attachment 158244: Patch 3
https://bugs.webkit.org/attachment.cgi?id=158244&action=review
------- Additional Comments from yosin at chromium.org
Could you review this patch?
Thanks in advance.
= Changes since the last review =
* Add test expectations for Chromium
* Remove InputType::stepRangeChanged()
** No changes for HTMLInputElement.cpp and InputType.{cpp,h}
* Introduce DateTimeEditControlOwner class for avoiding ugly #ifdef on base
class list.
More information about the webkit-reviews
mailing list