Moz2Dify CanvasBidiProcessor

RESOLVED FIXED in Firefox 46

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

unspecified
mozilla46
Points:
---

Firefox Tracking Flags

(firefox46 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
CanvasBidiProcessor::mThebes can be changed from a gfxContext* to a
DrawTarget*.
(Assignee)

Comment 1

3 years ago
Created attachment 8699825 [details] [diff] [review]
Moz2Dify CanvasBidiProcessor

I'm pretty sure the SetMatrix()-to-SetTransform() change at the end of the
patch is valid, but please check it carefully. The rest of the patch is
trivial.
Attachment #8699825 - Flags: review?(jfkthame)
Attachment #8699825 - Flags: review?(jfkthame) → review+

Comment 2

3 years ago
Waiting for something for checking in?

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/29e7fd863abd
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
(Assignee)

Comment 5

3 years ago
Looks like either this bug or bug 1233632 (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.