1022792 - "ASSERTION: standard generic font family type required"

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Jesse Ruderman]

Attachment: testcase

###!!! ASSERTION: standard generic font family type required: 'aGenericType >= eFamily_serif && aGenericType <= eFamily_fantasy', file gfx/thebes/gfxFont.cpp, line 4877

This assertion was added in:
changeset:   https://hg.mozilla.org/mozilla-central/rev/e39cfafa8517
user:        John Daggett
date:        Fri Jun 06 15:09:23 2014 +0900
summary:     Bug 280443 p3 - use fontlist struct in gfx. r=roc,fredw

Comment 1

[Jesse Ruderman]

Attachment: stack

Comment 2

[John Daggett (:jtd)]

Attachment: patch, treat -moz-fixed as monospace when resolving generics

This is an artifact of how -moz-fixed was handled prior to the fontlist struct changes in bug 280443. When -moz-fixed was the sole generic in a fontlist, it would set the font family to monospace and *also* set the size. However, when included with other fonts, it would be passed untranslated down to gfx and effectively ignored. Not sure the semantics of this "magic" name were well thought about but for now, just treat -moz-fixed as an alias for monospace within gfx.

David, please let me know if you think we should be making changes within the style system to accomodate this in a different way and I will file a new bug to make those changes.

Comment 3

[John Daggett (:jtd)]

Pushed to inbound
https://hg.mozilla.org/integration/mozilla-inbound/rev/100a14518e5b

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/100a14518e5b

Comment 5

[Paul Silaghi, QA [:pauly]]

Reproduced in Nightly 2014-06-09-mozilla-central-debug.
Verified fixed FF 33.0a1 2014-06-13-mozilla-central-debug Win 7 x64.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

(In reply to John Daggett (:jtd) from comment #2)
> This is an artifact of how -moz-fixed was handled prior to the fontlist
> struct changes in bug 280443. When -moz-fixed was the sole generic in a
> fontlist, it would set the font family to monospace and *also* set the size.
> However, when included with other fonts, it would be passed untranslated
> down to gfx and effectively ignored. Not sure the semantics of this "magic"
> name were well thought about but for now, just treat -moz-fixed as an alias
> for monospace within gfx.

Treating -moz-fixed the same as monospace sounds fine.