[iOS] Reduce UpdateDelay

components/history/core/browser/top_sites_impl.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/history/core/browser/top_sites_impl.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 57 matching lines @@
 
 // Max number of temporary images we'll cache. See comment above
 // temp_images_ for details.
 const size_t kMaxTempTopImages = 8;
 
 const int kDaysOfHistory = 90;
 // Time from startup to first HistoryService query.
 const int64 kUpdateIntervalSecs = 15;
 // Intervals between requests to HistoryService.
 const int64 kMinUpdateIntervalMinutes = 1;
+#if !defined(OS_IOS)
 const int64 kMaxUpdateIntervalMinutes = 60;
+#else
+// On mobile, having the max at 60 results in the topsites database being

[sky, 2015/05/11 15:10:00]

Your comment says mobile, you're ifdef says ios. Should the ifdef be for android
too?

[sdefresne, 2015/05/11 15:49:21]

I don't know. Do you know who I can ask?

+// not updated often enough since the app isn't usually running for long
+// stretches of time.
+const int64 kMaxUpdateIntervalMinutes = 5;
+#endif  // !defined(OS_IOS)
 
 // Use 100 quality (highest quality) because we're very sensitive to
 // artifacts for these small sized, highly detailed images.
 const int kTopSitesImageQuality = 100;
 
 }  // namespace
 
 // Initially, histogram is not recorded.
 bool TopSitesImpl::histogram_recorded_ = false;
 
@@ skipping 817 matching lines @@
     for (std::set<size_t>::reverse_iterator i = indices_to_delete.rbegin();
          i != indices_to_delete.rend(); i++) {
       new_top_sites.erase(new_top_sites.begin() + *i);
     }
     SetTopSites(new_top_sites, CALL_LOCATION_FROM_OTHER_PLACES);
   }
   StartQueryForMostVisited();
 }
 
 }  // namespace history