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

Unified Diff: chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.cc

Issue 8772064: Adds back BookmarkNode::IsVisible. Turns out we want to conditionally (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tweaks Created 9 years 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/ui/gtk/bookmarks/bookmark_tree_model.cc
diff --git a/chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.cc b/chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.cc
index 89bc9bd4e4ae0b551a96e142ecf85fd4c304af2e..04a9d799e4ba482358628965e3dd808876b5d651 100644
--- a/chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.cc
+++ b/chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.cc
@@ -118,7 +118,8 @@ void AddToTreeStore(BookmarkModel* model, int64 selected_id,
const BookmarkNode* root_node = model->root_node();
for (int i = 0; i < root_node->child_count(); ++i) {
const BookmarkNode* child = root_node->GetChild(i);
- AddToTreeStoreAt(child, selected_id, store, selected_iter, NULL);
+ if (child->IsVisible())
+ AddToTreeStoreAt(child, selected_id, store, selected_iter, NULL);
}
}

Powered by Google App Engine
This is Rietveld 408576698