Closed Bug 1230043 Opened 9 years ago Closed 7 years ago

Intermittent TEST-UNEXPECTED-FAIL | leakcheck | tab process: 6008 bytes leaked (AsyncFontInfoLoader, FontInfoData, FontInfoLoadCompleteEvent, nsNameThreadRunnable, nsRunnable, ...)

Categories

(Core :: Graphics: Text, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: philor, Assigned: jtd)

Details

(Keywords: intermittent-failure, memory-leak)

Attachments

(1 file)

This should happen via the shutdown observer but maybe there's some way for the platform fontlist to be torn down without that observer firing? Can't reproduce this leak at this point so this is speculative.
Attachment #8695601 - Flags: review?(jfkthame)
Assignee: nobody → jdaggett
Component: General → Graphics: Text
Comment on attachment 8695601 [details] [diff] [review]
patch, cancel loader on teardown

Review of attachment 8695601 [details] [diff] [review]:
-----------------------------------------------------------------

This would be called when the platformFontList is being destroyed, which happens during gfxPlatform::Shutdown(), right? I don't know quite where that fits into the overall shutdown process, but I'm wondering whether it's wise to be firing off a new Runnable at this point, which is what CancelLoader() will do if there's currently a live mFontLoaderThread. Is the main thread still in a position to handle this, or are we too far into shutdown by this time?
Happened once, in December 2015.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Twice.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Attachment #8695601 - Flags: review?(jfkthame)
This has been resolved by the fix on bug 1241931.
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → WORKSFORME
Nope.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Status: REOPENED → RESOLVED
Closed: 8 years ago7 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: