378836 - ASSERTION: Fonts still alive while shutting down gfxFontCache: 'mFonts.Count() == 0' file gfxFont.h, line 165, with new textframe

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Simon Montagu :smontagu]

With the new textframe I get this assertion almost every time on shutdown, followed by a crash.

Comment 1

[Karl Tomlinson (:karlt)]

I get this on Linux after viewing about:plugins.

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

As far as I can tell this happens only when we leak DOMWindows a la bug 380837. Removing blocking of new-textframe on that basis. If someone reproduces when DOMWindows are not leaked, please say so and restore the blocking link...

Comment 3

[Simon Montagu :smontagu]

Attachment: testcase

This causes the assertion without leaking DOMWindows. I suspect that it is leaking something else, though, so I won't restore the blocking.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Thanks for that! I found a nasty little leak which I have a fix for in my tree.

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Restoring blocking flag since I have a fix

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I checked in my fix. This problem may occur again if we leak textruns, but there are now assertions in place that should fire if that happens so we can get more specific bug reports. In any case new bug(s) should be filed.