Revert "Revert 84829 - Initial implementation of "Synced Bookmarks" folder."

chrome/browser/bookmarks/bookmark_utils.cc

Index: chrome/browser/bookmarks/bookmark_utils.cc
diff --git a/chrome/browser/bookmarks/bookmark_utils.cc b/chrome/browser/bookmarks/bookmark_utils.cc
index 22a2d21451ca348ee6bc130dee198b53fcdd0390..aed5d4bac9b256b11436a73b67e4c8599e35ee1b 100644
--- a/chrome/browser/bookmarks/bookmark_utils.cc
+++ b/chrome/browser/bookmarks/bookmark_utils.cc
@@ -479,6 +479,17 @@ std::vector<const BookmarkNode*> GetMostRecentlyModifiedFolders(
     if (nodes.size() < max_count &&
         find(nodes.begin(), nodes.end(), model->other_node()) == nodes.end()) {
       nodes.push_back(model->other_node());
+
+      if (nodes.size() < max_count &&
+          find(nodes.begin(), nodes.end(),
+               model->synced_node()) == nodes.end()) {
+        nodes.push_back(model->synced_node());
+        if (nodes.size() < max_count &&

[sky, 2011/05/17 15:59:08]

I don't think you want this inner if, do you?
Also, shouldn't you be checking IsVisible?

[Yaron, 2011/05/17 19:17:33]

Woah. Not sure what happened there. Fixed. And ya, added IsVisible guard.

+            std::find(nodes.begin(), nodes.end(),
+                      model->synced_node()) == nodes.end()) {
+          nodes.push_back(model->synced_node());
+        }
+      }
     }
   }
   return nodes;