Sync: Support nodes with implicit permanent folders: Node Browser and out-of-order loading from DB

sync/syncable/model_type.cc

Index: sync/syncable/model_type.cc
diff --git a/sync/syncable/model_type.cc b/sync/syncable/model_type.cc
index 602582beb2f2cc9dfebb410392fbedd0de2d1b9d..113480332bde54c532b0ec9f085c03b07fa1ab4d 100644
--- a/sync/syncable/model_type.cc
+++ b/sync/syncable/model_type.cc
@@ -1216,4 +1216,12 @@ bool IsTypeWithClientGeneratedRoot(ModelType model_type) {
          !IsTypeWithServerGeneratedRoot(model_type);
 }
 
+bool TypeSupportsHierarchy(ModelType model_type) {
+  return model_type == BOOKMARKS;

[Nicolas Zea, 2015/07/09 20:36:34]

should this include TOP_LEVEL_FOLDER?

[stanisc, 2015/07/09 21:45:16]

Perhaps. Is TOP_LEVEL_FOLDER type actually used other than in some old tests?

[stanisc, 2015/07/09 23:59:27]

Added TODO. Adding TOP_LEVEL_FOLDER might affect the code that calls
EntryKernel::ShouldMaintainHierarchy. We should review how TOP_LEVEL_FOLDER is
used in the code first.

+}
+
+bool TypeSupportsOrdering(ModelType model_type) {
+  return model_type == BOOKMARKS;
+}
+
 }  // namespace syncer