536296 - Fonts on webpages arent rendering properly

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Tony Chung [:tchung]]

Not all websites are rendering their fonts properly on fennec.   I've attached screenshots of Maemo Browser and Firefox Desktop vs fennec.  The spacing in the font is also an issue, but i believe that's known and being fixed.

Repro:
1) visit www.bensbargains.net
2) Verify the font in the "Ben's Bargains", and also the bolded title of the first deal differs on fennec versus maemo browser and desktop Firefox Browser

Expected:
- fennec should render the font to be the same as maemo and firefox

Actual:
- the font largely differs

Comment 1

[Tony Chung [:tchung]]

Attachment: Website_Firefox

Comment 2

[Tony Chung [:tchung]]

Attachment: Website_MaemoBrowser

Comment 3

[Tony Chung [:tchung]]

Attachment: Website_Fennec

Comment 4

[Tony Chung [:tchung]]

Using: Mozilla/5.0 (X11; U; Linux armv71; en-US; rv:1.9.2b6pre) Gecko/20091221
Firefox/3.6b6pre Fennec/1.0b6pre

Comment 5

[Ben Combee (:bcombee)]

The "Ben's Bargains" header that's shown differently is requesting font "verdana" via an inline CSS style attribute.   While we don't have a font named "verdana" on the N900, we should be mapping that to a sans-serif font.  I looked in MicroB using about:config, and they didn't set a non-serif default font, so maybe it's a separate font mapping change in how they use Freetype.

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

MicroB changes the defaults in all.js to use a sans-serif font named "SwissA" while we default to "serif"

http://mxr.mozilla.org/mozilla-central/source/modules/libpref/src/init/all.js#1560

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

"SwissA" appears in the following prefs:
x-baltic
x-central-euro
x-cyrillic
x-unicode
x-user-def
x-western

Comment 8

[Ben Combee (:bcombee)]

I didn't spot that the SwissA usage when I looked in about:config on MicroB, but checking again confirms Mark's investigation.

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

patch uses SwissA, a sans serif font, by default for content text.

Comment 11

[Mark Finkle (:mfinkle) (use needinfo?)]

matches MicroB

Comment 12

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 431096 [details] [diff] [review]
patch

I assume doing the same as microB is the right thing to do here.

Comment 13

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/22cec426fd8b

Comment 14

[Aakash Desai [:aakashd]]

verified FIXED on builds:
Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2.2pre) Gecko/20100309 Namoroka/3.6.2pre Fennec/1.1a1

and

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.2pre) Gecko/20100309 Namoroka/3.6.2pre Fennec/1.1a2pre

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a2pre) Gecko/20100309 Namoroka/3.7a2pre Fennec/1.1a2pre