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

Side by Side Diff: components/browser_sync/profile_sync_service.cc

Issue 2701003002: [Sync] Clean up ModelType code. (Closed)
Patch Set: Switched int to size_t to fix compile. Created 3 years, 9 months 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/sync/base/enum_set.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/browser_sync/profile_sync_service.h" 5 #include "components/browser_sync/profile_sync_service.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <cstddef> 9 #include <cstddef>
10 #include <map> 10 #include <map>
(...skipping 1500 matching lines...) Expand 10 before | Expand all | Expand 10 after
1511 syncer::user_selectable_type::TYPED_URLS, 1511 syncer::user_selectable_type::TYPED_URLS,
1512 syncer::user_selectable_type::EXTENSIONS, 1512 syncer::user_selectable_type::EXTENSIONS,
1513 syncer::user_selectable_type::APPS, 1513 syncer::user_selectable_type::APPS,
1514 #if BUILDFLAG(ENABLE_READING_LIST) 1514 #if BUILDFLAG(ENABLE_READING_LIST)
1515 syncer::user_selectable_type::READING_LIST, 1515 syncer::user_selectable_type::READING_LIST,
1516 #endif 1516 #endif
1517 syncer::user_selectable_type::PROXY_TABS, 1517 syncer::user_selectable_type::PROXY_TABS,
1518 }; 1518 };
1519 1519
1520 static_assert(39 == syncer::MODEL_TYPE_COUNT, 1520 static_assert(39 == syncer::MODEL_TYPE_COUNT,
1521 "custom config histogram must be updated"); 1521 "If adding a user selectable type, update "
1522 "UserSelectableSyncType in user_selectable_sync_type.h and "
1523 "histograms.xml.");
1522 1524
1523 if (!sync_everything) { 1525 if (!sync_everything) {
1524 const syncer::ModelTypeSet current_types = GetPreferredDataTypes(); 1526 const syncer::ModelTypeSet current_types = GetPreferredDataTypes();
1525 1527
1526 syncer::ModelTypeSet type_set = syncer::UserSelectableTypes(); 1528 syncer::ModelTypeSet type_set = syncer::UserSelectableTypes();
1527 syncer::ModelTypeSet::Iterator it = type_set.First(); 1529 syncer::ModelTypeSet::Iterator it = type_set.First();
1528 1530
1529 DCHECK_EQ(arraysize(user_selectable_types), type_set.Size()); 1531 DCHECK_EQ(arraysize(user_selectable_types), type_set.Size());
1530 1532
1531 for (size_t i = 0; i < arraysize(user_selectable_types) && it.Good(); 1533 for (size_t i = 0; i < arraysize(user_selectable_types) && it.Good();
(...skipping 888 matching lines...) Expand 10 before | Expand all | Expand 10 after
2420 2422
2421 DCHECK(startup_controller_->IsSetupInProgress()); 2423 DCHECK(startup_controller_->IsSetupInProgress());
2422 startup_controller_->SetSetupInProgress(false); 2424 startup_controller_->SetSetupInProgress(false);
2423 2425
2424 if (IsEngineInitialized()) 2426 if (IsEngineInitialized())
2425 ReconfigureDatatypeManager(); 2427 ReconfigureDatatypeManager();
2426 NotifyObservers(); 2428 NotifyObservers();
2427 } 2429 }
2428 2430
2429 } // namespace browser_sync 2431 } // namespace browser_sync
OLDNEW
« no previous file with comments | « no previous file | components/sync/base/enum_set.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698