Closed
Bug 751129
Opened 12 years ago
Closed 12 years ago
"ASSERTION: Whoever was caching this font group should have called UpdateFontList on it", take 2
Categories
(Core :: Graphics: Canvas2D, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: jruderman, Assigned: jfkthame)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(2 files)
451 bytes,
text/html
|
Details | |
1.69 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
###!!! 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•12 years ago
|
||
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•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/47bf5e590a0c
Target Milestone: --- → mozilla15
Comment 3•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/47bf5e590a0c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•