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)
Core
Graphics: Text
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: philor, Assigned: jtd)
Details
(Keywords: intermittent-failure, memory-leak)
Attachments
(1 file)
708 bytes,
patch
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•9 years ago
|
||
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 | ||
Updated•9 years ago
|
Assignee: nobody → jdaggett
Assignee | ||
Updated•9 years ago
|
Component: General → Graphics: Text
Comment 2•9 years ago
|
||
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?
Comment 3•8 years ago
|
||
Happened once, in December 2015.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Updated•8 years ago
|
Attachment #8695601 -
Flags: review?(jfkthame)
Assignee | ||
Comment 5•8 years ago
|
||
This has been resolved by the fix on bug 1241931.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → WORKSFORME
Comment hidden (Intermittent Failures Robot) |
Comment 8•8 years ago
|
||
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Updated•7 years ago
|
Status: REOPENED → RESOLVED
Closed: 8 years ago → 7 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•