Bug 98113

Summary: Remove the Safari 2 -> Safari 3 icon database import code.
Product: WebKit Reporter: Brady Eidson <beidson>
Component: WebKit Misc.Assignee: Brady Eidson <beidson>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, japhet, mifenton, rakuco, rwlbuis, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: All   
Attachments:
Description Flags
Patch v1 - Nothing but red (except for ChangeLogs)
buildbot: commit-queue-
Patch v2 - Now with more passing release builds! mjs: review+

Brady Eidson
Reported 2012-10-01 22:25:17 PDT
Remove the Safari 2 -> Safari 3 icon database import code. The Safari 3-style icon database has been in shipping products since Fall 2007. Supporting this import seems unimportant.
Attachments
Patch v1 - Nothing but red (except for ChangeLogs) (30.76 KB, patch)
2012-10-01 23:01 PDT, Brady Eidson
buildbot: commit-queue-
Patch v2 - Now with more passing release builds! (32.24 KB, patch)
2012-10-01 23:17 PDT, Brady Eidson
mjs: review+
Brady Eidson
Comment 1 2012-10-01 23:01:23 PDT
Created attachment 166610 [details] Patch v1 - Nothing but red (except for ChangeLogs)
Build Bot
Comment 2 2012-10-01 23:04:13 PDT
Comment on attachment 166610 [details] Patch v1 - Nothing but red (except for ChangeLogs) Attachment 166610 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14131119
Brady Eidson
Comment 3 2012-10-01 23:17:08 PDT
Created attachment 166613 [details] Patch v2 - Now with more passing release builds!
Maciej Stachowiak
Comment 4 2012-10-01 23:29:46 PDT
Comment on attachment 166613 [details] Patch v2 - Now with more passing release builds! r=me but please wait til EWS is happy before committing.
Brady Eidson
Comment 5 2012-10-02 00:05:34 PDT
EWS built fine, and test failures seem to be unrelated noise. http://trac.webkit.org/changeset/130128
Note You need to log in before you can comment on or make changes to this bug.