Closed Bug 1233632 Opened 7 years ago Closed 7 years ago

Moz2Dify GetBlurAndSpreadRadius()

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: n.nethercote, Assigned: n.nethercote)

Details

Attachments

(1 file)

Another small, easy Moz2Dification step.
The CurrentMatrix()-to-GetTransform() change in GetBlurAndSpreadRadius() is
valid because the operations performed on the resulting |transform| only
consult the _11, _12, _21 and _22 members; not the _31 and _32 members (which
are different in gfxContext vs. DrawTarget).
Attachment #8699826 - Flags: review?(dholbert)
Attachment #8699826 - Flags: review?(dholbert) → review+
Waiting for something for checking in?
Flags: needinfo?(n.nethercote)
I've been on PTO. I will land it today.
Flags: needinfo?(n.nethercote)
https://hg.mozilla.org/mozilla-central/rev/ee768634ba9c
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Looks like either this bug or bug 1233631 (I don't know which because I landed them together) caused this Talos improvement:

> (Improvement) Mozilla-Inbound-Non-PGO - CanvasMark - WINNT 6.1 (ix) (e10s) - 3.01%
> (Improvement) Mozilla-Inbound-Non-PGO - CanvasMark - WINNT 6.1 (ix) - 3.49%
> (Improvement) Mozilla-Inbound-Non-PGO - CanvasMark - WINNT 5.1 (ix) - 3.27%
You need to log in before you can comment on or make changes to this bug.