1189129 - add crashstack annotations when aborting in gfxFontGroup::GetDefaultFont

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[John Daggett (:jtd)]

There are currently a whole slew of crasher bugs related to aborts within gfxFontGroup::GetDefaultFont. This happens when, for some reason, we can't find *any* font on the system to use.

http://mxr.mozilla.org/mozilla-central/source/gfx/thebes/gfxTextRun.cpp#1899

Before we do this we should add more annotations to the crash report via gfxCriticalError. For example, how many fonts in the fontlist, whether errors occurred during global fontlist enumeration, whether the initialization routine was ever called, etc.

Comment 1

[John Daggett (:jtd)]

Attachment: patch, annotate no default font aborts

When no default font is found, note in the crashreport the number of calls to InitFontList, the number of fonts in the system fontlist, and the state of the fontloader thread.

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/638db7656bee

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/638db7656bee

Comment 4

[John Daggett (:jtd)]

To view the crashstack annotations added here:

1. Open the crashstack report

Ex: https://crash-stats.mozilla.com/report/index/35dd3094-92b7-442e-b78a-876662150807

2. Click on the 'Metadata' tab

3. Look for 'GraphicsCriticalError' in the list

Ex: |[0][GFX1]: no fonts - init: 1 fonts: 0 loader: 1

This indicates initialization occurred, no fonts are in the fontlist and the font loader thread is in the "waiting to fire" state, stateTimerOnDelay.