RESOLVED FIXED104413
There is no need to change cached resource storage policy through ResourceHandleClient 
https://bugs.webkit.org/show_bug.cgi?id=104413
Summary There is no need to change cached resource storage policy through ResourceHan...
Alexey Proskuryakov
Reported 2012-12-07 16:30:08 PST
There are two willCacheResource delegates in ResourceHandleClient, one that can replace the whole resource, and another that can change CacheStoragePolicy only. It's very confusing. And all the latter is ever used for is to prevent storing to disk when private browsing is enabled. But private browsing storage session has 0 disk cache capacity, so it never stores anything anyway! So, I'd like to remove this code.
Attachments
proposed patch (8.90 KB, patch)
2012-12-07 16:33 PST, Alexey Proskuryakov 		beidson: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2012-12-07 16:33:20 PST
Created attachment 178301 [details] proposed patch
Alexey Proskuryakov
Comment 2 2012-12-07 16:39:36 PST
Committed <http://trac.webkit.org/changeset/136998>.
Note You need to log in before you can comment on or make changes to this bug.