Closed Bug 683205 Opened 8 years ago Closed 8 years ago

Create reftests for RLE8 BMPs

Categories

(Core :: ImageLib, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: bbondy, Assigned: bbondy)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

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.
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
https://hg.mozilla.org/mozilla-central/rev/e2b6064a04ff
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Summary: Create reftests for newly supported ICO BMPs which have compression → Create reftests for RLE8 BMPs
Blocks: 715128
You need to log in before you can comment on or make changes to this bug.