Bug 90956 - [Mac] Expose time format related functions
Summary: [Mac] Expose time format related functions
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: yosin
URL:
Keywords:
Depends on: 90237
Blocks: 88970
  Show dependency tree / graph
 
Reported: 2012-07-11 02:07 PDT by yosin
Modified: 2012-07-11 02:44 PDT (History)
1 user (show)

See Also:

Attachments
Patch 1 (1.92 KB, patch)
2012-07-11 02:17 PDT, yosin 		no flags Details | Formatted Diff | Diff
Patch 2 (1.90 KB, patch)
2012-07-11 02:41 PDT, yosin 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description yosin 2012-07-11 02:07:35 PDT 
To use time format related functions implemented in LocaleMac class, we need to wrap them in LocalizedDateMac.cpp.
Comment 1 yosin 2012-07-11 02:17:17 PDT 
Created attachment 151655 [details]
Patch 1
Comment 2 yosin 2012-07-11 02:18:03 PDT 
Comment on attachment 151655 [details]
Patch 1

Could you review this patch?
Thanks in advance.
Comment 3 Kent Tamura 2012-07-11 02:37:28 PDT 
Comment on attachment 151655 [details]
Patch 1

View in context: https://bugs.webkit.org/attachment.cgi?id=151655&action=review

> Source/WebCore/ChangeLog:3
> +        [Chromium-Mac] Expose time format related functions

This can be used for Apple Mac port.  Please remove [Chromium-]
Comment 4 yosin 2012-07-11 02:41:58 PDT 
Created attachment 151659 [details]
Patch 2
Comment 5 yosin 2012-07-11 02:44:17 PDT 
Comment on attachment 151659 [details]
Patch 2

Clearing flags on attachment: 151659

Committed r122320: <http://trac.webkit.org/changeset/122320>
Comment 6 yosin 2012-07-11 02:44:28 PDT 
All reviewed patches have been landed.  Closing bug.