Bug 58854

Summary: LayoutTest demonstrating that filesystem: URLs are not cached in WebKit
Product: WebKit Reporter: Adam Klein <adamk>
Component: New BugsAssignee: Adam Klein <adamk>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, kinuko, levin, michaeln
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch
none
Patch levin: review+

Adam Klein
Reported 2011-04-18 17:37:28 PDT
LayoutTest demonstrating that filesystem: URLs are not cached in WebKit
Attachments
Patch (3.31 KB, patch)
2011-04-18 17:38 PDT, Adam Klein 		no flags
DetailsFormatted DiffDiff
Patch (3.32 KB, patch)
2011-05-03 10:15 PDT, Adam Klein 		no flags
DetailsFormatted DiffDiff
Patch (3.30 KB, patch)
2011-05-04 12:21 PDT, Adam Klein 		levin: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Adam Klein
Comment 1 2011-04-18 17:38:20 PDT
Created attachment 90122 [details] Patch
Adam Klein
Comment 2 2011-05-03 10:15:11 PDT
Created attachment 92086 [details] Patch
David Levin
Comment 3 2011-05-03 10:18:46 PDT
I'd like Kinuko to do a first pass on this.
Kinuko Yasuda
Comment 4 2011-05-04 12:06:46 PDT
Comment on attachment 92086 [details] Patch The test code looks good to me. View in context: https://bugs.webkit.org/attachment.cgi?id=92086&action=review > LayoutTests/http/tests/filesystem/no-cache-filesystem-url.html:29 > + + rand + ') ? "SUCCESS" : "FAILURE";'); style nit: no need to wrap line
Adam Klein
Comment 5 2011-05-04 12:21:38 PDT
Created attachment 92298 [details] Patch
Adam Klein
Comment 6 2011-05-04 12:21:59 PDT
Comment on attachment 92086 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=92086&action=review >> LayoutTests/http/tests/filesystem/no-cache-filesystem-url.html:29 >> + + rand + ') ? "SUCCESS" : "FAILURE";'); > > style nit: no need to wrap line Fixed.
Adam Klein
Comment 7 2011-05-10 16:28:45 PDT
Committed r86192: <http://trac.webkit.org/changeset/86192>
Note You need to log in before you can comment on or make changes to this bug.