Bug 1057247 - Increase favicon refetch time to four hours. r=mfinkle, a=sledru
authorRichard Newman <rnewman@mozilla.com>
Wed, 17 Sep 2014 09:22:11 -0700
changeset 216787 515fa121e700
parent 216786 6975bbd6c73a
child 216788 af1dbe183e3d
push id3910
push userryanvm@gmail.com
push date2014-09-18 14:48 +0000
treeherdermozilla-beta@af1dbe183e3d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle, sledru
bugs1057247
milestone33.0
Bug 1057247 - Increase favicon refetch time to four hours. r=mfinkle, a=sledru
mobile/android/base/favicons/cache/FaviconCache.java
--- a/mobile/android/base/favicons/cache/FaviconCache.java
+++ b/mobile/android/base/favicons/cache/FaviconCache.java
@@ -95,18 +95,18 @@ public class FaviconCache {
     private static final String LOGTAG = "FaviconCache";
 
     // The number of spaces to allocate for favicons in each node.
     private static final int NUM_FAVICON_SIZES = 4;
 
     // Dimensions of the largest favicon to store in the cache. Everything is downscaled to this.
     public final int maxCachedWidth;
 
-    // Retry failed favicons after 20 minutes.
-    public static final long FAILURE_RETRY_MILLISECONDS = 1000 * 60 * 20;
+    // Retry failed favicons after four hours.
+    public static final long FAILURE_RETRY_MILLISECONDS = 1000 * 60 * 60 * 4;
 
     // Map relating Favicon URLs with objects representing decoded favicons.
     // Since favicons may be container formats holding multiple icons, the underlying type holds a
     // sorted list of bitmap payloads in ascending order of size. The underlying type may be queried
     // for the least larger payload currently present.
     private final HashMap<String, FaviconsForURL> backingMap = new HashMap<String, FaviconsForURL>();
 
     // And the same, but never evicted.