Sync user's custom spellcheck dictionary

chrome/browser/sync/profile_sync_service.cc

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 ef237c28656965dde1d0ef5710896f1f9473ba03..1d1c2e191f4cc9ce0cb93463fbb1cb124715c68c 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -1285,6 +1285,7 @@ void ProfileSyncService::UpdateSelectedTypesHistogram(
     syncer::APPS,
     syncer::AUTOFILL,
     syncer::BOOKMARKS,
+    syncer::DICTIONARY,

[Nicolas Zea, 2013/01/02 23:12:15]

Do we want Dictionary to be a separately selectable datatype? I forget if we
have discussed this already or not, but it seems to me it makes sense to tie
this to the PREFERENCES datatype, in the same way as SEARCH_ENGINES is tied to
it.

[please use gerrit instead, 2013/01/04 23:30:50]

I do not recall this discussion, but it's a good idea. I've moved DICTIONARY
under PREFERENCES.

[groby-ooo-7-16, 2013/01/07 19:02:14]

Can't speak to the pros/cons, but I'd like to point out that dictionary data is
independent of system preferences. It's user created data, not a preference.

(I'm not sure if that should influence the discussion, since I'm not familiar
enough with syncing)

On 2013/01/04 23:30:50, Rouslan Solomakhin wrote:

[Nicolas Zea, 2013/01/07 19:07:41]

The user selectable option is actually called "Settings", and it enables
PREFERENCES, and by virtue of that SEARCH ENGINES. My point is that this seems
to me like it should fall under "Settings", and therefore be auto enabled if
PREFERENCES is enabled.

This isn't meant to imply that the dictionary is implemented via preferences.

     syncer::EXTENSIONS,
     syncer::PASSWORDS,
     syncer::PREFERENCES,
@@ -1298,6 +1299,7 @@ void ProfileSyncService::UpdateSelectedTypesHistogram(
     browser_sync::user_selectable_type::APPS,
     browser_sync::user_selectable_type::AUTOFILL,
     browser_sync::user_selectable_type::BOOKMARKS,
+    browser_sync::user_selectable_type::DICTIONARY,
     browser_sync::user_selectable_type::EXTENSIONS,
     browser_sync::user_selectable_type::PASSWORDS,
     browser_sync::user_selectable_type::PREFERENCES,
@@ -1306,7 +1308,7 @@ void ProfileSyncService::UpdateSelectedTypesHistogram(
     browser_sync::user_selectable_type::TYPED_URLS
   };
 
-  COMPILE_ASSERT(20 == syncer::MODEL_TYPE_COUNT, UpdateCustomConfigHistogram);
+  COMPILE_ASSERT(21 == syncer::MODEL_TYPE_COUNT, UpdateCustomConfigHistogram);
   COMPILE_ASSERT(arraysize(model_types) ==
                  browser_sync::user_selectable_type::SELECTABLE_DATATYPE_COUNT,
                  UpdateCustomConfigHistogram);