360616 - Crash [@ nsCSSFrameConstructor::WipeContainingBlock] on this site sometimes with Load Time Analyzer extension installed

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Martijn Wargers (dead)]

I'm getting a crash sometimes on this site with the Load Time Analyzer extension installed, when I have multiple tabs simultaneously loading.

I have a sort of slimmed down version, that sometimes crashes with the steps to reproduce that I'll describe shortly.

Talkback ID: TB25963856X
nsCSSFrameConstructor::WipeContainingBlock  [mozilla\layout\base\nscssframeconstructor.cpp, line 12980]
nsCSSFrameConstructor::ContentAppended  [mozilla\layout\base\nscssframeconstructor.cpp, line 8891]

Comment 1

[Martijn Wargers (dead)]

Attachment: not totally minimised testcase

To reproduce:
- Install nightly tester tools (to be able to install Load Time Analyzer): -> http://users.blueprintit.co.uk/~dave/web/firefox/nightly
- Install Load Time Analyzer: -> https://addons.mozilla.org/firefox/3371/
- Extract attached testcase
- Open in 8 tabs
- Do a reload on all tabs
- Now switch tabs back and forth, while all the tabs are reloading, result -> crash

Comment 2

[Martijn Wargers (dead)]

So I get a regression range for the crash between 2006-09-09 and 2006-09-10:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-09-09+12&maxdate=2006-09-10+06&cvsroot=%2Fcvsroot

I also see a regression in page loading performance, where the recent builds are very slow while loading the testcase and eating all cpu, that regressed between 2006-07-20 and 2006-07-21:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-07-20+04&maxdate=2006-07-21+06&cvsroot=%2Fcvsroot

Comment 3

[Nickolay_Ponomarev]

FWIW, I see the performance problems, but not the crash with a recent trunk build.

Comment 4

[Martijn Wargers (dead)]

I just had this crash without the Load Time Analyzer enabled, but with a load of other extensions installed, I suspect the Extended Statusbar 1.2.5 might be involved this time.

Comment 5

[Martijn Wargers (dead)]

This is now worksforme, using current trunk build and doing the steps to reproduce.
I'm still seeing the performance problem, however.

Comment 6

[Martijn Wargers (dead)]

Oops, maybe I shouldn't have closed this bug? Maybe "the underlying problem is still there"?

Comment 7

[Boris Zbarsky [:bzbarsky]]

Hard to tell, since we never figured out what that was.

Is there a separate bug on the performance issue?  That should be filed, and request blocking1.9.

Comment 8

[Martijn Wargers (dead)]

Sorry, in fact, I don't see a performance regression issue at all, anymore, using current trunk build.

I'm just marking it worksforme, since nobody figured out why the crash was happening, and it doesn't happen anymore with current trunk build.