Closed
Bug 1576846
Opened 5 years ago
Closed 5 years ago
Crash in [@ mozilla::fontlist::FontList::Alloc]
Categories
(Core :: Layout: Text and Fonts, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla70
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox69 | --- | unaffected |
firefox70 | --- | fixed |
People
(Reporter: over68, Assigned: jfkthame)
References
(Regression)
Details
(Keywords: regression)
Crash Data
Attachments
(2 files)
Steps to reproduce:
- Set gfx.e10s.font-list.shared to true.
- Restart Firefox.
- Download Font Loader.
- Download Franklin Gothic Book Regular.ttf.
- Open the testcase.
- Open the Font Loader, Click on the Add Fonts button, Select the font file Franklin Gothic Book Regular.ttf then click Open.
- Click on the Load button then Unload.
Actual results:
The tab crashed.
Crash report: bp-bbd366f2-00a3-4ba1-8dba-071700190827
Top 10 frames of crashing thread:
0 xul.dll mozilla::fontlist::FontList::Alloc gfx/thebes/SharedFontList.cpp:607
1 xul.dll mozilla::fontlist::FontList::SetAliases gfx/thebes/SharedFontList.cpp:660
2 xul.dll void gfxPlatformFontList::CancelInitOtherFamilyNamesTask gfx/thebes/gfxPlatformFontList.cpp:2123
3 xul.dll gfxPlatformFontList::InitFontList gfx/thebes/gfxPlatformFontList.cpp:403
4 xul.dll gfxPlatformFontList::UpdateFontList gfx/thebes/gfxPlatformFontList.cpp:671
5 xul.dll gfxPlatform::UpdateFontList gfx/thebes/gfxPlatform.cpp:1742
6 xul.dll mozilla::dom::ContentChild::RecvRebuildFontList dom/ipc/ContentChild.cpp:2626
7 xul.dll mozilla::dom::PContentChild::OnMessageReceived ipc/ipdl/PContentChild.cpp:9539
8 xul.dll mozilla::ipc::MessageChannel::DispatchMessage ipc/glue/MessageChannel.cpp:2108
9 xul.dll mozilla::ipc::MessageChannel::RunMessage ipc/glue/MessageChannel.cpp:1955
Assignee | ||
Comment 3•5 years ago
|
||
Huh - if I'm reading the crash report correctly, it looks like the content process is trying to do stuff that's supposed to be chrome-process-only.
Assignee: nobody → jfkthame
Flags: needinfo?(jfkthame)
Priority: -- → P3
Updated•5 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 4•5 years ago
|
||
This feature isn't enabled by default so I don't think we should worry about this for regression triage at this point.
Updated•5 years ago
|
Crash Signature: [@ mozilla::fontlist::FontList::Alloc] → [@ mozilla::fontlist::FontList::Alloc]
[@ mozilla::fontlist::Family::FindAllFacesForStyle ]
Assignee | ||
Comment 5•5 years ago
|
||
Assignee | ||
Comment 6•5 years ago
|
||
I believe this should resolve the crash, at least for the original signature here; will test more locally after my Windows machine finishes updating itself.... (sigh).
Updated•5 years ago
|
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3acfcd288f56
Fix the content-process's use of mAliasTable to temporarily record alternate font family names on Windows. r=jwatt
Comment 8•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Comment 9•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
status-firefox69:
--- → unaffected
status-firefox-esr60:
--- → unaffected
status-firefox-esr68:
--- → unaffected
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•