[Offline Pages] Add Android version check for timeout in offliner policy.

components/offline_pages/core/background/request_coordinator_unittest.cc

 // 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.
 
 #include "components/offline_pages/core/background/request_coordinator.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 28 matching lines @@
 // put test constants here
 const GURL kUrl1("http://universe.com/everything");
 const GURL kUrl2("http://universe.com/toinfinityandbeyond");
 const std::string kClientNamespace("bookmark");
 const std::string kId1("42");
 const std::string kId2("life*universe+everything");
 const ClientId kClientId1(kClientNamespace, kId1);
 const ClientId kClientId2(kClientNamespace, kId2);
 const int kRequestId1(1);
 const int kRequestId2(2);
-const long kTestTimeBudgetSeconds = 200;
 const int kBatteryPercentageHigh = 75;
 const int kMaxCompletedTries = 3;
 const bool kPowerRequired = true;
 const bool kUserRequested = true;
 const int kAttemptCount = 1;
 }  // namespace
 
 class ObserverStub : public RequestCoordinator::Observer {
  public:
   ObserverStub()
@@ skipping 1162 matching lines @@
   PumpLoop();
 
   // Sending the request to the offliner.
   EXPECT_TRUE(coordinator()->StartScheduledProcessing(device_conditions(),
                                                       waiting_callback()));
   PumpLoop();
 
   // Advance the mock clock far enough to exceed our time budget.
   // The first request will time out, and because we are over time budget,
   // the second request will not be started.
-  AdvanceClockBy(base::TimeDelta::FromSeconds(kTestTimeBudgetSeconds));
+  int over_time_budget_seconds =
+      OfflinerPolicy()
+          .GetProcessingTimeBudgetWhenBackgroundScheduledInSeconds() +
+      10;
+  AdvanceClockBy(base::TimeDelta::FromSeconds(over_time_budget_seconds));
   PumpLoop();
 
   // TryNextRequest should decide that there is no more work to be done,
   // and call back to the scheduler, even though there is another request in the
   // queue.  Both requests should be left in the queue.
   coordinator()->queue()->GetRequests(base::Bind(
       &RequestCoordinatorTest::GetRequestsDone, base::Unretained(this)));
   PumpLoop();
 
   // We should find two requests in the queue.
@@ skipping 336 matching lines @@
   PumpLoop();
 
   // The last time would trigger the snapshot on last retry and succeed.
   EXPECT_FALSE(OfflinerWasCanceled());
   EXPECT_EQ(RequestCoordinator::BackgroundSavePageResult::SUCCESS,
             observer().last_status());
   EXPECT_TRUE(observer().completed_called());
 }
 
 }  // namespace offline_pages