616692 - SVG pattern tiles should be painted without seams (Mac)

Status: RESOLVED WORKSFORME

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

On Mac 10.6 the rendering of attachment 493664 [details] has regressed between FF3.6 and FF4b7. Attached here is a semi-reduced testcase. You should see just a black square (no stripes).

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: testcase

Comment 2

[Robert Longson [:longsonr]]

Any change with shapeRendering="crispEdges". You need that to make your testcase valid and if you have it, it works on Windows 3.6 and trunk (without it doesn't work on either). 

Bug 615042 is similar.

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: Mac screenshot

Sorry, I didn't realize there are seam issues on Windows. This bug is actually quite different - see attached screenshot.

Comment 4

[Robert Longson [:longsonr]]

yup, totally different on Windows. crispEdges no effect?

Comment 5

[Jonathan Watt [:jwatt]]

None at all.

Comment 6

[Robert Longson [:longsonr]]

If you turn off hardware acceleration does it go away?

Comment 7

[Jonathan Watt [:jwatt]]

No.

One thing I'd note is that in FF3.6 on Mac the testcase has seam issues like Windows, but then in FF4b7 the seam issues are gone and instead there are the wide gaps.

Comment 8

[Boris Zbarsky [:bzbarsky]]

So on Mac I see two behavior changes, I think.  3.6 shows thin vertical seams.  Sometime later, these became thicker vertical gaps, which become horizontal seams like in the screenshot if the zoom is changed.  Then later again the thick gaps go horizontal.

The regression range for them becoming horizontal seems to be: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2370ecfdcbb7&tochange=6272ce1da6f5 (presumably the 32-to-64 switch; can be tested by running trunk in 32-bit mode, I assume).

Looking for the range for them becoming wide to start with.

Comment 9

[Boris Zbarsky [:bzbarsky]]

The horizontal gaps on zoom appeared in http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5fda39cd703c&tochange=96de199027d7 (perhaps retained layers).

The wider vertical gaps are older than that.  Is it worth looking up that regression range?

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Does this still reproduce?

Comment 11

[Robert Longson [:longsonr]]

It does.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

Seems highly unlikely we're going to get a useful regression range at this point, however.

Comment 13

[Robert Longson [:longsonr]]

There are no seams on my Mac and we've dropped support for 10.6 now.