Bug 29242 - [Qt] Assert hit when dropping a file in QtLauncher
Summary: [Qt] Assert hit when dropping a file in QtLauncher
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2009-09-14 05:50 PDT by Yael
Modified: 2009-09-14 08:37 PDT (History)
0 users

See Also:


Attachments
Patch (1.15 KB, patch)
2009-09-14 05:56 PDT, Yael
vestbo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yael 2009-09-14 05:50:27 PDT
Assert is hit if the file path contains a space. The reason is that KURL can only accept escaped strings and the file name is not escaped.
I checked on mac port, and the file name is escaped in DrapData::asURL()
Comment 1 Yael 2009-09-14 05:56:50 PDT
Created attachment 39541 [details]
Patch

DropData::asURL() now encodes the URL before returning it.
Comment 2 Yael 2009-09-14 08:37:37 PDT
Landed in r48354