RESOLVED FIXED Bug 65184
Web Inspector: Disable cache option should only clear memory cache, not disable it. 
https://bugs.webkit.org/show_bug.cgi?id=65184
Summary Web Inspector: Disable cache option should only clear memory cache, not disab...
Vsevolod Vlasov
Reported 2011-07-26 07:44:07 PDT
Disable cache option should only clear memory cache, not disable it. Apparently clearing memory cache on reload should be enough for developers. This will also have less side effects. E.g. disabling memory cache causes several willSendRequest calls for some resources.
Attachments
Patch (12.58 KB, patch)
2011-07-26 07:49 PDT, Vsevolod Vlasov 		no flags
DetailsFormatted DiffDiff
Patch (13.56 KB, patch)
2011-07-27 02:42 PDT, Vsevolod Vlasov 		pfeldman: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1 2011-07-26 07:49:09 PDT
Created attachment 102002 [details] Patch
WebKit Review Bot
Comment 2 2011-07-26 08:12:54 PDT
Comment on attachment 102002 [details] Patch Attachment 102002 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/9250393 New failing tests: http/tests/inspector/network/network-disable-cache-memory.html http/tests/inspector/network/network-disable-cache-xhrs.html
Vsevolod Vlasov
Comment 3 2011-07-27 02:42:09 PDT
Created attachment 102111 [details] Patch
Vsevolod Vlasov
Comment 4 2011-07-27 11:30:44 PDT
Committed r91858: <http://trac.webkit.org/changeset/91858>
Note You need to log in before you can comment on or make changes to this bug.