Bug 33051 - [Qt] layoutTestController.pathToLocalResource() should return a path, not an URL
Summary: [Qt] layoutTestController.pathToLocalResource() should return a path, not an URL
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P3 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-30 05:17 PST by Jakub Wieczorek
Modified: 2009-12-30 06:30 PST (History)
2 users (show)

See Also:


Attachments
proposed patch (2.09 KB, patch)
2009-12-30 05:20 PST, Jakub Wieczorek
no flags Details | Formatted Diff | Diff
proposed patch (2.09 KB, patch)
2009-12-30 05:32 PST, Jakub Wieczorek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Wieczorek 2009-12-30 05:17:01 PST
This is one of the reasons why http/tests/plugins/post-url-file.html fails.
There is another one, which I am going to address in another bug.
Comment 1 Jakub Wieczorek 2009-12-30 05:20:27 PST
Created attachment 45655 [details]
proposed patch
Comment 2 WebKit Review Bot 2009-12-30 05:21:09 PST
style-queue ran check-webkit-style on attachment 45655 [details] without any errors.
Comment 3 WebKit Review Bot 2009-12-30 05:23:57 PST
Attachment 45655 [details] did not build on qt:
Build output: http://webkit-commit-queue.appspot.com/results/152278
Comment 4 Jakub Wieczorek 2009-12-30 05:32:50 PST
Created attachment 45657 [details]
proposed patch
Comment 5 WebKit Review Bot 2009-12-30 05:38:14 PST
style-queue ran check-webkit-style on attachment 45657 [details] without any errors.
Comment 6 Simon Hausmann 2009-12-30 06:09:12 PST
Comment on attachment 45657 [details]
proposed patch

r=me
Comment 7 WebKit Commit Bot 2009-12-30 06:30:46 PST
Comment on attachment 45657 [details]
proposed patch

Clearing flags on attachment: 45657

Committed r52659: <http://trac.webkit.org/changeset/52659>
Comment 8 WebKit Commit Bot 2009-12-30 06:30:50 PST
All reviewed patches have been landed.  Closing bug.