The default bug view has changed. See this FAQ.

"ASSERTION: Whoever was caching this font group should have called UpdateFontList on it", take 2

RESOLVED FIXED in mozilla15

Status

()

Core
Canvas: 2D
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: jfkthame)

Tracking

(Blocks: 2 bugs, {assertion, regression, testcase})

Trunk
mozilla15
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 620265 [details]
testcase

###!!! ASSERTION: Whoever was caching this font group should have called UpdateFontList on it: '!mUserFontSet || mCurrGeneration == GetGeneration()', file ../../../gfx/thebes/gfxFont.h, line 3002

(Similar to bug 745699, which was fixed two weeks ago.)
(Assignee)

Comment 1

5 years ago
Created attachment 620280 [details] [diff] [review]
patch, ensure the font list is updated

Ah yes - the use of ctx.measureText("") bypasses the fix that was landed for bug 745699, as the zero-length string doesn't cause us to actually create a text run. So we need to explicitly make sure the font list is updated before accessing its first font's metrics here.
Assignee: nobody → jfkthame
Attachment #620280 - Flags: review?(roc)
Attachment #620280 - Flags: review?(roc) → review+
(Assignee)

Comment 2

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/47bf5e590a0c
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/47bf5e590a0c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.