Fix sync for reading list

components/sync/syncable/model_type.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 "components/sync/base/model_type.h"
 
 #include <stddef.h>
 
 #include "base/macros.h"
 #include "base/strings/string_split.h"
@@ skipping 135 matching lines @@
 };
 
 static_assert(arraysize(kModelTypeInfoMap) == MODEL_TYPE_COUNT,
               "kModelTypeInfoMap should have MODEL_TYPE_COUNT elements");
 
 // Notes:
 // 1) This list must contain exactly the same elements as the set returned by
 //    UserSelectableTypes().
 // 2) This list must be in the same order as the respective values in the
 //    ModelType enum.
 const char* kUserSelectableDataTypeNames[] = {

[pavely, 2016/12/05 18:48:42]

Please add reading list name in this list as well. Value should correspond to
the one used in javascript:
https://cs.chromium.org/chromium/src/chrome/test/data/webui/settings/people_p...

[gambard, 2016/12/06 10:15:42]

Thanks, I have added it here but I am not sure I need to add it to the
javascript: this options is available only for iOS.
WDYT?

     "bookmarks", "preferences", "passwords", "autofill", "themes",
     "typedUrls", "extensions",  "apps",      "tabs",
 };
 
 static_assert(
     39 == MODEL_TYPE_COUNT,
     "update kUserSelectableDataTypeName to match UserSelectableTypes");
 
 void AddDefaultFieldValue(ModelType datatype,
                           sync_pb::EntitySpecifics* specifics) {
@@ skipping 302 matching lines @@
   // Although the order doesn't technically matter here, it's clearer to keep
   // these in the same order as their definition in the ModelType enum.
   set.Put(BOOKMARKS);
   set.Put(PREFERENCES);
   set.Put(PASSWORDS);
   set.Put(AUTOFILL);
   set.Put(THEMES);
   set.Put(TYPED_URLS);
   set.Put(EXTENSIONS);
   set.Put(APPS);
+  set.Put(READING_LIST);
   set.Put(PROXY_TABS);
   return set;
 }
 
 bool IsUserSelectableType(ModelType model_type) {
   return UserSelectableTypes().Has(model_type);
 }
 
 ModelTypeNameMap GetUserSelectableTypeNameMap() {
   ModelTypeNameMap type_names;
@@ skipping 276 matching lines @@
 bool TypeSupportsHierarchy(ModelType model_type) {
   // TODO(stanisc): crbug/438313: Should this also include TOP_LEVEL_FOLDER?
   return model_type == BOOKMARKS;
 }
 
 bool TypeSupportsOrdering(ModelType model_type) {
   return model_type == BOOKMARKS;
 }
 
 }  // namespace syncer