F4 inside <input type=time> should not open calendar picker
Created attachment 171385 [details] Patch
Comment on attachment 171385 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=171385&action=review > Source/WebCore/ChangeLog:12 > + Test: platform/chromium/fast/forms/time/time-open-picker-key-bindings.html > + > + * html/BaseMultipleFieldsDateAndTimeInputType.cpp: Because this behavior is defined in BaseMultipleFIeldsDateAndTimeInputType, the test should be in fast/forms/time-multiple-fields/, not platform/chromium/fast/forms/time/.
Comment on attachment 171385 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=171385&action=review > LayoutTests/platform/chromium/fast/forms/time/time-open-picker-key-bindings.html:19 > +var input = document.getElementById('time'); > +sendKey(input, "F4", false, false); > +shouldBeNull('document.getElementById("mock-page-popup")'); > +sendKey(input, "Down", false, true); nit: single/double quote usage is inconsistent.
Created attachment 171406 [details] Patch
Comment on attachment 171406 [details] Patch Attachment 171406 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14551049 New failing tests: fast/forms/time-multiple-fields/time-open-picker-key-bindings.html
Created attachment 171419 [details] Patch
Comment on attachment 171419 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=171419&action=review > Source/WebCore/ChangeLog:10 > + Test: fast/forms/time-multiple-fields/time-open-picker-key-bindings.html Please rename it to a name starting with time-multiple-fields-.
Created attachment 171567 [details] Patch
Comment on attachment 171567 [details] Patch Clearing flags on attachment: 171567 Committed r132985: <http://trac.webkit.org/changeset/132985>
All reviewed patches have been landed. Closing bug.