Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Unified Diff: chrome/browser/bookmarks/bookmark_utils.cc

Issue 7012005: Revert "Revert 84829 - Initial implementation of "Synced Bookmarks" folder." (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Responding to latest round of comments. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..22e40efa16cea562867c7ab3250b36f500be4fdb 100644
--- a/chrome/browser/bookmarks/bookmark_utils.cc
+++ b/chrome/browser/bookmarks/bookmark_utils.cc
@@ -479,6 +479,12 @@ 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 && model->synced_node()->IsVisible() &&
sky 2011/05/17 20:00:05 I think you want this outside the if loop on 479.
Yaron 2011/05/17 21:38:37 Done.
+ find(nodes.begin(), nodes.end(),
+ model->synced_node()) == nodes.end()) {
+ nodes.push_back(model->synced_node());
+ }
}
}
return nodes;

Powered by Google App Engine
This is Rietveld 408576698