RESOLVED FIXED 16816
REGRESSION (Tiger-Leopard): webView:willPerformDragDestinationAction:forDraggingInfo: not called for a load in Leopard 
https://bugs.webkit.org/show_bug.cgi?id=16816
Summary REGRESSION (Tiger-Leopard): webView:willPerformDragDestinationAction:forDragg...
Rod Schmidt
Reported 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.
Attachments
Simple App to demonstrate bug (90.42 KB, application/zip)
2008-01-10 08:33 PST, Rod Schmidt 		no flags
Details
Another test case from t. ditchendorf showing issue on leo (48.63 KB, application/octet-stream)
2008-01-20 16:37 PST, Todd Ditchendorf 		no flags
Details
Fix (1.20 KB, patch)
2008-01-20 20:19 PST, Oliver Hunt 		mitz: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rod Schmidt
Comment 1 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.
David Kilzer (:ddkilzer)
Comment 2 2008-01-10 20:40:31 PST
<rdar://problem/5682985>
Todd Ditchendorf
Comment 3 2008-01-20 16:37:00 PST
Created attachment 18569 [details] Another test case from t. ditchendorf showing issue on leo
Todd Ditchendorf
Comment 4 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?
Oliver Hunt
Comment 5 2008-01-20 20:19:46 PST
Created attachment 18573 [details] Fix
mitz
Comment 6 2008-01-20 20:23:13 PST
Comment on attachment 18573 [details] Fix r=me
Oliver Hunt
Comment 7 2008-01-20 20:31:26 PST
Fix landed r29681
Note You need to log in before you can comment on or make changes to this bug.