ENABLE_DATE_AND_TIME_INPUT_TYPES is just an alias of ENABLE(INPUT_TYPE_DATE) || ENABLE(INPUT_TYPE_DATETIME) || ENABLE(INPUT_TYPE_DATETIMELOCAL) || ENABLE(INPUT_TYPE_MONTH) || ENABLE(INPUT_TYPE_TIME) || ENABLE(INPUT_TYPE_WEEK). ENABLE_CALENDAR_PICKER should mean "using WebCore/Resources/pagepopups/*.js"
Created attachment 171971 [details] Cook on EWS
Created attachment 171981 [details] Fix android build
Comment on attachment 171981 [details] Fix android build Attachment 171981 [details] did not pass cr-android-ews (chromium-android): Output: http://queues.webkit.org/results/14693707
Created attachment 171986 [details] Patch
Comment on attachment 171986 [details] Patch Attachment 171986 [details] did not pass cr-android-ews (chromium-android): Output: http://queues.webkit.org/results/14684724
Created attachment 171991 [details] Patch 2 Android fix
Comment on attachment 171991 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=171991&action=review rs=me > Source/WebCore/ChangeLog:10 > + ENABLE_INPUT_MULTIPLE_FIELDS_UI is disabled. So they should be warpped Typo: wrapped > Source/WebCore/ChangeLog:32 > + 2) WebCore/Resources/pagepopus/ is used for the calendar UI implementation. Typo: pagepopups > Source/WebCore/ChangeLog:39 > + date/tiem InputType classes. Typo: date/time > Source/WebCore/ChangeLog:42 > + * page/ChromeClient.h: Change the flag for openDateTimChooser, and add a comment. Typo: openDateTimeChooser > Source/WebCore/ChangeLog:51 > + ENABEL(INPUT_MULTIPLE_FIELDS_UI). Typo: ENABLE > Source/WebCore/page/ChromeClient.h:233 > + // returns true, if ENABEL(INPUT_MULTIPLE_FIELDS_UI) Typo: ENABLE > Source/WebCore/page/ChromeClient.h:235 > + // ENABEL(INPUT_MULTIPLE_FIELDS_UI) Typo: ENABLE > Source/WebCore/platform/text/PlatformLocale.cpp:351 > + Nit: A redundant empty line.
oops, thank you for pointing out a lot of typos :-(
Committed r133284: <http://trac.webkit.org/changeset/133284>