[shared font list] macOS "hidden" system font families should not be recognized in the CSS `font-family` list
Categories
(Core :: Layout: Text and Fonts, defect, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox74 | --- | fixed |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
Testcase:
data:text/html,<div style="font:36px '.SF NS Text', Times">Hello World
renders (correctly) in Times Roman without the shared font list enabled, but in .SF NS Text (which should not be exposed by name to the user) with the shared list enabled.
|
Assignee | |
Comment 1•5 years ago
|
||
Inspecting the shared font list code shows that it correctly sets a "hidden family" flag on these fonts when building the list on macOS.... but subsequently fails to pay any attention to that flag when it's time to look up font names.
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
||
Updated•5 years ago
|
|
||
Comment 4•5 years ago
|
||
Backed out for assertion failures on StaticPrefList_gfx.h
Backout link: https://hg.mozilla.org/integration/autoland/rev/558a2ae20a30f33cc1b042dab27ef04eccb4ea58
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284303527&repo=autoland&lineNumber=18597
|
|
Assignee | |
Comment 5•5 years ago
|
||
Ah, we can't use the reftest manifest to tweak a mirror: once pref. (And changing the pref to mirror: always won't help, as it is only checked at startup.) So we should just drop the pref annotation from this testcase and not worry about having it run for now. When we actually flip the default value, it'll be here to let us know if things are broken.
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
Description
•