227496 - text disappears when you move the mouse over the main menu

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect, P2)

Description

[Bernard Alleysson]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.6b) Gecko/20031202 Firebird/0.7+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.6b) Gecko/20031202 Firebird/0.7+

text disappear when you move the mouse over the main menu

Reproducible: Always

Steps to Reproduce:
1.move the mouse over the main menu on the page
2.
3.

Actual Results:  
text disappears for main menu options

Expected Results:  
text for menu options should be displayed

this is a regression from mozilla 1.5

Comment 1

[Hermann Schwab]

the site doesn´t have a doctype
the site reproducably delivers 500 Internal Server Error when checked by
http://validator.w3.org
http://validator.w3.org/check?uri=http%3A%2F%2Fwww.activeplus.com%2F
though clicking on the URL shown on the error page was bringing me to the site.

The js code is unreadable for somebody not knowing much about js, they don´t
send whitespace over the expensive network connection.
The first line in sniffer.js is so long it doesn´t show completely in the
browser, but it isn´t wrapped either.

http://www.activeplus.com/us/menu/sniffer.js

the menu: ULTIMATE DROPDOWN MENU VERSION 3.6.1 by Brothercake
http://www.activeplus.com/us/menu/custom.js

the stylesheet:
http://www.activeplus.com/us/menu/style.js

Comment 2

[Bernard Alleysson]

make the summary clearer and asking blocking1.6 because it is a highly visible
regression

Comment 3

[Brendan Eich [:brendan]]

Need some kind of reduced testcase, or a "bonsai regression window", if not a
full diagnosis.  Cc'ing people who may be able to help.

/be

Comment 4

[Boris Zbarsky [:bzbarsky]]

I can't reproduce this with my normal profile, but a vanilla profile shows it. 
The regression is sometime between 2003-09-13-05 and 2003-09-14-05.

Chances are, this is either bug 173277 or bug 218222.  More likely the former.

I won't have time to testcase this before I leave, but ccing some people who may
be able to help with that....

Comment 5

[Brendan Eich [:brendan]]

dbaron, can you take this for investigation?  Do we need to fix it for 1.6?

/be

Comment 6

[Andrew Schultz]

Attachment: testcase

this testcase exhibits the bug with linux trunk 2003121014

mouseover "foo" and it will disappear (it should not disappear).

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It sounds more like something that could have been caused by bug 217604, but
that's way out of the regression window...

Comment 8

[Mats Palmgren (inactive)]

This is somewhat similar to bug 227816 (rel.pos. TD is not painted)

Comment 9

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The problem seems to be that when the mouse is moved over the testcase, the view
for the TD ends up with a width,height of 0,0.

Comment 10

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

In nsContainerFrame::SyncFrameViewAfterReflow, I'm seeing a view being resized
to 0,0 based on having the NS_FRAME_OUTSIDE_CHILDREN bit set and *aCombinedArea
being {0,0,0,0}.  I didn't check that it's the frame in question, but I think
that's a safe assumption.

Comment 11

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

Comment 12

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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

desiredSize.mOverflowArea was previously uninitialized in this codepath.  The
other change is to avoid calling cellFrame->GetDesiredSize() twice -- it was
called just before the |if| for the |else| in the diff.

Comment 13

[Brendan Eich [:brendan]]

If the patch is considered safe, we should avoid shipping this regression in 1.6.

/be

Comment 14

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

taking

Comment 15

[Robert O'Callahan (:roc) (email my personal email if necessary)]

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

Yum

Comment 16

[tor]

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

a=tor for 1.6 checkin.

Comment 17

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in to trunk, 2003-12-12 14:58 -0800.