Bug 7084 - REGRESSION: Multiple load events dispatched to <body> (makes layout tests hang)
Summary: REGRESSION: Multiple load events dispatched to <body> (makes layout tests hang)
Status: VERIFIED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Blocker
Assignee: Maciej Stachowiak
URL: http://www.haaretz.co.il/?from=hasot
Keywords: HasReduction, Regression
: 7100 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-02-05 06:55 PST by mitz
Modified: 2006-02-06 11:22 PST (History)
3 users (show)

See Also:

Attachments
Testcase (120 bytes, text/html)
2006-02-05 07:25 PST, mitz 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2006-02-05 06:55:13 PST 
Safari is unresponsive and its CPU usage is very high when going to the above URL.
Comment 1 Alexey Proskuryakov 2006-02-05 07:06:38 PST 
That would be very new - I don't see this problem with r12570. Possibly a regression from a fix in bug 7079 - it's known to hang on at least one regression test.
Comment 2 mitz 2006-02-05 07:14:56 PST 
(In reply to comment #1)
> That would be very new - I don't see this problem with r12570. Possibly a
> regression from a fix in bug 7079 - it's known to hang on at least one
> regression test.
> 

This could very well be it: it looks like the root cause for this bug is that <body>'s onload handler is being called multiple times.
Comment 3 mitz 2006-02-05 07:25:28 PST 
Created attachment 6273 [details]
Testcase

This demonstrates that the onload handler is being called twice.
Comment 4 Darin Adler 2006-02-05 09:21:57 PST 
Maciej was looking at this last night -- it's from his last check-in with "event clean-up". We'll probably need to roll that out.
Comment 5 Darin Adler 2006-02-05 09:25:17 PST 
Alexey believes Maciej had a fix ready when he went to sleep last night.
Comment 6 Darin Adler 2006-02-05 10:59:09 PST 
Rolling out the change because it's preventing us from running layout tests. Maciej will re-land it soon.
Comment 7 Joost de Valk (AlthA) 2006-02-06 11:22:46 PST 
*** Bug 7100 has been marked as a duplicate of this bug. ***