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

Unified Diff: chrome/browser/sync/syncable/model_type_payload_map.cc

Issue 8772074: [Sync] Convert syncable/ directory to ModelEnumSet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo 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
Index: chrome/browser/sync/syncable/model_type_payload_map.cc
diff --git a/chrome/browser/sync/syncable/model_type_payload_map.cc b/chrome/browser/sync/syncable/model_type_payload_map.cc
index ea87c52af234c6b2db4e315f30bcf1aed0b54421..737dd21f9639ad527f340aa7dd5864315398a45a 100644
--- a/chrome/browser/sync/syncable/model_type_payload_map.cc
+++ b/chrome/browser/sync/syncable/model_type_payload_map.cc
@@ -13,24 +13,23 @@
using browser_sync::ModelSafeRoutingInfo;
namespace syncable {
-ModelTypePayloadMap ModelTypePayloadMapFromBitSet(
- const syncable::ModelTypeBitSet& types,
+ModelTypePayloadMap ModelTypePayloadMapFromEnumSet(
+ syncable::ModelEnumSet types,
const std::string& payload) {
ModelTypePayloadMap types_with_payloads;
- for (size_t i = syncable::FIRST_REAL_MODEL_TYPE;
- i < types.size(); ++i) {
- if (types[i]) {
- types_with_payloads[syncable::ModelTypeFromInt(i)] = payload;
- }
+ for (syncable::ModelEnumSet::Iterator it = types.First();
+ it.Good(); it.Inc()) {
+ types_with_payloads[it.Get()] = payload;
}
return types_with_payloads;
}
-ModelTypeSet ModelTypePayloadMapToSet(const ModelTypePayloadMap& payload_map) {
- ModelTypeSet types;
+ModelEnumSet ModelTypePayloadMapToEnumSet(
+ const ModelTypePayloadMap& payload_map) {
+ ModelEnumSet types;
for (ModelTypePayloadMap::const_iterator it = payload_map.begin();
it != payload_map.end(); ++it) {
- types.insert(it->first);
+ types.Put(it->first);
}
return types;
}
« no previous file with comments | « chrome/browser/sync/syncable/model_type_payload_map.h ('k') | chrome/browser/sync/syncable/model_type_payload_map_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698