Closed
Bug 496293
Opened 15 years ago
Closed 15 years ago
loading http://www.mtv3.fi locks up browser
Categories
(Core :: Layout, defect)
Tracking
()
VERIFIED
DUPLICATE
of bug 494332
People
(Reporter: smaug, Unassigned)
References
Details
(Keywords: regression)
Attachments
(6 files)
Loading or re-loading http://www.mtv3.fi (a Finnish news site) locks up the whole browser. When looking at the stack, browser seems to be reflowing the page. This is trunk only.
Comment 1•15 years ago
|
||
Cannot see the lock with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090603 Minefield/3.6a1pre ID:20090603031250
Reporter | ||
Comment 2•15 years ago
|
||
Based on Shark-profile, we're looping somewhere under blockframe reflowing inlines.
Comment 3•15 years ago
|
||
Comment 4•15 years ago
|
||
Comment 5•15 years ago
|
||
Comment 6•15 years ago
|
||
Is this a regression from bug 25888?
Comment 8•15 years ago
|
||
This one I see after I have disabled Flash.
Comment 9•15 years ago
|
||
The following assertion is visible when doing a shift+reload.
Comment 10•15 years ago
|
||
I believe the last added assertion is the one which is causing this problem. We loop in nsBlockFrame::PlaceLine: http://mxr.mozilla.org/mozilla-central/source/layout/generic/nsBlockFrame.cpp#4042
Reporter | ||
Comment 11•15 years ago
|
||
(In reply to comment #7) > Is this a regression from bug 25888? Probably. I can't reproduce this using 2009/05/19, but can with 2009/05/21
Blocks: 25888
Comment 12•15 years ago
|
||
I'll bisect: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=815856a7144b&tochange=be75e40365ce
There's probably not much point in bisecting since we know what bug it was.
Comment 14•15 years ago
|
||
(In reply to comment #13) > There's probably not much point in bisecting since we know what bug it was. We only have the date but yes, after running builds from both changesets I can second that it is a regression from bug 25888. Shall I file separate bugs for the other assertions?
Keywords: regression
(In reply to comment #9) > Created an attachment (id=381497) [details] > Assertion (available space should never grow: 'aOldAvailableSpace.x <= > aNewAvailableSpace.x && aOldAvailableSpace.XMost() >= > aNewAvailableSpace.XMost()) > > The following assertion is visible when doing a shift+reload. This assertion is what causes the problem in this bug. The other assertions are other bugs, though may already be filed.
Comment 16•15 years ago
|
||
(In reply to comment #15) > The other assertions are other bugs, though may already be filed. I filed bug 496336 for the UTF8 assertions.
This works for me, but it wouldn't surprise me if the patch I'm going to attach to bug 494332 fixes it.
Keywords: regression
Reporter | ||
Comment 18•15 years ago
|
||
I believe something has changed in the page, since I'm no longer getting lockups either.
Comment 19•15 years ago
|
||
It's still there with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090609 Minefield/3.6a1pre ID:20090609212135. As I said in comment 9 just do a shift+reload. Only then you will see the assertion.
Could somebody test with a build from 2009-06-17 or later? I suspect this was fixed by bug 494332?
Updated•15 years ago
|
Flags: blocking1.9.2?
Keywords: regression
Comment 22•15 years ago
|
||
(In reply to comment #21) > Could somebody test with a build from 2009-06-17 or later? I suspect this was > fixed by bug 494332? With the most recent OS X build (20090709), no more problems.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
Comment 24•15 years ago
|
||
Looks good for me too. The assertion is not visible anymore.
Status: RESOLVED → VERIFIED
Flags: blocking1.9.2?
You need to log in
before you can comment on or make changes to this bug.
Description
•