Enable dictionary sync by default

chrome/browser/sync/profile_sync_components_factory_impl.cc

Index: chrome/browser/sync/profile_sync_components_factory_impl.cc
diff --git a/chrome/browser/sync/profile_sync_components_factory_impl.cc b/chrome/browser/sync/profile_sync_components_factory_impl.cc
index 48207449510ab599c2173ac213821aee9461f5f3..eb64a02e6ad170d63928ed1737190fd21e452b41 100644
--- a/chrome/browser/sync/profile_sync_components_factory_impl.cc
+++ b/chrome/browser/sync/profile_sync_components_factory_impl.cc
@@ -235,12 +235,13 @@ void ProfileSyncComponentsFactoryImpl::RegisterDesktopDataTypes(
             syncer::HISTORY_DELETE_DIRECTIVES, this, profile_, pss));
   }
 
-  // Dictionary sync is disabled by default.  Register only if explicitly
-  // enabled.
-  if (command_line_->HasSwitch(switches::kEnableSyncDictionary)) {
+#if !defined(OS_ANDROID) && !defined(OS_MACOSX)

[groby-ooo-7-16, 2013/02/01 22:27:56]

I'd rather explicitly enable it for a fixed set of platforms than exclude a
fixed set of platforms.

I.e. #if defined(OS_LINUX) || defined (OS_WIN) || defined(OS_CROS)

[please use gerrit instead, 2013/02/01 22:34:29]

Done.

+  // Dictionary sync is enabled by default.
+  if (!command_line_->HasSwitch(switches::kDisableSyncDictionary)) {
     pss->RegisterDataTypeController(
         new UIDataTypeController(syncer::DICTIONARY, this, profile_, pss));
   }
+#endif
 }
 
 DataTypeManager* ProfileSyncComponentsFactoryImpl::CreateDataTypeManager(