Closed Bug 788087 Opened 9 years ago Closed 9 years ago

[Azure] Cairo surface patterns do not apply the pattern matrix

Categories

(Core :: Graphics, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: ajones, Assigned: ajones)

Details

Attachments

(1 file, 1 obsolete file)

When a surface is used as a pattern the pattern matrix is not applied in DrawTargeCairo. This causes images to display incorrectly when using DrawTarget directly for example with Azure Cairo content.
QA Contact: ajones
Try run for fec6e51fba98 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=fec6e51fba98
Results (out of 252 total builds):
    success: 231
    warnings: 16
    failure: 5
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/ajones@mozilla.com-fec6e51fba98
Comment on attachment 657996 [details] [diff] [review]
Apply pattern matrix when using a surface pattern.

The fact that the matrix is inverted is due to us being in device space, correct? Probably deserves a comment.
Attachment #657996 - Flags: review?(joe) → review+
From cairo docs of cairo_pattern_set_matrix:

 * Important: Please note that the direction of this transformation 
 * matrix is from user space to pattern space. This means that if you 
 * imagine the flow from a pattern to user space (and on to device 
 * space), then coordinates in that flow will be transformed by the 
 * inverse of the pattern matrix.
Attachment #657996 - Attachment is obsolete: true
Comment on attachment 658270 [details] [diff] [review]
Apply pattern matrix when using a surface pattern v2 (with comments)

Review of attachment 658270 [details] [diff] [review]:
-----------------------------------------------------------------

when I r+ I don't need to see it again :)
Attachment #658270 - Flags: review+
Try run for fec6e51fba98 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=fec6e51fba98
Results (out of 257 total builds):
    success: 234
    warnings: 17
    failure: 6
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/ajones@mozilla.com-fec6e51fba98
Patch has changed since try push but only comments.
Keywords: checkin-needed
(In reply to Mozilla RelEng Bot from comment #7)
>     https://tbpl.mozilla.org/?tree=Try&rev=fec6e51fba98

Green on Try.

https://hg.mozilla.org/integration/mozilla-inbound/rev/bc9be79a2eeb
Assignee: nobody → ajones
Flags: in-testsuite-
Keywords: checkin-needed
QA Contact: ajones
https://hg.mozilla.org/mozilla-central/rev/bc9be79a2eeb
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.