Closed Bug 1233632 Opened 9 years ago Closed 8 years ago

Moz2Dify GetBlurAndSpreadRadius()

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla46
Tracking Flags:
Tracking Status
firefox46 --- fixed

People

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

Details

Attachments

(1 file)

Moz2Dify GetBlurAndSpreadRadius()
5.59 KB, patch
dholbert
: review+
Details | Diff | Splinter Review 
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: 8 years ago
status-firefox46: --- → fixed
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.

Attachment

General

Creator:
Created:
Updated:
Size: