Changeset 201857 in webkit

Timestamp:

Jun 9, 2016, 1:41:54 AM (9 years ago)

Author:

Antti Koivisto

Message:

Increase disk cache capacity when there is lots of free space
​https://bugs.webkit.org/show_bug.cgi?id=158526

Reviewed by Chris Dumez.

Our maximum disk cache capacity has been 175MB for a very long time. Meanwhile the average resource size has grown
massively. 175MB is now enough for cacheable resources of a few dozen sites at most. Using bigger caches when there is
room for it can reduce network traffic, reduce disk writes, save battery and speed up page loading.

This patch changes cache size for PrimaryWebBrowser cache model

=16GB free: 175MB -> 500MB

8-16GB free: 150MB -> 250MB

and modestly for DocumentBrowser

=16GB free: 50MB -> 75MB

Note that if system (on Cocoa platforms) really needs disk space it will wipe out caches so this doesn't reduce
available disk space in scenarios like software installation.

• Shared/CacheModel.cpp:

(WebKit::calculateCacheSizes):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• Shared/CacheModel.cpp (modified) (3 diffs)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2016-06-08  Antti Koivisto  <antti@apple.com>
+
+        Increase disk cache capacity when there is lots of free space
+        https://bugs.webkit.org/show_bug.cgi?id=158526
+
+        Reviewed by Chris Dumez.
+
+        Our maximum disk cache capacity has been 175MB for a very long time. Meanwhile the average resource size has grown
+        massively. 175MB is now enough for cacheable resources of a few dozen sites at most. Using bigger caches when there is
+        room for it can reduce network traffic, reduce disk writes, save battery and speed up page loading.
+
+        This patch changes cache size for PrimaryWebBrowser cache model
+
+        >=16GB free: 175MB -> 500MB
+        8-16GB free: 150MB -> 250MB
+
+        and modestly for DocumentBrowser
+
+        >=16GB free: 50MB -> 75MB
+
+        Note that if system (on Cocoa platforms) really needs disk space it will wipe out caches so this doesn't reduce
+        available disk space in scenarios like software installation.
+
+        * Shared/CacheModel.cpp:
+        (WebKit::calculateCacheSizes):
+
 2016-06-08  Youenn Fablet  <youenn.fablet@crf.canon.fr>
 

trunk/Source/WebKit2/Shared/CacheModel.cpp

@@ -56 +56 @@
         urlCacheMemoryCapacity = 0;
 
-        // Foundation disk cache capacity (in bytes)
+        // Disk cache capacity (in bytes)
         urlCacheDiskCapacity = 0;
 
@@ -93 +93 @@
             urlCacheMemoryCapacity =      512 * 1024; 
 
-        // Foundation disk cache capacity (in bytes)
+        // Disk cache capacity (in bytes)
         if (diskFreeSize >= 16384)
-            urlCacheDiskCapacity = 50 * 1024 * 1024;
+            urlCacheDiskCapacity = 75 * 1024 * 1024;
         else if (diskFreeSize >= 8192)
             urlCacheDiskCapacity = 40 * 1024 * 1024;
@@ -154 +154 @@
 #endif
 
-        // Foundation disk cache capacity (in bytes)
+        // Disk cache capacity (in bytes)
         if (diskFreeSize >= 16384)
-            urlCacheDiskCapacity = 175 * 1024 * 1024;
+            urlCacheDiskCapacity = 500 * 1024 * 1024;
         else if (diskFreeSize >= 8192)
-            urlCacheDiskCapacity = 150 * 1024 * 1024;
+            urlCacheDiskCapacity = 250 * 1024 * 1024;
         else if (diskFreeSize >= 4096)
             urlCacheDiskCapacity = 125 * 1024 * 1024;