[webkit-reviews] review requested: [Bug 95660] [Forms] Empty visible value of AM/PM field of multiple fields time input UI should display variable number of "-" based on maximum number of labels : [Attachment 161863] Patch 4

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Sep 2 22:31:49 PDT 2012


yosin at chromium.org has asked  for review:
Bug 95660: [Forms] Empty visible value of AM/PM field of multiple fields time
input UI should display variable number of "-" based on maximum number of
labels
https://bugs.webkit.org/show_bug.cgi?id=95660

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

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

= Changes since the last patch =
* Use StringBuilder::toAtomicString() instead of AtomicString constructor with
StringBuilder::toString()
* Fix indendation


More information about the webkit-reviews mailing list