Bug 16816

Summary: REGRESSION (Tiger-Leopard): webView:willPerformDragDestinationAction:forDraggingInfo: not called for a load in Leopard
Product: WebKit Reporter: Rod Schmidt <rod.schmidt>
Component: WebKit APIAssignee: Oliver Hunt <oliver>
Status: RESOLVED FIXED    
Severity: Normal CC: oliver, todd, vicki
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Simple App to demonstrate bug
none
Another test case from t. ditchendorf showing issue on leo
none
Fix mitz: review+

Description Rod Schmidt 2008-01-10 08:30:12 PST
In Tiger, when you dragged a web archive to a web view, the UI delegate method webView:willPerformDragDestinationAction:forDraggingInfo: was called. We used this to differentiate between a normal load and a dragging load. In Leopard and the latest WebKit, this method is not called.
Comment 1 Rod Schmidt 2008-01-10 08:33:29 PST
Created attachment 18365 [details]
Simple App to demonstrate bug

This is a simple app to demonstrate the bug. Once it launches (under Leopard), drag a web archive to the window. Look at the console to see what gets printed. You will see that the webView:willPerformDragDestionationAction:... method is not called.

You can run the same app under Tiger (you will have to change the build settings to target 10.4), and you will see that the method is called.
Comment 2 David Kilzer (:ddkilzer) 2008-01-10 20:40:31 PST
<rdar://problem/5682985>
Comment 3 Todd Ditchendorf 2008-01-20 16:37:00 PST
Created attachment 18569 [details]
Another test case from t. ditchendorf showing issue on leo
Comment 4 Todd Ditchendorf 2008-01-20 16:37:53 PST
I also see this issue on leo. I've attached another test case. This seems very serious, is there any workaround? is my test case missing some vital step to make this work?
Comment 5 Oliver Hunt 2008-01-20 20:19:46 PST
Created attachment 18573 [details]
Fix
Comment 6 mitz 2008-01-20 20:23:13 PST
Comment on attachment 18573 [details]
Fix

r=me
Comment 7 Oliver Hunt 2008-01-20 20:31:26 PST
Fix landed r29681