Bug 16816 - REGRESSION (Tiger-Leopard): webView:willPerformDragDestinationAction:forDraggingInfo: not called for a load in Leopard
Summary: REGRESSION (Tiger-Leopard): webView:willPerformDragDestinationAction:forDragg...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Oliver Hunt
URL:
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2008-01-10 08:30 PST by Rod Schmidt
Modified: 2008-01-21 13:10 PST (History)
3 users (show)

See Also:


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+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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