For displaying millisecond in Time input fields UI needs to know localized decimal separator, e.g. "." in English, "," in French. Because date time format obtained from some platforms don't have millisecond, e.g. ICU.
Created attachment 149683 [details] Patch 1
Comment on attachment 149683 [details] Patch 1 Could you review this patch? Thanks in advance. * Local build on CR-Linux * Local build on Mac
Comment on attachment 149683 [details] Patch 1 Attachment 149683 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13097680
Created attachment 149689 [details] Patch 2
Comment on attachment 149689 [details] Patch 2 Could you review this patch? Thanks in advance. == Change since last request == * Add #if ENABLE(INPUT_TIME_FIELDS) to LocalizedNumberICU.cpp
Comment on attachment 149689 [details] Patch 2 Attachment 149689 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13087780
Created attachment 149692 [details] Patch 3
Comment on attachment 149692 [details] Patch 3 Could you review this patch? Thanks in advance. == Changes since last request == * Fix LocalizedNumberICUTest.cpp
Comment on attachment 149692 [details] Patch 3 ok except the ENABLE flag name.
Created attachment 149863 [details] Patch 4
Comment on attachment 149692 [details] Patch 3 Could you review this patch? Thanks in advance. == Change since last review == * Rename ENABLE flag to ENABLE_INPUT_TYPE_TIME_MULTIPLE_FIELDS
Comment on attachment 149863 [details] Patch 4 ok
Comment on attachment 149863 [details] Patch 4 Clearing flags on attachment: 149863 Committed r121404: <http://trac.webkit.org/changeset/121404>
All reviewed patches have been landed. Closing bug.
Comment on attachment 149692 [details] Patch 3 Cleared review? from obsolete attachment 149692 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).