Bug 45698

Summary: [chromium] Remove unused ICU headers
Product: WebKit Reporter: Kwang Yul Seo <skyul>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch
none
Patch none

Kwang Yul Seo
Reported 2010-09-13 12:51:00 PDT
ICU headers included in FontLinux.cpp and FontCacheLinux.cpp are not used.Remove them.
Attachments
Patch (1.59 KB, patch)
2010-09-13 12:52 PDT, Kwang Yul Seo 		no flags
DetailsFormatted DiffDiff
Patch (1022 bytes, patch)
2010-09-13 18:06 PDT, Kwang Yul Seo 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kwang Yul Seo
Comment 1 2010-09-13 12:52:43 PDT
Created attachment 67458 [details] Patch
WebKit Review Bot
Comment 2 2010-09-13 16:05:32 PDT
Attachment 67458 [details] did not build on chromium: Build output: http://queues.webkit.org/results/3933436
Kwang Yul Seo
Comment 3 2010-09-13 18:06:42 PDT
Created attachment 67496 [details] Patch <unicode/utf16.h> is not used.
Kwang Yul Seo
Comment 4 2010-09-13 18:07:07 PDT
Reopen the bug.
Andreas Kling
Comment 5 2010-09-14 06:44:55 PDT
Comment on attachment 67496 [details] Patch r=me
WebKit Commit Bot
Comment 6 2010-09-14 12:44:38 PDT
Comment on attachment 67496 [details] Patch Clearing flags on attachment: 67496 Committed r67485: <http://trac.webkit.org/changeset/67485>
WebKit Commit Bot
Comment 7 2010-09-14 12:44:43 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.