UNCONFIRMED 120514
[WebKit2]SetCacheModel must be callled when ResourceCachesToClear is AllResourceCaches 
https://bugs.webkit.org/show_bug.cgi?id=120514
Summary [WebKit2]SetCacheModel must be callled when ResourceCachesToClear is AllResou...
Sangho Kim
Reported 2013-08-30 01:08:29 PDT
WebProcess has clearResourceCaches function. It deletes Disk cache by using setCacheModel(DocumentViewer) But I think if ResourceCachesToClear is onlyInMemory, Disk cache should not be deleted.
Attachments
patch (1.60 KB, patch)
2013-08-30 01:18 PDT, Sangho Kim 		beidson: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sangho Kim
Comment 1 2013-08-30 01:18:38 PDT
Created attachment 210075 [details] patch When the argument of clearResourceCaches is only AllResourceCaches, SetCacheModel can be called.
Alexey Proskuryakov
Comment 2 2013-09-03 12:14:32 PDT
This doesn't look right to me - we still need to call setCacheModel to clear memory cache here.
Sangho Kim
Comment 3 2013-09-03 16:55:06 PDT
(In reply to comment #2) > This doesn't look right to me - we still need to call setCacheModel to clear memory cache here. If the setCacheModel(CacheModelDocumentViewer) is called, Disk Cache is deleted. I think Disk Cache should not be deleted when user want to delete only caches in memory.
Alexey Proskuryakov
Comment 4 2013-09-04 09:45:19 PDT
This is not necessarily accurate - for example, Mac implementation doesn't shrink disk cache.
Brady Eidson
Comment 5 2016-05-24 22:02:37 PDT
Comment on attachment 210075 [details] patch Assuming that patches for review since 2013 are stale, r-
Note You need to log in before you can comment on or make changes to this bug.