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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/sync/engine/sync_scheduler_unittest.cc

Issue 8919021: [Sync] Rename ModelEnumSet to ModelTypeSet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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 | « chrome/browser/sync/engine/sync_scheduler.cc ('k') | chrome/browser/sync/engine/syncer_proto_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/engine/sync_scheduler_unittest.cc
diff --git a/chrome/browser/sync/engine/sync_scheduler_unittest.cc b/chrome/browser/sync/engine/sync_scheduler_unittest.cc
index 672aa63c4ab00e7c7c4d0c54c6a957ed5f37abce..b44a88c037838c32eeed855521c44e49d970d923 100644
--- a/chrome/browser/sync/engine/sync_scheduler_unittest.cc
+++ b/chrome/browser/sync/engine/sync_scheduler_unittest.cc
@@ -32,7 +32,7 @@ namespace browser_sync {
using sessions::SyncSession;
using sessions::SyncSessionContext;
using sessions::SyncSessionSnapshot;
-using syncable::ModelEnumSet;
+using syncable::ModelTypeSet;
using sync_pb::GetUpdatesCallerInfo;
class MockSyncer : public Syncer {
@@ -150,7 +150,7 @@ class SyncSchedulerTest : public testing::Test {
}
bool GetBackoffAndResetTest() {
- ModelEnumSet nudge_types;
+ ModelTypeSet nudge_types;
StartSyncScheduler(SyncScheduler::NORMAL_MODE);
RunLoop();
@@ -178,10 +178,10 @@ class SyncSchedulerTest : public testing::Test {
scheduler_->delay_provider_.reset(delay_);
}
- // Compare a ModelEnumSet to a ModelTypePayloadMap, ignoring
+ // Compare a ModelTypeSet to a ModelTypePayloadMap, ignoring
// payload values.
- bool CompareModelEnumSetToModelTypePayloadMap(
- ModelEnumSet lhs,
+ bool CompareModelTypeSetToModelTypePayloadMap(
+ ModelTypeSet lhs,
const syncable::ModelTypePayloadMap& rhs) {
size_t count = 0;
for (syncable::ModelTypePayloadMap::const_iterator i = rhs.begin();
@@ -239,7 +239,7 @@ ACTION(QuitLoopNowAction) {
// Test nudge scheduling.
TEST_F(SyncSchedulerTest, Nudge) {
SyncShareRecords records;
- ModelEnumSet model_types(syncable::BOOKMARKS);
+ ModelTypeSet model_types(syncable::BOOKMARKS);
EXPECT_CALL(*syncer(), SyncShare(_,_,_))
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess),
@@ -254,7 +254,7 @@ TEST_F(SyncSchedulerTest, Nudge) {
RunLoop();
ASSERT_EQ(1U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records.snapshots[0]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::LOCAL,
records.snapshots[0]->source.updates_source);
@@ -273,7 +273,7 @@ TEST_F(SyncSchedulerTest, Nudge) {
RunLoop();
ASSERT_EQ(1U, records2.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records2.snapshots[0]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::LOCAL,
records2.snapshots[0]->source.updates_source);
@@ -283,7 +283,7 @@ TEST_F(SyncSchedulerTest, Nudge) {
// errors.
TEST_F(SyncSchedulerTest, Config) {
SyncShareRecords records;
- const ModelEnumSet model_types(syncable::BOOKMARKS);
+ const ModelTypeSet model_types(syncable::BOOKMARKS);
EXPECT_CALL(*syncer(), SyncShare(_,_,_))
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess),
@@ -297,7 +297,7 @@ TEST_F(SyncSchedulerTest, Config) {
RunLoop();
ASSERT_EQ(1U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records.snapshots[0]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::RECONFIGURATION,
records.snapshots[0]->source.updates_source);
@@ -309,7 +309,7 @@ TEST_F(SyncSchedulerTest, ConfigWithBackingOff) {
EXPECT_CALL(*delay(), GetDelay(_))
.WillRepeatedly(Return(TimeDelta::FromMilliseconds(1)));
SyncShareRecords records;
- const ModelEnumSet model_types(syncable::BOOKMARKS);
+ const ModelTypeSet model_types(syncable::BOOKMARKS);
EXPECT_CALL(*syncer(), SyncShare(_,_,_))
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateCommitFailed),
@@ -329,7 +329,7 @@ TEST_F(SyncSchedulerTest, ConfigWithBackingOff) {
RunLoop();
ASSERT_EQ(2U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records.snapshots[1]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::SYNC_CYCLE_CONTINUATION,
records.snapshots[1]->source.updates_source);
@@ -338,7 +338,7 @@ TEST_F(SyncSchedulerTest, ConfigWithBackingOff) {
// Issue 2 config commands. Second one right after the first has failed
// and make sure LATEST is executed.
TEST_F(SyncSchedulerTest, MultipleConfigWithBackingOff) {
- const ModelEnumSet
+ const ModelTypeSet
model_types1(syncable::BOOKMARKS),
model_types2(syncable::AUTOFILL);
UseMockDelayProvider();
@@ -371,7 +371,7 @@ TEST_F(SyncSchedulerTest, MultipleConfigWithBackingOff) {
RunLoop();
ASSERT_EQ(3U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types2,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types2,
records.snapshots[2]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::RECONFIGURATION,
records.snapshots[2]->source.updates_source);
@@ -380,7 +380,7 @@ TEST_F(SyncSchedulerTest, MultipleConfigWithBackingOff) {
// Issue a nudge when the config has failed. Make sure both the config and
// nudge are executed.
TEST_F(SyncSchedulerTest, NudgeWithConfigWithBackingOff) {
- const ModelEnumSet model_types(syncable::BOOKMARKS);
+ const ModelTypeSet model_types(syncable::BOOKMARKS);
UseMockDelayProvider();
EXPECT_CALL(*delay(), GetDelay(_))
.WillRepeatedly(Return(TimeDelta::FromMilliseconds(50)));
@@ -419,12 +419,12 @@ TEST_F(SyncSchedulerTest, NudgeWithConfigWithBackingOff) {
ASSERT_EQ(4U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records.snapshots[2]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::SYNC_CYCLE_CONTINUATION,
records.snapshots[2]->source.updates_source);
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(model_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(model_types,
records.snapshots[3]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::LOCAL,
records.snapshots[3]->source.updates_source);
@@ -440,7 +440,7 @@ TEST_F(SyncSchedulerTest, NudgeCoalescing) {
EXPECT_CALL(*syncer(), SyncShare(_,_,_))
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess),
WithArg<0>(RecordSyncShare(&r))));
- const ModelEnumSet
+ const ModelTypeSet
types1(syncable::BOOKMARKS),
types2(syncable::AUTOFILL),
types3(syncable::THEMES);
@@ -454,7 +454,7 @@ TEST_F(SyncSchedulerTest, NudgeCoalescing) {
ASSERT_EQ(1U, r.snapshots.size());
EXPECT_GE(r.times[0], optimal_time);
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(
Union(types1, types2), r.snapshots[0]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::LOCAL,
r.snapshots[0]->source.updates_source);
@@ -470,7 +470,7 @@ TEST_F(SyncSchedulerTest, NudgeCoalescing) {
RunLoop();
ASSERT_EQ(1U, r2.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(types3,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(types3,
r2.snapshots[0]->source.types));
EXPECT_EQ(GetUpdatesCallerInfo::NOTIFICATION,
r2.snapshots[0]->source.updates_source);
@@ -652,7 +652,7 @@ TEST_F(SyncSchedulerTest, SessionsCommitDelay) {
RunLoop();
EXPECT_EQ(delay1, scheduler()->sessions_commit_delay());
- const ModelEnumSet model_types(syncable::BOOKMARKS);
+ const ModelTypeSet model_types(syncable::BOOKMARKS);
scheduler()->ScheduleNudge(
zero(), NUDGE_SOURCE_LOCAL, model_types, FROM_HERE);
RunLoop();
@@ -671,7 +671,7 @@ TEST_F(SyncSchedulerTest, HasMoreToSync) {
RunLoop();
scheduler()->ScheduleNudge(
- zero(), NUDGE_SOURCE_LOCAL, ModelEnumSet(), FROM_HERE);
+ zero(), NUDGE_SOURCE_LOCAL, ModelTypeSet(), FROM_HERE);
RunLoop();
// If more nudges are scheduled, they'll be waited on by TearDown, and would
// cause our expectation to break.
@@ -679,7 +679,7 @@ TEST_F(SyncSchedulerTest, HasMoreToSync) {
// Test that no syncing occurs when throttled.
TEST_F(SyncSchedulerTest, ThrottlingDoesThrottle) {
- const ModelEnumSet types(syncable::BOOKMARKS);
+ const ModelTypeSet types(syncable::BOOKMARKS);
TimeDelta poll(TimeDelta::FromMilliseconds(5));
TimeDelta throttle(TimeDelta::FromMinutes(10));
scheduler()->OnReceivedLongPollIntervalUpdate(poll);
@@ -740,20 +740,20 @@ TEST_F(SyncSchedulerTest, ConfigurationMode) {
StartSyncScheduler(SyncScheduler::CONFIGURATION_MODE);
RunLoop();
- const ModelEnumSet nudge_types(syncable::AUTOFILL);
+ const ModelTypeSet nudge_types(syncable::AUTOFILL);
scheduler()->ScheduleNudge(
zero(), NUDGE_SOURCE_LOCAL, nudge_types, FROM_HERE);
scheduler()->ScheduleNudge(
zero(), NUDGE_SOURCE_LOCAL, nudge_types, FROM_HERE);
- const ModelEnumSet config_types(syncable::BOOKMARKS);
+ const ModelTypeSet config_types(syncable::BOOKMARKS);
scheduler()->ScheduleConfig(
config_types, GetUpdatesCallerInfo::RECONFIGURATION);
RunLoop();
ASSERT_EQ(1U, records.snapshots.size());
- EXPECT_TRUE(CompareModelEnumSetToModelTypePayloadMap(config_types,
+ EXPECT_TRUE(CompareModelTypeSetToModelTypePayloadMap(config_types,
records.snapshots[0]->source.types));
}
@@ -801,7 +801,7 @@ TEST_F(SyncSchedulerTest, BackoffTriggers) {
TEST_F(SyncSchedulerTest, BackoffDropsJobs) {
SyncShareRecords r;
TimeDelta poll(TimeDelta::FromMilliseconds(5));
- const ModelEnumSet types(syncable::BOOKMARKS);
+ const ModelTypeSet types(syncable::BOOKMARKS);
scheduler()->OnReceivedLongPollIntervalUpdate(poll);
UseMockDelayProvider();
@@ -965,7 +965,7 @@ TEST_F(SyncSchedulerTest, SyncerSteps) {
RunLoop();
scheduler()->ScheduleNudge(
- zero(), NUDGE_SOURCE_LOCAL, ModelEnumSet(), FROM_HERE);
+ zero(), NUDGE_SOURCE_LOCAL, ModelTypeSet(), FROM_HERE);
PumpLoop();
// Pump again to run job.
PumpLoop();
@@ -992,7 +992,7 @@ TEST_F(SyncSchedulerTest, SyncerSteps) {
RunLoop();
scheduler()->ScheduleConfig(
- ModelEnumSet(), GetUpdatesCallerInfo::RECONFIGURATION);
+ ModelTypeSet(), GetUpdatesCallerInfo::RECONFIGURATION);
PumpLoop();
PumpLoop();
@@ -1044,7 +1044,7 @@ TEST_F(SyncSchedulerTest, StartWhenNotConnected) {
RunLoop();
scheduler()->ScheduleNudge(
- zero(), NUDGE_SOURCE_LOCAL, ModelEnumSet(), FROM_HERE);
+ zero(), NUDGE_SOURCE_LOCAL, ModelTypeSet(), FROM_HERE);
// Should save the nudge for until after the server is reachable.
PumpLoop();
@@ -1064,7 +1064,7 @@ TEST_F(SyncSchedulerTest, SetsPreviousRoutingInfo) {
RunLoop();
scheduler()->ScheduleNudge(
- zero(), NUDGE_SOURCE_LOCAL, ModelEnumSet(), FROM_HERE);
+ zero(), NUDGE_SOURCE_LOCAL, ModelTypeSet(), FROM_HERE);
PumpLoop();
// Pump again to run job.
PumpLoop();
« no previous file with comments | « chrome/browser/sync/engine/sync_scheduler.cc ('k') | chrome/browser/sync/engine/syncer_proto_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698