Add code for input[type=date] with the multiple fields UI
Created attachment 167040 [details] Patch
Comment on attachment 167040 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167040&action=review Basically looks good. Hope we have tests soon. > Source/WebCore/html/BaseMultipleFieldsDateAndTimeInputType.h:93 > + bool m_pickerIndicatorIsAlwaysVisible; Do we need this member variable? It looks this can be computed on the fly.
Comment on attachment 167040 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167040&action=review >> Source/WebCore/html/BaseMultipleFieldsDateAndTimeInputType.h:93 >> + bool m_pickerIndicatorIsAlwaysVisible; > > Do we need this member variable? It looks this can be computed on the fly. It is referred in BaseMultipleFieldsDateAndTimeInputType::updatePickerIndicatorVisibility.
(In reply to comment #3) > (From update of attachment 167040 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=167040&action=review > > >> Source/WebCore/html/BaseMultipleFieldsDateAndTimeInputType.h:93 > >> + bool m_pickerIndicatorIsAlwaysVisible; > > > > Do we need this member variable? It looks this can be computed on the fly. > > It is referred in BaseMultipleFieldsDateAndTimeInputType::updatePickerIndicatorVisibility. Oh, did you mean calling RenderTheme::supportsCalendarPicker in updatePickerIndicatorVisibility? You're right. I'll fix later.
Comment on attachment 167040 [details] Patch Clearing flags on attachment: 167040 Committed r130369: <http://trac.webkit.org/changeset/130369>
All reviewed patches have been landed. Closing bug.