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

Unified Diff: chrome/browser/sync/profile_sync_service.cc

Issue 8851004: [Sync] Replace all instances of ModelTypeBitSet with ModelEnumSet (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/internal_api/sync_manager.cc ('k') | chrome/browser/sync/sync_prefs.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/profile_sync_service.cc
diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index f0074ff10257f8b74aacbd4b863b328bde4e0d84..d77fc6352e0cb6d1be5a1e364ef21f7d0784f275 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -354,12 +354,14 @@ bool ProfileSyncService::IsEncryptedDatatypeEnabled() const {
GetPreferredDataTypes(&preferred_types);
syncable::ModelTypeSet encrypted_types;
GetEncryptedDataTypes(&encrypted_types);
- syncable::ModelTypeBitSet preferred_types_bitset =
- syncable::ModelTypeBitSetFromSet(preferred_types);
- syncable::ModelTypeBitSet encrypted_types_bitset =
- syncable::ModelTypeBitSetFromSet(encrypted_types);
+ const syncable::ModelEnumSet preferred_types_enum_set =
+ syncable::ModelTypeSetToEnumSet(preferred_types);
+ const syncable::ModelEnumSet encrypted_types_enum_set =
+ syncable::ModelTypeSetToEnumSet(encrypted_types);
DCHECK(encrypted_types.count(syncable::PASSWORDS));
- return (preferred_types_bitset & encrypted_types_bitset).any();
+ return
+ !Intersection(preferred_types_enum_set,
+ encrypted_types_enum_set).Empty();
}
void ProfileSyncService::OnSyncConfigureDone(
« no previous file with comments | « chrome/browser/sync/internal_api/sync_manager.cc ('k') | chrome/browser/sync/sync_prefs.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698