Closed
Bug 1385861
Opened 7 years ago
Closed 7 years ago
Fine tune nsDisplaySVGText::Paint
Categories
(Core :: Web Painting, enhancement, P2)
Core
Web Painting
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox57 | --- | fixed |
People
(Reporter: u459114, Assigned: u459114)
References
Details
Attachments
(4 files, 1 obsolete file)
No description provided.
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
mozilla::TextRenderedRunIterator::First take time which out of my expectation
Duplicate gfxSave/Restore - I sould find a way to catch all this kind of problem
|
||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Updated•7 years ago
|
Whiteboard: [qf] → [qf:p1]
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
Attachment #8892039 -
Attachment is obsolete: true
Attachment #8891991 -
Flags: review?(matt.woodrow)
Attachment #8891992 -
Flags: review?(matt.woodrow)
Attachment #8892040 -
Flags: review?(matt.woodrow)
Attachment #8891993 -
Flags: review?(matt.woodrow)
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 25•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8891991 [details]
Bug 1385861 - Part 1. Remove redundant gfxContext::Save/Restore in nsDisplaySVGText::Paint.
https://reviewboard.mozilla.org/r/163004/#review169050
Attachment #8891991 -
Flags: review?(matt.woodrow) → review+
|
|
||
Comment 26•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8891992 [details]
Bug 1385861 - Part 2. Save the state of input context in SVGTextFrame::PaintSVG only when need.
https://reviewboard.mozilla.org/r/163006/#review169052
Attachment #8891992 -
Flags: review?(matt.woodrow) → review+
|
|
||
Comment 27•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8891993 [details]
Bug 1385861 - Part 3. Rename nsSVGUtils::SetupCairoStrokeGeometry as nsSVGUtils::SetupStrokeGeometry.
https://reviewboard.mozilla.org/r/163008/#review169058
Attachment #8891993 -
Flags: review?(matt.woodrow) → review+
|
|
||
Comment 28•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8892040 [details]
Bug 1385861 - Part 4. Reduce usage of SetMatrix in SVGTextFrame::PaintSVG.
https://reviewboard.mozilla.org/r/163044/#review169062
Attachment #8892040 -
Flags: review?(matt.woodrow) → review+
|
||
Comment 29•7 years ago
|
||
Pushed by cku@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/615e94f1555b
Part 1. Remove redundant gfxContext::Save/Restore in nsDisplaySVGText::Paint. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/e8c4ad3acfd0
Part 2. Save the state of input context in SVGTextFrame::PaintSVG only when need. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/61072ea18a88
Part 3. Rename nsSVGUtils::SetupCairoStrokeGeometry as nsSVGUtils::SetupStrokeGeometry. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/5d10630d73e2
Part 4. Reduce usage of SetMatrix in SVGTextFrame::PaintSVG. r=mattwoodrow
|
||
Comment 30•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/615e94f1555b
https://hg.mozilla.org/mozilla-central/rev/e8c4ad3acfd0
https://hg.mozilla.org/mozilla-central/rev/61072ea18a88
https://hg.mozilla.org/mozilla-central/rev/5d10630d73e2
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
|
||
Updated•3 years ago
|
Performance Impact: --- → P1
Whiteboard: [qf:p1]
You need to log in
before you can comment on or make changes to this bug.
Description
•