Check time budget before starting new requests, and unittest.

components/offline_pages/background/offliner_policy.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_POLICY_H_
 #define COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_POLICY_H_
 
 namespace {
 const int kMaxRetries = 2;
-const int kBackgroundTimeBudgetSeconds = 170;
-const int kSinglePageTimeBudgetSeconds = 120;
+const int kBackgroundProcessingTimeBudgetSeconds = 170;
+const int kSinglePageTimeLimitSeconds = 120;
 const int kMinimumBatteryPercentageForNonUserRequestOfflining = 50;
 }  // namespace
 
 namespace offline_pages {
 
 // Policy for the Background Offlining system.  Some policy will belong to the
 // RequestCoordinator, some to the RequestQueue, and some to the Offliner.
 class OfflinerPolicy {
  public:
   OfflinerPolicy()
@@ skipping 39 matching lines @@
   int BatteryPercentageRequiredForUserRequestedPage() const { return 50; }
 
   // This is so low because we require the device to be plugged in and charging.
   // If we decide to allow non-user requested pages when not plugged in, we
   // should raise this somewhat higher.
   int BatteryPercentageRequiredForNonUserRequestedPage() const { return 25; }
 
   // How many seconds to keep trying new pages for, before we give up,  and
   // return to the scheduler.
   int GetBackgroundProcessingTimeBudgetSeconds() const {
-    return kBackgroundTimeBudgetSeconds;
+    return kBackgroundProcessingTimeBudgetSeconds;
   }
 
   // How long do we allow a page to load before giving up on it
-  int GetSinglePageTimeBudgetInSeconds() const {
-    return kSinglePageTimeBudgetSeconds;
+  int GetSinglePageTimeLimitInSeconds() const {
+    return kSinglePageTimeLimitSeconds;
   }
 
   // How much battery must we have before fetching a page not explicitly
   // requested by the user?
   int GetMinimumBatteryPercentageForNonUserRequestOfflining() const {
     return kMinimumBatteryPercentageForNonUserRequestOfflining;
   }
 
  private:
   bool prefer_untried_requests_;
   bool prefer_earlier_requests_;
   bool retry_count_is_more_important_than_recency_;
 };
 }
 
 #endif  // COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_POLICY_H_