519368 - <pattern> viewBox not treated correctly

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Martin Hejral]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; cs; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; cs; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1

By my opinion, there can be problem with the temporary memory used when replicate pattern...

Reproducible: Always

Steps to Reproduce:
Please try the SVG test file
Actual Results:  
See screenshot from the FF

Expected Results:  
See screenshot from the Safari

Comment 1

[Martin Hejral]

Attachment: test file

Comment 2

[Martin Hejral]

Attachment: Expected result

Comment 3

[Martin Hejral]

Attachment: Bad rendering in FF

Comment 4

[Robert Longson [:longsonr]]

The test file looks the same to me in Opera and Firefox i.e. a big black square. That's what both the screenshots look like too. What difference am I supposed to be seeing?

Comment 5

[Takeshi Kurosawa :takenspc]

Attachment: Testcase (remove viewBox from pattern and translate manually)

Removing |viewBox| from <pattern> (and translating pattern shape manually), this doesn't happened.
The problem of viewBox handling of patterns?

Comment 6

[Robert Longson [:longsonr]]

Ahh I see now. Red on black is pretty much invisible on my laptop.

Comment 7

[Robert Longson [:longsonr]]

I wonder if this is something to do with overflow? I don't think we support overflow on patterns currently.

Comment 8

[Robert Longson [:longsonr]]

Can the testcase be reduced further please?

Comment 9

[Robert Longson [:longsonr]]

Attachment: patch

I think this is right. It is copied from nsSVGMarkerElement::GetViewBoxTransform

Comment 10

[Robert Longson [:longsonr]]

Attachment: same patch with reftest

Comment 11

[Robert Longson [:longsonr]]

checked in http://hg.mozilla.org/mozilla-central/rev/540cddf958e3