summaryrefslogtreecommitdiffstats
path: root/components/sync_driver/sync_prefs.cc
diff options
context:
space:
mode:
authorzea <zea@chromium.org>2015-07-10 15:11:02 -0700
committerCommit bot <commit-bot@chromium.org>2015-07-10 22:11:34 +0000
commit2b0454e851d76ab224b68df692855da75e08c534 (patch)
tree0a2b2975b67e5d1b96660589535226ea4f7f21bb /components/sync_driver/sync_prefs.cc
parent285317b72701f6f5581dfc5a214b3cb5c3ef06b3 (diff)
downloadchromium_src-2b0454e851d76ab224b68df692855da75e08c534.zip
chromium_src-2b0454e851d76ab224b68df692855da75e08c534.tar.gz
chromium_src-2b0454e851d76ab224b68df692855da75e08c534.tar.bz2
[Sync] Add priority preferences datatype for mobile
This also adds support for the normal preferences datatype because that's the one hardcoded as a user selectable type, but because it doesn't have a datatype controller on mobile, it will never be turned on (and it therefore won't be preferred, so we won't receive invalidations for it). BUG=505542 Review URL: https://codereview.chromium.org/1222203008 Cr-Commit-Position: refs/heads/master@{#338379}
Diffstat (limited to 'components/sync_driver/sync_prefs.cc')
-rw-r--r--components/sync_driver/sync_prefs.cc3
1 files changed, 1 insertions, 2 deletions
diff --git a/components/sync_driver/sync_prefs.cc b/components/sync_driver/sync_prefs.cc
index 03fb65a..def047f 100644
--- a/components/sync_driver/sync_prefs.cc
+++ b/components/sync_driver/sync_prefs.cc
@@ -200,8 +200,8 @@ syncer::ModelTypeSet SyncPrefs::GetPreferredDataTypes(
void SyncPrefs::SetPreferredDataTypes(syncer::ModelTypeSet registered_types,
syncer::ModelTypeSet preferred_types) {
DCHECK(CalledOnValidThread());
- DCHECK(registered_types.HasAll(preferred_types));
preferred_types = ResolvePrefGroups(registered_types, preferred_types);
+ DCHECK(registered_types.HasAll(preferred_types));
for (syncer::ModelTypeSet::Iterator i = registered_types.First(); i.Good();
i.Inc()) {
SetDataTypePreferred(i.Get(), preferred_types.Has(i.Get()));
@@ -432,7 +432,6 @@ void SyncPrefs::SetDataTypePreferred(syncer::ModelType type,
syncer::ModelTypeSet SyncPrefs::ResolvePrefGroups(
syncer::ModelTypeSet registered_types,
syncer::ModelTypeSet types) const {
- DCHECK(registered_types.HasAll(types));
syncer::ModelTypeSet types_with_groups = types;
for (PrefGroupsMap::const_iterator i = pref_groups_.begin();
i != pref_groups_.end();