766505 - update the default Mac font prefs to include Indic and other fonts now shipped with OS X

Status: RESOLVED FIXED

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

Description

[Jonathan Kew [:jfkthame]]

Attachment: patch, update the default Mac font prefs

The default fonts listed in all.js do not include many of the Indic (etc) fonts that are now shipped with OS X, which causes us to unnecessarily hit system fallback and potentially choose Arial Unicode MS in many cases where this isn't really the best option.

Comment 1

[Jonathan Kew [:jfkthame]]

Pushed this to tryserver to check that changing these prefs won't break any existing tests:
https://tbpl.mozilla.org/?tree=Try&rev=3fd665951565

Comment 2

[John Daggett (:jtd)]

Comment on attachment 634829 [details] [diff] [review]
patch, update the default Mac font prefs

Looks good but I think we should trim out the old Bengali font settings.  Having a font name like this in the pref list will trigger localized name loading for any place that includes even the smallest snippet of Bengali script (e.g. a language list on a BBC page).  Non-default fonts should still be picked up by CoreText system fallback.

Comment 3

[Jonathan Kew [:jfkthame]]

OK, pushed with the old (3rd-party) Bengali names removed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5ad6c86d9f60

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/5ad6c86d9f60