29515 - Moving mouse over links at abcnews.com freezes browser

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[shrirang khanzode]

Pls reassign to correct component if incorrect. Thnx !

I see this on today's commercial build on all platforms (2000022808m15)

To recreate the problem, please do the following:

1. Install and launch hte browser
2. Type the url 'www.abcnews.com' in the Locationbox and press ENTER
3. Observe the page loads up
4. Now just move the mouse over the links in the left frame( with blue    		  
background). 
5. Observe that the browser freezes and no links work.

Comment 1

[/\/\arcio Galli]

Same here - with: 2000022908, win98
A cut from the code of this page in the area of the problem:
http://www.geckonnection.com/testcases/testlinks.html
- start moving the mouse slowly over the links
- then, move DeSpErAtElY! over the links and see that performance goes down...

Comment 2

[/\/\arcio Galli]

New testcase -->
http://www.geckonnection.com/testcases/testlinks1.html

Note: NOT FREEZING anymore here (2000022908), I cut the <font....> tags from the 
file. 

Comment 3

[shrirang khanzode]

Yup, I see this too..

Comment 4

[ekrock's old account (dead)]

Marking crash and top100 and beta1 and m14. Setting severity critical. 
Freezing/crashing on home page of a top100 site is serious.

Comment 5

[Phil Peterson]

[PDT+] w/b minus on 3/10

Comment 6

[joki (gone)]

Hmm.  Trying to decide if this is a dupe of 18539 that we closed a long time 
ago.  Its still abcnews but that test case still seems to work.  Useful info 
though.

Comment 7

[joki (gone)]

Attachment: somewhat narrowed down testcase

Comment 8

[harishd]

Attachment: A clearer test case...

Comment 9

[harishd]

Attachment: Yet another one...

Comment 10

[joki (gone)]

Okay weird, the test case at the bottom works (as in fails) for me on the server 
but no locally.  Fun.  Gonna put up a smaller testcase to check this out.

Comment 11

[joki (gone)]

Attachment: Test case 6308 with 3 instead of 10 elements for easier study

Comment 12

[joki (gone)]

Okay, got a fix.  Need to get a review and checkin authorization

Comment 13

[joki (gone)]

Harish, by the way.  I found that the parser is most definitely not to blame 
here.  It's definitely a lower level frame issue.  I managed to create identical 
content trees, both of which recursively nested the content nodes, one of which 
showed the freezing problem, one of which did not.

Comment 14

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

If you're curious, this seems to work OK on my current tree (with all my event
targetting changes that I'm waiting until after beta1 to check in).  Not that
that will help much...

Comment 15

[joki (gone)]

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

Comment 16

[joki (gone)]

Okay, this should be much better now.

Comment 17

[Jan Carpenter]

Verified
WinNT, Win98:  2000-03-13-18-M15-nb1b
Mac: 2000-03-13-17-M15-nb1b

Linux: build not working - will verify later

Comment 18

[Jan Carpenter]

verified on linux too