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

Unified Diff: sync/engine/sync_scheduler_whitebox_unittest.cc

Issue 12317104: Remove canary member from SyncSessionJob (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 10 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
« no previous file with comments | « sync/engine/sync_scheduler_impl.cc ('k') | sync/engine/sync_session_job.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 335caff12f518bcc5aa0c5fec4f676db24fe00c8..01adb85ee79235a2e179980e6fef2fdf583cd63f 100644
--- a/sync/engine/sync_scheduler_whitebox_unittest.cc
+++ b/sync/engine/sync_scheduler_whitebox_unittest.cc
@@ -93,8 +93,9 @@ class SyncSchedulerWhiteboxTest : public testing::Test {
}
SyncSchedulerImpl::JobProcessDecision DecideOnJob(
- const SyncSessionJob& job) {
- return scheduler_->DecideOnJob(job);
+ const SyncSessionJob& job,
+ SyncSchedulerImpl::JobPriority priority) {
+ return scheduler_->DecideOnJob(job, priority);
}
void InitializeSyncerOnNormalMode() {
@@ -107,7 +108,7 @@ class SyncSchedulerWhiteboxTest : public testing::Test {
scoped_ptr<SyncSession> s(scheduler_->CreateSyncSession(SyncSourceInfo()));
SyncSessionJob job(purpose, TimeTicks::Now(), s.Pass(),
ConfigurationParams());
- return DecideOnJob(job);
+ return DecideOnJob(job, SyncSchedulerImpl::NORMAL_PRIORITY);
}
SyncSessionContext* context() { return context_.get(); }
@@ -158,7 +159,8 @@ TEST_F(SyncSchedulerWhiteboxTest, SaveNudgeWhileTypeThrottled) {
TimeTicks::Now(),
s.Pass(),
ConfigurationParams());
- SyncSchedulerImpl::JobProcessDecision decision = DecideOnJob(job);
+ SyncSchedulerImpl::JobProcessDecision decision =
+ DecideOnJob(job, SyncSchedulerImpl::NORMAL_PRIORITY);
// TODO(tim): This shouldn't drop. Bug 177659.
EXPECT_EQ(decision, SyncSchedulerImpl::DROP);
}
@@ -257,8 +259,8 @@ TEST_F(SyncSchedulerWhiteboxTest, ContinueCanaryJobConfig) {
TimeTicks::Now(), scoped_ptr<SyncSession>(),
ConfigurationParams());
- job.GrantCanaryPrivilege();
- SyncSchedulerImpl::JobProcessDecision decision = DecideOnJob(job);
+ SyncSchedulerImpl::JobProcessDecision decision =
+ DecideOnJob(job, SyncSchedulerImpl::CANARY_PRIORITY);
EXPECT_EQ(decision, SyncSchedulerImpl::CONTINUE);
}
« no previous file with comments | « sync/engine/sync_scheduler_impl.cc ('k') | sync/engine/sync_session_job.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698