Bug 600556 introduced support for ICO BMP's which contain compression by using the BMP decoder. We should create at least one BMP and one ICO reftests for each of the following compression types for both ICO and BMP: - BI_BITFIELDS - BI_RLE4 - BI_RLE8
- RLE8 should have reftests for 8BPP - RLE4 should have reftests for 4BPP and as for 1BPP as of Bug 683610. - BI_BITFIELDS for 16BPP and 32BPP
Comment on attachment 557208 [details] 1BPP RLE4 reference bitmap Is it OK to use this Microsoft BMP image (anything legal I mean) in our tree? To see it you'll have to use Chrome.
Attachment #557208 - Flags: feedback?(joe)
> To see it you'll have to use Chrome Or FF, just not nightly, nor IE :)
Comment on attachment 557208 [details] 1BPP RLE4 reference bitmap It'd be better to craft our own, and make it small too.
Attachment #557208 - Flags: feedback?(joe) → feedback-
OK no prob. /me needs to re-read about RLE so I can determine what to put in the file with the hex editor.
Created attachment 584951 [details] [diff] [review] Reftests for RLE8 BMPs I'll add in another patch for RLE4 BMPs and RLE1 BMPs once I figure out how to generate them.
Attachment #584951 - Flags: review?(joe)
Attachment #584951 - Flags: review?(joe) → review+
Pushed to mozilla-inbound: http://hg.mozilla.org/integration/mozilla-inbound/rev/e2b6064a04ff
Target Milestone: --- → mozilla12
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Summary: Create reftests for newly supported ICO BMPs which have compression → Create reftests for RLE8 BMPs
You need to log in before you can comment on or make changes to this bug.