1826412 - Improve Font Visibility Restrictions for Android

Status: RESOLVED FIXED

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

Description

[Tom Ritter [:tjr]]

Currently Android does not apply Font Visibility Restrictions (even if they are enabled) because it lacks a hardcoded list of preloaded fonts on any android devices.

Comment 1

[Tom Ritter [:tjr]]

Attachment: font-list.txt

After a bunch of data spelunking, I've come up with the attached list of fonts for Android devices, separated by Android Release level. Chromebook and Amazon Fire devices will have a separate list though.

Comment 2

[Tom Ritter [:tjr]]

Attachment: font-list.txt

Comment 3

[Tom Ritter [:tjr]]

Attachment: font-list.txt

Comment 4

[Tom Ritter [:tjr]]

Attachment: font-list.txt

Comment 5

[Tom Ritter [:tjr]]

Attachment: font-list.txt

Comment 6

[Tom Ritter [:tjr]]

Attachment: Bug 1826412: Apply Font Visibility Restrictions to Android r?jfkthame

Comment 7

[Tom Ritter [:tjr]]

Attachment: Bug 1826412: Do not apply Font Visbility Restrictions to Android in FPP by default r?timhuang

Comment 8

[Tom Ritter [:tjr]]

Attachment: Bug 1826412: Rip out all the ugly and move it to nsIGfxInfo r?jfkthame

This also adds debug information to about:support

Comment 9

[Tom Ritter [:tjr]]

Attachment: Bug 1826412: Move the Linux Distro stuff to nsIGfxInfo also r?jfkthame

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc0f90aef845
Apply Font Visibility Restrictions to Android r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/889c2c0ffd50
Do not apply Font Visbility Restrictions to Android in FPP by default r=tschuster
https://hg.mozilla.org/integration/autoland/rev/af653a9af72d
Rip out all the ugly and move it to nsIGfxInfo r=jfkthame,fluent-reviewers,flod
https://hg.mozilla.org/integration/autoland/rev/c11b5221fac4
Move the Linux Distro stuff to nsIGfxInfo also r=jfkthame

Comment 11

[Sandor Molnar[:smolnar]]

Backed out for causing leakcheck failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/948128206207a630ad7dce3fcf0bd356616176d7

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | leakcheck | default 8 bytes leaked (nsStringBuffer)

Comment 12

[Tom Ritter [:tjr]]

Oops; alright added a ClearOnShutdown.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3e69e06c6fcc
Apply Font Visibility Restrictions to Android r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/32b267d7fd31
Do not apply Font Visbility Restrictions to Android in FPP by default r=tschuster
https://hg.mozilla.org/integration/autoland/rev/a5f0b8602ce1
Rip out all the ugly and move it to nsIGfxInfo r=jfkthame,fluent-reviewers,flod
https://hg.mozilla.org/integration/autoland/rev/ef615f845f6a
Move the Linux Distro stuff to nsIGfxInfo also r=jfkthame

Comment 14

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3e69e06c6fcc
https://hg.mozilla.org/mozilla-central/rev/32b267d7fd31
https://hg.mozilla.org/mozilla-central/rev/a5f0b8602ce1
https://hg.mozilla.org/mozilla-central/rev/ef615f845f6a

Comment 15

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3e69e06c6fcc
https://hg.mozilla.org/mozilla-central/rev/32b267d7fd31
https://hg.mozilla.org/mozilla-central/rev/a5f0b8602ce1
https://hg.mozilla.org/mozilla-central/rev/ef615f845f6a