| Index: chrome/browser/sync/sessions/sync_session_unittest.cc
|
| diff --git a/chrome/browser/sync/sessions/sync_session_unittest.cc b/chrome/browser/sync/sessions/sync_session_unittest.cc
|
| index 64c2cec64d5f423607a1c801052b87cfc0a431f0..d78ab4c521bc9160e232df62a0fcc0397d5a1704 100644
|
| --- a/chrome/browser/sync/sessions/sync_session_unittest.cc
|
| +++ b/chrome/browser/sync/sessions/sync_session_unittest.cc
|
| @@ -108,17 +108,14 @@ class SyncSessionTest : public testing::Test,
|
| FAIL() << msg;
|
| }
|
|
|
| - syncable::ModelTypeBitSet ParamsMeaningAllEnabledTypes() {
|
| - syncable::ModelTypeBitSet request_params;
|
| - request_params[syncable::BOOKMARKS] = true;
|
| - request_params[syncable::AUTOFILL] = true;
|
| + syncable::ModelEnumSet ParamsMeaningAllEnabledTypes() {
|
| + syncable::ModelEnumSet request_params(
|
| + syncable::BOOKMARKS, syncable::AUTOFILL);
|
| return request_params;
|
| }
|
|
|
| - syncable::ModelTypeBitSet ParamsMeaningJustOneEnabledType() {
|
| - syncable::ModelTypeBitSet request_params;
|
| - request_params[syncable::AUTOFILL] = true;
|
| - return request_params;
|
| + syncable::ModelEnumSet ParamsMeaningJustOneEnabledType() {
|
| + return syncable::ModelEnumSet(syncable::AUTOFILL);
|
| }
|
|
|
| MessageLoop message_loop_;
|
| @@ -337,11 +334,11 @@ TEST_F(SyncSessionTest, Coalesce) {
|
| std::vector<ModelSafeWorker*> workers_one, workers_two;
|
| ModelSafeRoutingInfo routes_one, routes_two;
|
| syncable::ModelTypePayloadMap one_type =
|
| - syncable::ModelTypePayloadMapFromBitSet(
|
| + syncable::ModelTypePayloadMapFromEnumSet(
|
| ParamsMeaningJustOneEnabledType(),
|
| std::string());
|
| syncable::ModelTypePayloadMap all_types =
|
| - syncable::ModelTypePayloadMapFromBitSet(
|
| + syncable::ModelTypePayloadMapFromEnumSet(
|
| ParamsMeaningAllEnabledTypes(),
|
| std::string());
|
| SyncSourceInfo source_one(sync_pb::GetUpdatesCallerInfo::PERIODIC, one_type);
|
| @@ -394,11 +391,11 @@ TEST_F(SyncSessionTest, RebaseRoutingInfoWithLatestRemoveOneType) {
|
| std::vector<ModelSafeWorker*> workers_one, workers_two;
|
| ModelSafeRoutingInfo routes_one, routes_two;
|
| syncable::ModelTypePayloadMap one_type =
|
| - syncable::ModelTypePayloadMapFromBitSet(
|
| + syncable::ModelTypePayloadMapFromEnumSet(
|
| ParamsMeaningJustOneEnabledType(),
|
| std::string());
|
| syncable::ModelTypePayloadMap all_types =
|
| - syncable::ModelTypePayloadMapFromBitSet(
|
| + syncable::ModelTypePayloadMapFromEnumSet(
|
| ParamsMeaningAllEnabledTypes(),
|
| std::string());
|
| SyncSourceInfo source_one(sync_pb::GetUpdatesCallerInfo::PERIODIC, one_type);
|
| @@ -464,7 +461,7 @@ TEST_F(SyncSessionTest, RebaseRoutingInfoWithLatestWithSameType) {
|
| std::vector<ModelSafeWorker*> workers_first, workers_second;
|
| ModelSafeRoutingInfo routes_first, routes_second;
|
| syncable::ModelTypePayloadMap all_types =
|
| - syncable::ModelTypePayloadMapFromBitSet(
|
| + syncable::ModelTypePayloadMapFromEnumSet(
|
| ParamsMeaningAllEnabledTypes(),
|
| std::string());
|
| SyncSourceInfo source_first(sync_pb::GetUpdatesCallerInfo::PERIODIC,
|
| @@ -540,18 +537,18 @@ TEST_F(SyncSessionTest, RebaseRoutingInfoWithLatestWithSameType) {
|
|
|
|
|
| TEST_F(SyncSessionTest, MakeTypePayloadMapFromBitSet) {
|
| - syncable::ModelTypeBitSet types;
|
| + syncable::ModelEnumSet types;
|
| std::string payload = "test";
|
| syncable::ModelTypePayloadMap types_with_payloads =
|
| - syncable::ModelTypePayloadMapFromBitSet(types,
|
| - payload);
|
| + syncable::ModelTypePayloadMapFromEnumSet(types, payload);
|
| EXPECT_TRUE(types_with_payloads.empty());
|
|
|
| - types[syncable::BOOKMARKS] = true;
|
| - types[syncable::PASSWORDS] = true;
|
| - types[syncable::AUTOFILL] = true;
|
| + types.Put(syncable::BOOKMARKS);
|
| + types.Put(syncable::PASSWORDS);
|
| + types.Put(syncable::AUTOFILL);
|
| payload = "test2";
|
| - types_with_payloads = syncable::ModelTypePayloadMapFromBitSet(types, payload);
|
| + types_with_payloads =
|
| + syncable::ModelTypePayloadMapFromEnumSet(types, payload);
|
|
|
| ASSERT_EQ(3U, types_with_payloads.size());
|
| EXPECT_EQ(types_with_payloads[syncable::BOOKMARKS], payload);
|
|
|