Closed Bug 802997 Opened 12 years ago Closed 9 years ago

Thunderbird crash in nsLayoutUtils::FontSizeInflationEnabled when changing font size

Categories

(Core :: Layout, defect)

16 Branch
x86
Windows NT
defect
Not set
critical

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox18 - ---

People

(Reporter: Usul, Unassigned)

References

()

Details

(Keywords: crash, regression, topcrash-thunderbird, Whiteboard: [tbird topcrash][regression:TB16][gs])

Crash Data

This bug was filed from the Socorro interface and is report bp-b14dfbbf-605e-4a15-a04f-6e7b02121015 . ============================================================= 0 xul.dll nsLayoutUtils::FontSizeInflationEnabled layout/base/nsLayoutUtils.cpp:4902 1 xul.dll nsFrame::Init layout/generic/nsFrame.cpp:508 2 xul.dll nsSplittableFrame::Init layout/generic/nsSplittableFrame.cpp:23 3 xul.dll nsContainerFrame::Init layout/generic/nsContainerFrame.cpp:64 4 xul.dll nsInlineFrame::Init layout/generic/nsInlineFrame.cpp:48 5 xul.dll nsCSSFrameConstructor::InitAndRestoreFrame layout/base/nsCSSFrameConstructor.cpp:4526 6 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11014 7 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 8 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 9 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 10 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 11 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 12 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 13 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 14 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 15 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 16 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 17 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 18 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 19 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 20 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 21 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 22 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 23 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 24 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 25 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 26 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 27 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 28 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 29 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 30 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 31 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 32 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 33 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 34 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 35 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 36 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 37 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 38 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 39 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 40 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 41 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 42 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 43 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 44 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 45 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 46 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 47 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 48 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 49 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 50 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 51 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 52 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 53 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 54 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 55 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 56 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 57 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 58 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 59 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 60 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 61 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 62 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 63 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 64 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 65 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 66 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 67 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 68 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 69 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 70 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 71 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 72 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 73 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 74 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 75 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 76 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 77 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 78 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 79 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 80 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 81 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 82 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 83 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 84 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 85 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 86 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 87 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 88 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 89 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 90 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 91 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 92 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 93 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 94 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 95 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 96 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 97 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 98 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 99 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 100 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 1015 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 1016 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 1017 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 1018 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 1019 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 1020 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 1021 xul.dll nsCSSFrameConstructor::ConstructFramesFromItemList layout/base/nsCSSFrameConstructor.cpp:9736 1022 xul.dll nsCSSFrameConstructor::ConstructInline layout/base/nsCSSFrameConstructor.cpp:11032 1023 xul.dll nsCSSFrameConstructor::ConstructFrameFromItemInternal layout/base/nsCSSFrameConstructor.cpp:3601 1024 xul.dll nsCSSFrameConstructor::ConstructFramesFromItem layout/base/nsCSSFrameConstructor.cpp:5527 Crash stats comments : Interesting.... the mail changes font sizes as it pleases and has the neerv to CRASH the second I try to change font size on a few lines. Unbelievable! n middle of an email, body of message turned to blue, then my message disappeared. before the crash, controls failed. could not maintained larger font size. could not downsize.
roland, iirc I've seen this reported in gsfn - not the crash, but the randomly changing fonts. ludo, can we correlate this to other font related issues we are seeing? #33 crash for tb16.0.1, but not a topcrash for TB15.0.1, so something regressed. even so, source line in nsLayoutUtils::FontSizeInflationEnabled is same for both versions. So what changed to make this a topcrash? Firefox Crash is 1-2 per version per month, so essentially zero. SO what we have seems specific to Thunderbird. In fact tbird stacks doesn't match most firefox stacks like bp-de2efe4e-cded-4220-aa58-a1b352121015.
Keywords: regression
Whiteboard: [tbird topcrash][TB16 regression]
Version: 17 Branch → 16 Branch
(In reply to Wayne Mery (:wsmwk) from comment #1) > > ludo, can we correlate this to other font related issues we are seeing? I made a poor attempt with no success.
Summary: crash in nsLayoutUtils::FontSizeInflationEnabled → Thunderbird crash in nsLayoutUtils::FontSizeInflationEnabled when changing font size
Blocks: 801400
Can someone @mozilla please look at this? It is very frustrating for users composing mail. AFAICT, not related to quicktext addon crashes (which started in TB17, not TB16 as for this bug) Top 10 crash for TB17, when counts of 3 crash signatures are combined - bug 801400 - nsCycleCollectingAutoRefCnt::incr(void*) #23 crash examples bp-78fa775e-3c5b-4769-9321-5dc712121201 bp-f1b0c77e-4b7b-423b-a630-56a682121129 "trying to increase the font size on a block of text" Does bug belong in editor?
Crash Signature: [@ nsLayoutUtils::FontSizeInflationEnabled(nsPresContext*)] → [@ nsLayoutUtils::FontSizeInflationEnabled(nsPresContext*)] [@ nsCycleCollectingAutoRefCnt::incr(void*) ]
Flags: needinfo?
Keywords: topcrash
This has nothing to do with the editor.
Flags: needinfo?
Since this is a TB-specific crash it won't block Firefox 18, hence not tracking. Please re-nom if investigations lead to Firefox critical issues. We're always open to low-risk uplift nominations too, should a solution be available.
Depends on: 824926
Whiteboard: [tbird topcrash][TB16 regression] → [tbird topcrash][regression:TB16][gs]
Blocks: 833689
TB17.0.5 crash rank still very high, at #16. But like bug 833689 about 70% of crashes have McAfee addon (bug 803460). I've pinged some users who do and don't have McAfee to get some feedback. Leaving open for now. Note ludo's original crash report bp-b14dfbbf-605e-4a15-a04f-6e7b02121015 did not involve McAfee.
No crashes on TB >24 in the last 28 days.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.