Canvas2D renders black with 16M colors on OS/2

VERIFIED FIXED

Status

Core Graveyard
GFX: OS/2
VERIFIED FIXED
11 years ago
9 years ago

People

(Reporter: Peter Weilbacher, Assigned: Peter Weilbacher)

Tracking

({verified1.8.1.1})

Trunk
x86
OS/2
verified1.8.1.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
As noted in the newsgroup the canvas that is created by the Firefox Showcase extension or the SeaMonkey tab preview tooltips displays black when the system is set to 16M colors with the SNAP driver. With the 64K colors setting it displays just fine.
(Assignee)

Comment 1

11 years ago
Created attachment 243934 [details] [diff] [review]
add pixel format for BGR_A8

The problem is simply because nsOffscreenSurface::GetPixelFormat() does not implement 32bit depths, just 24bit depth. I don't know why SNAP implements 16M colors as 32bit instead of 24bit but this patch seems to fix the problem here. It implements 32bit as BGR_A8 which is what OS/2+SNAP seems to be using (at least that is the format that always worked in the SVG code). It uses basically the same bit masks as GetPixelFormat() on Windows.

Andy, would be great if you could confirm that this patch fixes the problem for you, too.
Attachment #243934 - Flags: superreview?(mozilla)
Attachment #243934 - Flags: review?(abwillis1)
Comment on attachment 243934 [details] [diff] [review]
add pixel format for BGR_A8

Works like a charm.
Attachment #243934 - Flags: review?(abwillis1) → review+

Comment 3

11 years ago
Comment on attachment 243934 [details] [diff] [review]
add pixel format for BGR_A8

sr=mkaply (OS2 only)
Attachment #243934 - Flags: superreview?(mozilla) → superreview+
(Assignee)

Comment 4

11 years ago
Fix checked into trunk (with an additional "break;" that was missing from the patch).
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

11 years ago
Comment on attachment 243934 [details] [diff] [review]
add pixel format for BGR_A8

Would be great to get this OS/2 only, trunk tested fix into 1.8 branch.
Attachment #243934 - Flags: approval1.8.1.1?
Comment on attachment 243934 [details] [diff] [review]
add pixel format for BGR_A8

approved for 1.8 branch, a=dveditz for drivers
Attachment #243934 - Flags: approval1.8.1.1? → approval1.8.1.1+
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1.1
(Assignee)

Comment 7

11 years ago
Verifying some older bugs.
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1.1 → verified1.8.1.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.