929441 - Stop unnecessarily setting a transform between emitting a path and stroking it

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

To support non-scaling stroke we undo the current transform by multiplying the inverse transform onto the gfxContext when we set up the stroke, which occurs between when we emit the path and when we actually call gfxContext::Stroke(). Changing the transform between the point when we emit the path and when we use it prevents us switching to Moz2D, so we should stop doing that. This patch doesn't stop us from doing that, but it does eliminate the useless matrix setting (setting of the identity matrix) is this scenario, which helps in tracking down the real cases where the setting is needed.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jonathan Watt [:jwatt]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a8680e083efa

Comment 3

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/a8680e083efa