Closed Bug 1687577 Opened 4 years ago Closed 4 years ago

Heavy use of CFF2 fonts crashes Firefox

Categories

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

Firefox 84
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: frankrolf, Unassigned)

Details

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15

Steps to reproduce:

I am working on a font specimen page for Source Serif, which basically is a wall of text. The page currently is live at this URL: https://frankrolf.github.io
In the CSS font stack, CFF2 fonts are prioritized. It seems that these fonts do crash Firefox. If I remove the CFF2 files from the stack, the page works normally.

step-by-step:

Actual results:

  • page crashes FF completely
  • re-starts of FF are followed by immediate crashes
  • only “refreshing” FF helps

Expected results:

  • page should not crash

I should notice that other pages with CFF2 fonts do not crash FF, for example https://adobe-fonts.github.io/source-sans-pro/ or https://v-fonts.com/fonts/source-code-variable – however the amount of text is much lower on these.

Further remark:
I just tested the same page on Firefox Nightly 86.0a1 (2021-01-19) – no crash.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: Untriaged → Layout: Text and Fonts
Product: Firefox → Core

Based on local testing, I believe this should be fixed in Firefox 85, which is due for release next week.

Great news! Thanks for the quick response.

frankrolf, can you please do double-check that the issue has been fixed in 85 release? Thank you!

Flags: needinfo?(frankrolf)

Yes I tested the page on Firefox 85 earlier today, and it just performs normally.
In the meantime the content has moved to https://adobe-fonts.github.io/source-serif/ (and will stay there for a while).

Thanks again for the fix!

Flags: needinfo?(frankrolf)

Thanks! Closing as FIXED and putting "dupme".

Status: UNCONFIRMED → RESOLVED
Closed: 4 years ago
Keywords: dupeme
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.