Closed
Bug 1258636
Opened 10 years ago
Closed 10 years ago
Use struct to pass params for decoration line painting
Categories
(Core :: Layout: Text and Fonts, defect)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
mozilla48
| Tracking | Status | |
|---|---|---|
| firefox48 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
Details
Attachments
(2 files)
No description provided.
|
Assignee | |
Comment 1•10 years ago
|
||
|
|
Assignee | |
Comment 2•10 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/41857/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/41857/
Attachment #8733659 -
Flags: review?(jfkthame)
|
|
Assignee | |
Comment 3•10 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/41859/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/41859/
Attachment #8733660 -
Flags: review?(jfkthame)
|
||
Comment 4•10 years ago
|
||
Comment on attachment 8733659 [details]
MozReview Request: Bug 1258636 part 1 - Use structs to pass params for decoration-related functions in nsCSSRendering.
https://reviewboard.mozilla.org/r/41857/#review38437
Yes, seems like nice cleanup - let's do it.
Attachment #8733659 -
Flags: review?(jfkthame) → review+
|
|
||
Comment 5•10 years ago
|
||
Comment on attachment 8733660 [details]
MozReview Request: Bug 1258636 part 2 - Use struct to pass params for nsTextFrame::PaintDecorationLine.
https://reviewboard.mozilla.org/r/41859/#review38439
Attachment #8733660 -
Flags: review?(jfkthame) → review+
|
||
Comment 7•10 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/cf926fcefda2
https://hg.mozilla.org/mozilla-central/rev/e4d32bd5e11e
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•