Summary: | LocalizedDateICU should ignore timezones | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kent Tamura <tkent> | ||||
Component: | Forms | Assignee: | Kent Tamura <tkent> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Major | CC: | haraken, morrita | ||||
Priority: | P1 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 83852 | ||||||
Attachments: |
|
Description
Kent Tamura
2012-04-12 22:16:19 PDT
Created attachment 137038 [details]
Patch
This is a root cause of fast/forms/date-input-visible-string.html failure. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=fast%2Fforms%2Fdate%2Fdate-input-visible-string Comment on attachment 137038 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=137038&action=review Looks good. Picked a few nits. > Source/WebCore/platform/text/LocalizedDateICU.cpp:48 > + const UChar gmtTimezone[3] = {'G', 'M', 'T'}; Don't we need L prefix for each char? > Source/WebCore/platform/text/LocalizedDateICU.cpp:50 > + return udat_open(UDAT_NONE, UDAT_SHORT, defaultLanguage().utf8().data(), gmtTimezone, 3, 0, -1, &status); We could use WTF_ARRAY_LENGTH? Thanks. (In reply to comment #3) > > Source/WebCore/platform/text/LocalizedDateICU.cpp:48 > > + const UChar gmtTimezone[3] = {'G', 'M', 'T'}; > > Don't we need L prefix for each char? No. UChar is not always wchar_t. > > Source/WebCore/platform/text/LocalizedDateICU.cpp:50 > > + return udat_open(UDAT_NONE, UDAT_SHORT, defaultLanguage().utf8().data(), gmtTimezone, 3, 0, -1, &status); > > We could use WTF_ARRAY_LENGTH? Yes. will do. Committed r114085: <http://trac.webkit.org/changeset/114085> |