Bug 52096

Summary: [GTK] Filenames are converted to URLs during edit drags
Product: WebKit Reporter: Martin Robinson <mrobinson>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: Gtk
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on: 40833    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Martin Robinson 2011-01-07 18:02:30 PST
This is a companion fix to https://bugs.webkit.org/show_bug.cgi?id=38826. It can be observed by the failure of editing/pasteboard/file-drag-to-editable.html.
Comment 1 Martin Robinson 2011-01-07 18:55:35 PST
Created attachment 78298 [details]
Patch
Comment 2 Eric Seidel (no email) 2011-01-10 13:48:03 PST
Comment on attachment 78298 [details]
Patch

I'm confused by what you're doing for local files.  Is this tested?
Comment 3 Martin Robinson 2011-01-10 13:51:40 PST
(In reply to comment #2)
> (From update of attachment 78298 [details])
> I'm confused by what you're doing for local files.  Is this tested?

Essentially, during edit drags, filenames should not be advertised as a URL on the drag, but otherwise they should be. This bug was fixed for Chromium (and Mac I believe) in https://bugs.webkit.org/show_bug.cgi?id=38826. The test in that bug relied on beginDragWithFiles, so it didn't fail on GTK+ at the time (it was skipped).

This patch fixes the bug and unskips the test in question (it depends on the landing of beginDragWithFiles) patch.
Comment 4 Martin Robinson 2011-01-10 13:51:41 PST
(In reply to comment #2)
> (From update of attachment 78298 [details])
> I'm confused by what you're doing for local files.  Is this tested?

Essentially, during edit drags, filenames should not be advertised as a URL on the drag, but otherwise they should be. This bug was fixed for Chromium (and Mac I believe) in https://bugs.webkit.org/show_bug.cgi?id=38826. The test in that bug relied on beginDragWithFiles, so it didn't fail on GTK+ at the time (it was skipped).

This patch fixes the bug and unskips the test in question (it depends on the landing of beginDragWithFiles) patch.
Comment 5 Eric Seidel (no email) 2011-01-11 03:05:33 PST
Comment on attachment 78298 [details]
Patch

OK.
Comment 6 Martin Robinson 2011-01-18 11:06:34 PST
Committed r76039: <http://trac.webkit.org/changeset/76039>