Closed Bug 1451656 Opened 7 years ago Closed 6 years ago

limit support for variation fonts to secure contexts

Categories

(Core :: Layout: Text and Fonts, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox60 --- wontfix
firefox61 --- affected

People

(Reporter: dbaron, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: dev-doc-needed)

I realize I missed this when reviewing bug 1447163, but per the policy described in https://blog.mozilla.org/security/2018/01/15/secure-contexts-everywhere/ support for variation fonts should be limited to secure contexts. I don't *think* this is implemented, although I might have missed this. I think it's important that we stick to this policy uniformly for substantive new features, so I think this support should be limited to secure contexts. This includes making the feature detection points for the feature (new properties/values) detect as false in non-secure contexts.
Flags: needinfo?(jfkthame)
Hmmm.... yes, we've overlooked this so far. I wonder, though, if it's necessarily appropriate in this case. Are any other UAs implementing such a restriction? A brief local test suggests that both Edge and Chrome will happily use a variation font from a non-secure context. While it's true that variation font support involves a couple of new CSS properties, it is largely an extension to existing properties/features; when support is enabled, variation fonts will respond differently to properties like font-weight. Restricting variation support to secure contexts will mean that the exact same page, using a variable font resource but with _no_ explicit use of variation-specific properties (and hence no obvious motive to do feature detection, etc) may render quite differently in secure vs non-secure contexts. I don't have macOS 10.13 on hand for testing; Markus, could I ask you to see whether Safari on 10.13 restricts variation font support to secure contexts?
Flags: needinfo?(jfkthame) → needinfo?(mstange)
(In reply to Jonathan Kew (:jfkthame) from comment #1) > I don't have macOS 10.13 on hand for testing; Markus, could I ask you to see > whether Safari on 10.13 restricts variation font support to secure contexts? It does not seem to restrict it. I rehosted https://jfkthame.github.io/variation-fonts/gingham.html at http://tests.themasta.com/gingham.html (just the html page and the font, not the screenshots at the bottom), and the two look identical in Safari 11.1 on macOS 10.13.4.
Flags: needinfo?(mstange)
OK, thanks for testing. So given that all the other major browsers are shipping this without a secure-context restriction, I think we should do the same. :dbaron, are you OK with resolving this as WONTFIX, or do we need further discussion first?
Flags: needinfo?(dbaron)
Keywords: site-compat
Keywords: site-compat
Depends on: 1471807

WONTFIX given other browsers ship this without restriction. (Feel free to resolve other bugs similarly, ask me when in doubt.)

Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(dbaron)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.