Closed Bug 1385861 Opened 3 years ago Closed 3 years ago

Fine tune nsDisplaySVGText::Paint

Categories

(Core :: Web Painting, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: u459114, Assigned: u459114)

References

(Blocks 1 open bug)

Details

(Whiteboard: [qf:p1])

Attachments

(4 files, 1 obsolete file)

No description provided.
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
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [qf]
Whiteboard: [qf] → [qf:p1]
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 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 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 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 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+
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
You need to log in before you can comment on or make changes to this bug.