nsPresContext::mBaseMinFontSize and co are dead code.

RESOLVED FIXED in Firefox 66

Status

()

enhancement
P3
normal
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: emilio, Assigned: emilio)

Tracking

(Blocks 1 bug)

unspecified
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(2 attachments)

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 months ago

Nobody uses it from js, and we only thread the value around in layout. Let's
kill all this code.

Comment 3

5 months 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 months ago
bugherder
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66

Comment 5

5 months ago
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/1a43fcfbf783
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.