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 API

Assignee:

Oliver Hunt <oliver>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

oliver, todd, vicki

Priority:

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+

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