Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(167)

Unified Diff: components/offline_pages/core/background/pick_request_task_unittest.cc

Issue 2776203003: Revert of [Offline Pages] Add Android version check for timeout in offliner policy. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/core/background/pick_request_task_unittest.cc
diff --git a/components/offline_pages/core/background/pick_request_task_unittest.cc b/components/offline_pages/core/background/pick_request_task_unittest.cc
index 2cae77bd3e0cb94477b7cf5fe31852301375112b..7f68624655161a62bfdd135646afb28cee476a48 100644
--- a/components/offline_pages/core/background/pick_request_task_unittest.cc
+++ b/components/offline_pages/core/background/pick_request_task_unittest.cc
@@ -42,6 +42,7 @@
const bool kPreferUntried = false;
const bool kPreferEarlier = true;
const bool kPreferRetryCount = true;
+const int kBackgroundProcessingTimeBudgetSeconds = 170;
// Default request
const SavePageRequest kEmptyRequest(0UL,
@@ -244,9 +245,9 @@
TEST_F(PickRequestTaskTest, ChooseRequestWithHigherRetryCount) {
// Set up policy to prefer higher retry count.
- policy_.reset(new OfflinerPolicy(kPreferUntried, kPreferEarlier,
- kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries + 1));
+ policy_.reset(new OfflinerPolicy(
+ kPreferUntried, kPreferEarlier, kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries + 1, kBackgroundProcessingTimeBudgetSeconds));
MakePickRequestTask();
base::Time creation_time = base::Time::Now();
@@ -289,9 +290,9 @@
TEST_F(PickRequestTaskTest, ChooseEarlierRequest) {
// We need a custom policy object prefering recency to retry count.
- policy_.reset(new OfflinerPolicy(kPreferUntried, kPreferEarlier,
- !kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries));
+ policy_.reset(new OfflinerPolicy(
+ kPreferUntried, kPreferEarlier, !kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries, kBackgroundProcessingTimeBudgetSeconds));
MakePickRequestTask();
base::Time creation_time1 =
@@ -315,9 +316,9 @@
TEST_F(PickRequestTaskTest, ChooseSameTimeRequestWithHigherRetryCount) {
// We need a custom policy object preferring recency to retry count.
- policy_.reset(new OfflinerPolicy(kPreferUntried, kPreferEarlier,
- !kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries + 1));
+ policy_.reset(new OfflinerPolicy(
+ kPreferUntried, kPreferEarlier, !kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries + 1, kBackgroundProcessingTimeBudgetSeconds));
MakePickRequestTask();
base::Time creation_time = base::Time::Now();
@@ -339,9 +340,9 @@
TEST_F(PickRequestTaskTest, ChooseRequestWithLowerRetryCount) {
// We need a custom policy object preferring lower retry count.
- policy_.reset(new OfflinerPolicy(!kPreferUntried, kPreferEarlier,
- kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries + 1));
+ policy_.reset(new OfflinerPolicy(
+ !kPreferUntried, kPreferEarlier, kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries + 1, kBackgroundProcessingTimeBudgetSeconds));
MakePickRequestTask();
base::Time creation_time = base::Time::Now();
@@ -363,9 +364,9 @@
TEST_F(PickRequestTaskTest, ChooseLaterRequest) {
// We need a custom policy preferring recency over retry, and later requests.
- policy_.reset(new OfflinerPolicy(kPreferUntried, !kPreferEarlier,
- !kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries));
+ policy_.reset(new OfflinerPolicy(
+ kPreferUntried, !kPreferEarlier, !kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries, kBackgroundProcessingTimeBudgetSeconds));
MakePickRequestTask();
base::Time creation_time1 =
@@ -460,9 +461,9 @@
}
TEST_F(PickRequestTaskTest, ChooseRequestThatIsNotDisabled) {
- policy_.reset(new OfflinerPolicy(kPreferUntried, kPreferEarlier,
- kPreferRetryCount, kMaxStartedTries,
- kMaxCompletedTries + 1));
+ policy_.reset(new OfflinerPolicy(
+ kPreferUntried, kPreferEarlier, kPreferRetryCount, kMaxStartedTries,
+ kMaxCompletedTries + 1, kBackgroundProcessingTimeBudgetSeconds));
// put request 2 on disabled list, ensure request1 picked instead,
// even though policy would prefer 2.

Powered by Google App Engine
This is Rietveld 408576698