Closed
Bug 1064056
Opened 10 years ago
Closed 10 years ago
Convert all consumers of nsRenderingContext transform related methods to use gfxContext::SetMatrix()/Multiply()
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: jwatt, Assigned: jwatt)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
Attachments
(3 files, 1 obsolete file)
12.15 KB,
patch
|
bas.schouten
:
review+
jwatt
:
checkin+
|
Details | Diff | Splinter Review |
2.38 KB,
patch
|
bas.schouten
:
review+
jwatt
:
checkin+
|
Details | Diff | Splinter Review |
9.62 KB,
patch
|
bas.schouten
:
review+
jwatt
:
checkin+
|
Details | Diff | Splinter Review |
To fix bug 1064055 my plan is to first get us to the point where all non-DrawTarget using consumers are using only gfxContext::SetMatrix() and gfxContext::Multiply(). We can then much more easily convert the consumers of those methods to use the Moz2D API all in one go, avoiding issues like bug 1060762.
![]() |
Assignee | |
Comment 1•10 years ago
|
||
Attachment #8485502 -
Flags: review?(bas)
![]() |
Assignee | |
Comment 2•10 years ago
|
||
Attachment #8485503 -
Flags: review?(bas)
![]() |
Assignee | |
Comment 3•10 years ago
|
||
Attachment #8485504 -
Flags: review?(bas)
![]() |
Assignee | |
Updated•10 years ago
|
![]() |
Assignee | |
Comment 4•10 years ago
|
||
The patches I have for this bug also coalesce various calls that change gfxContext matrices, which should improve perf, Bas tells me.
Keywords: perf
Updated•10 years ago
|
Attachment #8485502 -
Flags: review?(bas) → review+
![]() |
Assignee | |
Comment 5•10 years ago
|
||
Attachment #8485503 -
Attachment is obsolete: true
Attachment #8485503 -
Flags: review?(bas)
Attachment #8487466 -
Flags: review?(bas)
Updated•10 years ago
|
Attachment #8485504 -
Flags: review?(bas) → review+
Updated•10 years ago
|
Attachment #8487466 -
Flags: review?(bas) → review+
![]() |
Assignee | |
Comment 6•10 years ago
|
||
![]() |
Assignee | |
Updated•10 years ago
|
Attachment #8485502 -
Flags: checkin+
![]() |
Assignee | |
Updated•10 years ago
|
Attachment #8485504 -
Flags: checkin+
![]() |
Assignee | |
Updated•10 years ago
|
Attachment #8487466 -
Flags: checkin+
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/125aa9f4bd3d
https://hg.mozilla.org/mozilla-central/rev/22001b6093df
https://hg.mozilla.org/mozilla-central/rev/13937d4616b2
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•