Closed
Bug 1521137
Opened 5 years ago
Closed 5 years ago
nsPresContext::mBaseMinFontSize and co are dead code.
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla66
| Tracking | Status | |
|---|---|---|
| firefox66 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Nobody calls nsIContentViewer.minFontSize from JS. This is just dead code, let's rip it off so I can kill another dependency from style to the pres context.
|
Assignee | |
Comment 1•5 years ago
|
||
Nobody uses it from js, and we only thread the value around in layout. Let's
kill all this code.
|
|
Assignee | |
Comment 2•5 years ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/39f5f9728b69 The pres context's base min font size is dead code. r=heycam
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Pushed by emilio@crisal.io: https://hg.mozilla.org/integration/autoland/rev/1a43fcfbf783 Remove another font API outside of the pres context. r=heycam
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
|
||
Comment 7•5 years ago
|
||
https://hg.mozilla.org/projects/cedar/rev/1a43fcfbf783fe3d134c30394d5d83f474faf93b Bug 1521137 - Remove another font API outside of the pres context. r=heycam
You need to log in
before you can comment on or make changes to this bug.
Description
•