Bug 92822 – [Chromium] Enable ENABLE_INPUT_TYPE_TIME_MULTIPLE_FIELDS

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

This bug is one of preparation of landing multiple fields time UI for input type "time".

Enabling ENABLE_INPUT_TYPE_TIME_MULTIPLE_FIELDS makes sure DateTimeFormat related functions, localizedTimeFormatText(), localizedShortTimeFormatText() and 
timeAMPMLabels(), built without failure.

Enabling ENABLE_INPUT_TYPE_MULTIPLE_FIELDS also adds localizedDecimalSeparator().


Following files have ENABLE_INPUT_TYPE_MULTIPLE_FIELDS

Source/WebCore/platform/text/DateTimeFormat.h
  class DateTimeFormat
Source/WebCore/platform/text/LocalizedNumber.h
  localizedDecimalSeparator()
Source/WebCore/platform/text/LocalizedDate.h
  localizedTimeFormatText(), localizedShortTimeFormatText() and 
timeAMPMLabels()
Source/WebCore/platform/text/LocaleWin.cpp
  include DateTimeFormat.h
Source/WebCore/platform/text/LocalizedDateWin.cpp
  include LocalizedDate.h
Source/WebCore/platform/text/DateTimeFormat.cpp
  include DateTimeFormat.h
Source/WebCore/platform/text/LocalizedNumberICU.cpp
  include LocalizedNumber.h
  include LocaleICU.h
Source/WebCore/platform/text/LocalizedNumberNone.cpp
  include LocalizedNumber.h
Source/WebCore/platform/text/LocalizedDateICU.cpp
  include LocalizedDate.h
  include LocaleICU.h
Source/WebCore/platform/text/LocaleICU.h
Source/WebCore/platform/text/LocaleWin.h
Source/WebCore/platform/text/mac/LocalizedDateMac.cpp
  include LocalizedDate.h
Source/WebCore/platform/text/mac/LocaleMac.h
Source/WebCore/platform/text/mac/LocaleMac.mm
  include LocalizedDate.h
Source/WebCore/platform/text/mac/LocalizedNumberMac.mm
  include LocalizedNumber.h
Source/WebCore/platform/text/LocaleICU.cpp
  include LocaleICU.h
Source/WebKit/chromium/tests/DateTimeFormatTest.cpp
  include DateTimeFormat.h
Source/WebKit/chromium/tests/LocalizedDateICUTest.cpp
  include LocaleICU.h
Source/WebKit/chromium/tests/LocaleMacTest.cpp
  include LocaleMac.h
Source/WebKit/chromium/tests/LocaleWinTest.cpp
  include LocaleWin.h
Source/WebKit/chromium/tests/LocalizedNumberICUTest.cpp
  include LocaleICU.h

Created attachment 155704 [details]
Patch 1

Comment on attachment 155704 [details]
Patch 1

Could you review this patch?
Thanks in advance.

[Kent Tamura]

Comment on attachment 155704 [details]
Patch 1

ok

Comment on attachment 155704 [details]
Patch 1

Clearing flags on attachment: 155704

Committed r124290: <http://trac.webkit.org/changeset/124290>

All reviewed patches have been landed.  Closing bug.