Bug 3445

Summary: REGRESSION: Browser in Apple widget download page stops working after download
Product: WebKit Reporter: Maciej Stachowiak <mjs>
Component: DOMAssignee: Maciej Stachowiak <mjs>
Severity: Normal    
Priority: P1    
Version: 412   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.apple.com/downloads/dashboard/
Description Flags
Latest reduction
crash log none

Description Maciej Stachowiak 2005-06-11 14:33:46 PDT
6/8/05 10:16 AM Sean MacMillan:
I was browsing the new widget download (http://www.apple.com/downloads/dashboard/).  After I 
download the ReefSanctuary widget, I could not select anything else in either column.  Closing the tab 
caused Safari to crash.
1)Go to http://www.apple.com/downloads/dashboard
2)Select a widget and download it
3)After downloading, I cannot browse or select any other widgets
4)Safari may crash when closing the tab
Continue browsing widgets and Safari to not crash
Couldn't browse widgets, Safari crashed
Browsing widgets works as expected until a widget is downloaded.
Page works as expected in Firefox 1.0.4
log attached

6/8/05 1:07 PM James McComb:
Reproduced with 8C29 - once a widget is downloaded, I am unable to browse other widgets.
Safari did not crash when closing tab, though.

Send to me & I'll try with 10.4.1.

6/9/05 12:10 PM James McComb:
Does not occur with 10.4.1 - updating to 8C30 shows the problem.
This is a regression in Cambridge.

6/9/05 3:51 PM Chris Petersen:
I can reproduce a portion of the problem but not the crash. After starting the widget download, I  can  
no-longer navigate by clicking on the subjects in the Category list field. Basically, clicking on a different 
subjects name in the Category doesn't load the set of items in the Widgets list field.

If I reload the page at this point, I will regain the ability to navigate in the list.

6/9/05 3:53 PM Chris Petersen:
Tested with 8C29. Going to see when this problem first occurred.

6/9/05 4:36 PM Chris Petersen:
From my testing, this regression first occurred with 8C21.

6/10/05 12:10 PM Chris Petersen:
Is this crash happening on consistent basis ? I can reproduce the broken functionality of the list 
navigation but not the crash.

6/10/05 1:26 PM Adele Peterson:
This regression appears to be caused by our fix for 3598589 (fixes onunload).  I'm working on getting a 

6/10/05 1:39 PM Chris Petersen:
Ok, I have attached a reduced test case of the problem. I noticed that on the original page after 
downloading a widget, the select menu (that redirects to different page after you choose a item) wasn't 
working as well. 

The reduced test case contains this simple select element and link that references a widget.zip file. 

To reproduce:

1) Load test "download_reduction.html" case in Safari
2) Verify that selecting any item in the select menu redirects to that specific page
3) Now return back to "download_reduction.html" and click on the link
4) When the DL window appears, bring the browser window to the front and choose a different menu 
item in the list.
5) Notice the URL is loaded for the select menu item now.

Hope this reproduces !

6/10/05 2:28 PM Adele Peterson:
note- spoofing as WinIE fixes this.  I wonder if they were trying to work around our unonload bug 

6/10/05 5:48 PM Adele Peterson:
Maciej is working on a fix for this.

Radar: rdar://problem/4142247
Comment 1 Maciej Stachowiak 2005-06-11 14:39:27 PDT
Created attachment 2252 [details]
Latest reduction
Comment 2 Maciej Stachowiak 2005-06-11 14:43:32 PDT
Created attachment 2253 [details]
crash log
Comment 3 Maciej Stachowiak 2005-06-12 03:21:52 PDT
fix sent for review