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

Unified Diff: sync/engine/sync_scheduler_whitebox_unittest.cc

Issue 13422003: sync: Refactor job ownership in SyncScheduler (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed upload Created 7 years, 8 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: sync/engine/sync_scheduler_whitebox_unittest.cc
diff --git a/sync/engine/sync_scheduler_whitebox_unittest.cc b/sync/engine/sync_scheduler_whitebox_unittest.cc
index 9aceadab965e7854da868417295c70ffc79937f0..7a550fa67986ab540d83aecbf25ebfab32a2ae1a 100644
--- a/sync/engine/sync_scheduler_whitebox_unittest.cc
+++ b/sync/engine/sync_scheduler_whitebox_unittest.cc
@@ -88,10 +88,6 @@ class SyncSchedulerWhiteboxTest : public testing::Test {
TimeDelta::FromSeconds(1)));
}
- void SetWaitIntervalHadNudge(bool had_nudge) {
- scheduler_->wait_interval_->had_nudge = had_nudge;
- }
-
SyncSchedulerImpl::JobProcessDecision DecideOnJob(
const SyncSessionJob& job,
SyncSchedulerImpl::JobPriority priority) {
@@ -233,22 +229,10 @@ TEST_F(SyncSchedulerWhiteboxTest, SaveNudgeWhileThrottled) {
EXPECT_EQ(decision, SyncSchedulerImpl::SAVE);
}
-TEST_F(SyncSchedulerWhiteboxTest, ContinueNudgeWhileExponentialBackOff) {
- InitializeSyncerOnNormalMode();
- SetMode(SyncScheduler::NORMAL_MODE);
- SetWaitIntervalToExponentialBackoff();
-
- SyncSchedulerImpl::JobProcessDecision decision = CreateAndDecideJob(
- SyncSessionJob::NUDGE);
-
- EXPECT_EQ(decision, SyncSchedulerImpl::CONTINUE);
-}
-
TEST_F(SyncSchedulerWhiteboxTest, DropNudgeWhileExponentialBackOff) {
InitializeSyncerOnNormalMode();
SetMode(SyncScheduler::NORMAL_MODE);
SetWaitIntervalToExponentialBackoff();
- SetWaitIntervalHadNudge(true);
SyncSchedulerImpl::JobProcessDecision decision = CreateAndDecideJob(
SyncSessionJob::NUDGE);

Powered by Google App Engine
This is Rietveld 408576698