Closed
Bug 167204
Opened 23 years ago
Closed 22 years ago
[CSS] using :first-letter with floating elements locks mozilla
Categories
(Core :: Layout: Floats, defect, P2)
Core
Layout: Floats
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: pascalc, Unassigned)
Details
(Keywords: hang, testcase)
Attachments
(1 file)
|
528 bytes,
text/html
|
Details |
1 go to the testcase I will put in attachment
2 [details] [diff] [review] put your mouse over the text
Expected result: all the <span> elements inside <h2></h2> should become block
elements floating on the left and the first letter of the <h2> title should
become red.
Actual result: Mozilla gets locked and unresponsive with 99% CPU use and I have
to kill the task
Here is the code of the page :
<?xml version="1.0" encoding="iso-8859-1"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xml:lang="fr">
<head>
<title>test</title>
<style type="text/css">
div.bloc:hover > h2 span { display:block; float:left}
div.bloc > h2:first-letter {color:red;}
</style>
</head>
<body>
<div class="bloc">
<h2>
<span>vendredi</span>
<span>06</span>
<span>septembre</span>
<span>2002</span>
</h2>
</div>
</body>
</html>
->Layout
Assignee: dbaron → attinasi
Component: Style System → Layout
QA Contact: ian → petersen
| Reporter | ||
Comment 2•23 years ago
|
||
If you delete float:left, Mozilla no longer gets locked.
The infinite loop is in:
#0 nsCSSFrameConstructor::RemoveFloatingFirstLetterFrames (this=0x87c6d90,
aPresContext=0x888c700, aPresShell=0x888f930, aFrameManager=0x888ffa8,
aBlockFrame=0x88a5d8c, aStopLooking=0xbfffe570)
at /builds/trunk/mozilla/layout/html/style/src/nsCSSFrameConstructor.cpp:12994
#1 0x41c69285 in nsCSSFrameConstructor::RemoveLetterFrames (this=0x87c6d90,
aPresContext=0x888c700, aPresShell=0x888f930, aFrameManager=0x888ffa8,
aBlockFrame=0x88a5d8c)
at /builds/trunk/mozilla/layout/html/style/src/nsCSSFrameConstructor.cpp:13181
#2 0x41c5ede7 in nsCSSFrameConstructor::ContentRemoved (this=0x87c6d90,
aPresContext=0x888c700, aContainer=0x88a7e20, aChild=0x88a7f78,
aIndexInContainer=5, aInContentReplaced=0)
at /builds/trunk/mozilla/layout/html/style/src/nsCSSFrameConstructor.cpp:9726
...
because |floater| is its own next sibling, so the loop over next siblings never
terminates.
| Reporter | ||
Comment 4•23 years ago
|
||
I forgot to tell about my configuration :
Windows Xp Athlon 1700XP 256 MB Ram
Build 2002090415, the problem also affects NS7
Keywords: hang
Severity: major → critical
OS: Windows XP → All
Hardware: PC → All
Comment 5•23 years ago
|
||
this regressed between linux trunk builds 2002041008 and 2002041108
and 1.0 branch builds 20020421 and 20020430
look like bug 5693
Severity: critical → major
Hardware: All → PC
Updated•23 years ago
|
QA Contact: petersen → amar
Updated•23 years ago
|
Priority: -- → P2
Updated•23 years ago
|
Target Milestone: --- → Future
Comment 7•23 years ago
|
||
==> floats
Assignee: attinasi → float
Component: Layout → Layout: Floats
QA Contact: amar → ian
Comment 8•23 years ago
|
||
Reproduced on 12/21/02 Trunk build, Linux.
Comment 9•23 years ago
|
||
I just reproduced the problem with this test case on a pc with win98 and
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.3b) Gecko/20030210. I was looking to
see if there was bug that has been already reported. Thnis problem has not shown
up in earlier builds for me than the one I just loaded. I've used builds from 1,
1.1, 1.1a, 1.1b, 1.2.1, 1.2b, 1.3a.
I get simular results on
http://www.weather.com/weather/climatology/monthly/USFL0372
sorta like bug 177574 but I see the contents before locking
http://moonfeyrie.homestead.com/linkmusic.html
mine. With both of these pages I can run the scroll bar around a while before
it locks up.
| Reporter | ||
Comment 10•22 years ago
|
||
FYI, problem is still there with build 2003060908
Comment 11•22 years ago
|
||
This is fixed?
WFM Mozilla 1.6a
Comment 12•22 years ago
|
||
This bug worksforme with Windows Seamonkey builds 2004020909
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•