Enable App List Sync by default

chrome/browser/about_flags.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/about_flags.h"
 
 #include <algorithm>
 #include <iterator>
 #include <map>
 #include <set>
@@ skipping 1292 matching lines @@
   {
     "enable-sync-synced-notifications",
     IDS_FLAGS_ENABLE_SYNCED_NOTIFICATIONS_NAME,
     IDS_FLAGS_ENABLE_SYNCED_NOTIFICATIONS_DESCRIPTION,
     kOsDesktop,
     ENABLE_DISABLE_VALUE_TYPE(switches::kEnableSyncSyncedNotifications,
                               switches::kDisableSyncSyncedNotifications)
   },
 #if defined(ENABLE_APP_LIST)
   {
     "enable-sync-app-list",

[tapted, 2014/03/05 22:33:38]

Pretty sure this needs to change too. The comment about it ("you shouldn't
change the name of existing flags") is a bit misleading -- I think the result
would be that any early-adopters would become auto-disabled when this blips out.

[jennyz, 2014/03/05 22:39:20]

+1

[stevenjb, 2014/03/05 23:20:00]

DOh. Thanks, you're right.
Done.

-    IDS_FLAGS_ENABLE_SYNC_APP_LIST_NAME,
-    IDS_FLAGS_ENABLE_SYNC_APP_LIST_DESCRIPTION,
+    IDS_FLAGS_DISABLE_SYNC_APP_LIST_NAME,
+    IDS_FLAGS_DISABLE_SYNC_APP_LIST_DESCRIPTION,
     kOsDesktop,
-    SINGLE_VALUE_TYPE(switches::kEnableSyncAppList)
+    SINGLE_VALUE_TYPE(switches::kDisableSyncAppList)
   },
 #endif
   {
     "disable-full-history-sync",
     IDS_FLAGS_FULL_HISTORY_SYNC_NAME,
     IDS_FLAGS_FULL_HISTORY_SYNC_DESCRIPTION,
     kOsAll,
     SINGLE_VALUE_TYPE(switches::kHistoryDisableFullHistorySync)
   },
   {
@@ skipping 1092 matching lines @@
 }
 
 const Experiment* GetExperiments(size_t* count) {
   *count = num_experiments;
   return experiments;
 }
 
 }  // namespace testing
 
 }  // namespace about_flags