Open Bug 450140 Opened 16 years ago Updated 2 years ago

Canvas drawWindow with context.scale adds borders around some images

Categories

(Core :: Graphics: Canvas2D, defect, P3)

x86
Linux
defect

Tracking

()

People

(Reporter: dholbert, Unassigned)

References

Details

Attachments

(2 files)

Steps to Reproduce: 1. Open these URLs in tabs: http://www.google.com/ http://www.mozilla.org/projects/minefield/ [anything else] 2. Use Ctrl-Tab to bring up the new Tab-Switcher 3. Examine the previews for google and the minefield startup page. ACTUAL RESULTS: Some images in the previews have borders added around them. These aren't just JPEG artifacts, AFAICT. EXPECTED RESULTS: No new borders should be added in the previews. See screenshots for examples. (I've added red arrows to point to the borders.) I'm testing using a fresh mozilla-central debug build from today.
This screenshot shows that the added borders aren't always the same. In this case, a top border was added around the Q-shaped QMO logo on the left. (This top border was not present in screenshot 1.)
Component: Tabbed Browser → Layout: Canvas
Product: Firefox → Core
QA Contact: tabbed.browser → layout.canvas
My guess is that what's happening is that the image is RGB24, with a solid white background.. but we're hitting the EXTEND_PAD failure here, so we're sampling outside of the image bounds and getting solid black (because we're doing RGB24) compositing. This is kinda broken in a lot of ways, but I really don't know what the right fix should be.
Summary: Ctrl-Tab Preview adds borders around some images → Canvas drawWindow with context.scale adds borders around some images
Blocks: 436304
Flags: blocking1.9.1?
Yep. This might be fixed by roc's work in bug 458487.
Flags: wanted1.9.1+
Flags: blocking1.9.1?
Flags: blocking1.9.1-
Priority: -- → P3
(In reply to comment #3) > Yep. This might be fixed by roc's work in bug 458487. I'm still seeing the borders around the QMO logos (attachment 333272 [details]).
Currently we're not pixel-snapping when there's a scale in the context, so these fringes are sure to happen.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: