Bug 27823 - Remove XBM support
: Remove XBM support
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: Images
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To: Peter Kasting
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-29 17:43 PDT by Peter Kasting
Modified: 2010-01-12 11:53 PST (History)
0 users

See Also:


Attachments
patch v1 (15.29 KB, patch)
2010-01-11 11:23 PST, Peter Kasting
sam: review-
Details | Formatted Diff | Diff
patch v2 (19.93 KB, patch)
2010-01-11 12:48 PST, Peter Kasting
dglazkov: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Kasting 2009-07-29 17:43:57 PDT
Mozilla has removed XBM support on https://bugzilla.mozilla.org/show_bug.cgi?id=504822 , and noted that IE removed it in IE6.  We might as well rip it out too (at least in the cross-platform image decoders).

Added bonus: this will obsolete bug 26527.
Comment 1 Peter Kasting 2010-01-11 11:23:09 PST
Created attachment 46292 [details]
patch v1
Comment 2 Sam Weinig 2010-01-11 11:41:35 PST
Comment on attachment 46292 [details]
patch v1

This will break some builds. You need to update the build-system files.
Comment 3 Peter Kasting 2010-01-11 11:42:41 PST
Oh yeah.  Good catch.  (I feel silly now.)
Comment 4 Peter Kasting 2010-01-11 12:48:27 PST
Created attachment 46301 [details]
patch v2

Now with more build file fixup.
Comment 5 Dimitri Glazkov (Google) 2010-01-12 11:33:11 PST
Comment on attachment 46301 [details]
patch v2

Looks like apply-patch can't handle rm dirs?

rs=me.

Good luck landing :)
Comment 6 Peter Kasting 2010-01-12 11:53:13 PST
Landed in r53152.