OGURA Junya Reporter
	Description • 23 years ago

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.6) Gecko/20011120
BuildID:    2001112009

If "line-height" property defined in "userContent.css" then Mozilla crashes then
redering simply HTML that has one TABLE element and one TEXTAREA element.

userContent.css ----------------------------------------

* { line-height:120%; }

--------------------------------------------------------


simply.html --------------------------------------------
<TABLE>
  <TBODY>
    <TR>
      <TD>
        <TEXTAREA name="text" rows="10" cols="60"></TEXTAREA>
      </TD>
    </TR>
  </TBODY>
</TABLE>
--------------------------------------------------------


Reproducible: Always
Steps to Reproduce:
1.Add "* { line-height:120%; }" in userContent.css
2.Run Mozilla
3.Open simply.html
4.Crashes


Actual Results:  Mozilla crashes.


Expected Results:  Rendering one TABLE and one TEXTAREA form.

	Julien Cegarra
	Comment 1 • 23 years ago

Confirmed. BuildID: 2001112104 (Win98).

the sample miss a <link rel="stylesheet" type="text/css" 
href="userContent.css" />

attaching a right one.

	Julien Cegarra
	Comment 4 • 23 years ago

After a bit more work, i find that the prob is due to line-height applied to a
<TABLE ..> containing a textarea.

<TABLE STYLE="line-height:120%;"><TR><TD>
<TEXTAREA name="text" rows="10" cols="60"></TEXTAREA>
</TD></TR></TABLE>

	Adam Hauner
	Comment 6 • 23 years ago

Sev: normail --> critical
Key: +crash

TB38365087W with 2001112104/WinNT4

	Matthias Versen [:Matti]
	Comment 7 • 23 years ago

confirming with http://bugzilla.mozilla.org/showattachment.cgi?attach_id=58848 
and
win2k build 20011120..

	David Baron :dbaron: Assignee
	Comment 9 • 23 years ago

The problem was that I was computing percentage line heights relative to the
parent's font size rather than the font size of the element itself.  Once I fix
that, there's no need to worry about a null check.

	Pierre Saslawsky
	Comment 11 • 23 years ago

Comment on attachment 59118 [details] [diff] [review]
patch

r=pierre

	David Hyatt
	Comment 12 • 23 years ago

sr=hyatt

	David Baron :dbaron: Assignee
	Comment 13 • 23 years ago

Fix checked in 2001-11-27 20:35 PDT.

	David Baron :dbaron: Assignee
	Comment 14 • 23 years ago

*** Bug 112787 has been marked as a duplicate of this bug. ***

	Jay Patel [:jay]
	Comment 15 • 23 years ago

Adding M096 and topcrash keyword for tracking...this was a topcrasher with
Mozilla 0.9.6.

	Madhur Bhatia
	Comment 16 • 23 years ago

verified fixed on buildid :2002-04-22-1.0.0 branch