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

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

Issue 8851006: [Sync] Replace all instances of ModelTypeSet with ModelEnumSet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup 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/engine/syncer_proto_util.cc
diff --git a/chrome/browser/sync/engine/syncer_proto_util.cc b/chrome/browser/sync/engine/syncer_proto_util.cc
index ca782f326e6498a2cb02dca26c0b82fdf74baae0..e7f6ff32b09ff9140219189fb80dbc3592e1d924 100644
--- a/chrome/browser/sync/engine/syncer_proto_util.cc
+++ b/chrome/browser/sync/engine/syncer_proto_util.cc
@@ -194,7 +194,7 @@ void SyncerProtoUtil::HandleThrottleError(
sessions::SyncSessionContext* context,
sessions::SyncSession::Delegate* delegate) {
DCHECK_EQ(error.error_type, browser_sync::THROTTLED);
- if (error.error_data_types.size() > 0) {
+ if (error.error_data_types.Size() > 0) {
Nicolas Zea 2011/12/08 01:49:37 !Empty()
akalin 2011/12/09 19:10:10 Done.
context->SetUnthrottleTime(error.error_data_types, throttled_until);
} else {
// No datatypes indicates the client should be completely throttled.
@@ -278,7 +278,7 @@ browser_sync::SyncProtocolError ConvertErrorPBToLocalType(
// THROTTLED is currently the only error code that uses |error_data_types|.
DCHECK_EQ(error.error_type(), ClientToServerResponse::THROTTLED);
for (int i = 0; i < error.error_data_type_ids_size(); ++i) {
- sync_protocol_error.error_data_types.insert(
+ sync_protocol_error.error_data_types.Put(
syncable::GetModelTypeFromExtensionFieldNumber(
error.error_data_type_ids(i)));
}

Powered by Google App Engine
This is Rietveld 408576698