97289 – [Platform] There are memory leak in LocaleICU

Bug 97289 - [Platform] There are memory leak in LocaleICU

Summary: [Platform] There are memory leak in LocaleICU

Attachments
Patch 1 (1.90 KB, patch) 2012-09-20 21:10 PDT, yosin	no flags	Details \| Formatted Diff \| Diff
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-09-20 20:52:50 PDT

m_mediumTimeFormat and m_shortTimeFormat of UDateFormat aren't released.

Comment 1 yosin 2012-09-20 21:01:20 PDT

This bug is imported from http://crbug.com/151006
Valgrind found this memory leak.

Comment 2 yosin 2012-09-20 21:10:44 PDT

Created attachment 165042 [details]
Patch 1

Comment 3 yosin 2012-09-20 21:11:14 PDT

Comment on attachment 165042 [details]
Patch 1

Could you review this patch?
Thanks in advance.

Comment 4 Kent Tamura 2012-09-20 21:27:21 PDT

Comment on attachment 165042 [details]
Patch 1

ok

Comment 5 yosin 2012-09-20 21:43:55 PDT

Comment on attachment 165042 [details]
Patch 1

Clearing flags on attachment: 165042

Committed r129191: <http://trac.webkit.org/changeset/129191>

Comment 6 yosin 2012-09-20 21:43:59 PDT

All reviewed patches have been landed.  Closing bug.