Index: components/bookmarks/browser/bookmark_model.cc |
diff --git a/components/bookmarks/browser/bookmark_model.cc b/components/bookmarks/browser/bookmark_model.cc |
index 64897a1a631a1e9dfc8f420c2ce38bb5899a65e9..575dc50f78c6b131712129ee44c4c5e6fd9ff524 100644 |
--- a/components/bookmarks/browser/bookmark_model.cc |
+++ b/components/bookmarks/browser/bookmark_model.cc |
@@ -150,7 +150,7 @@ |
const BookmarkNode* BookmarkModel::GetParentForNewNodes() { |
std::vector<const BookmarkNode*> nodes = |
- bookmark_utils::GetMostRecentlyModifiedUserFolders(this, 1); |
+ bookmark_utils::GetMostRecentlyModifiedFolders(this, 1); |
DCHECK(!nodes.empty()); // This list is always padded with default folders. |
return nodes[0]; |
} |
@@ -213,7 +213,7 @@ |
for (int i = 0; i < root_.child_count(); ++i) { |
BookmarkNode* permanent_node = root_.GetChild(i); |
- if (!client_->CanBeEditedByUser(permanent_node)) |
+ if (!client_->CanRemovePermanentNodeChildren(permanent_node)) |
continue; |
for (int j = permanent_node->child_count() - 1; j >= 0; --j) { |
@@ -503,19 +503,15 @@ |
} |
} |
-const BookmarkNode* BookmarkModel::GetMostRecentlyAddedUserNodeForURL( |
+const BookmarkNode* BookmarkModel::GetMostRecentlyAddedNodeForURL( |
const GURL& url) { |
std::vector<const BookmarkNode*> nodes; |
GetNodesByURL(url, &nodes); |
+ if (nodes.empty()) |
+ return NULL; |
+ |
std::sort(nodes.begin(), nodes.end(), &bookmark_utils::MoreRecentlyAdded); |
- |
- // Look for the first node that the user can edit. |
- for (size_t i = 0; i < nodes.size(); ++i) { |
- if (client_->CanBeEditedByUser(nodes[i])) |
- return nodes[i]; |
- } |
- |
- return NULL; |
+ return nodes.front(); |
} |
bool BookmarkModel::HasBookmarks() { |
@@ -624,7 +620,7 @@ |
} |
void BookmarkModel::SortChildren(const BookmarkNode* parent) { |
- DCHECK(client_->CanBeEditedByUser(parent)); |
+ DCHECK(client_->CanReorderChildren(parent)); |
if (!parent || !parent->is_folder() || is_root_node(parent) || |
parent->child_count() <= 1) { |
@@ -653,7 +649,7 @@ |
void BookmarkModel::ReorderChildren( |
const BookmarkNode* parent, |
const std::vector<const BookmarkNode*>& ordered_nodes) { |
- DCHECK(client_->CanBeEditedByUser(parent)); |
+ DCHECK(client_->CanReorderChildren(parent)); |
// Ensure that all children in |parent| are in |ordered_nodes|. |
DCHECK_EQ(static_cast<size_t>(parent->child_count()), ordered_nodes.size()); |