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

Unified Diff: chrome/browser/extensions/api/bookmarks/bookmarks_api.h

Issue 308273002: Made the bookmarks extension APIs aware of managed bookmarks. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments Created 6 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/extensions/api/bookmarks/bookmarks_api.h
diff --git a/chrome/browser/extensions/api/bookmarks/bookmarks_api.h b/chrome/browser/extensions/api/bookmarks/bookmarks_api.h
index fc73e0c76f045cd6b62fdc4d324ea31f3095fd89..28abe86f094099ddfbcbd5d87d2627f8d1a2a4d9 100644
--- a/chrome/browser/extensions/api/bookmarks/bookmarks_api.h
+++ b/chrome/browser/extensions/api/bookmarks/bookmarks_api.h
@@ -146,6 +146,12 @@ class BookmarksFunction : public ChromeAsyncExtensionFunction,
// error_ to the appropriate error string.
bool EditBookmarksEnabled();
+ // Helper that checks if |node| can be modified. Returns false if |node|
+ // is NULL, or a managed node, or the root node. In these cases the node
+ // can't be edited, can't have new child nodes appended, and its direct
+ // children can't be moved or reordered.
not at google - send to devlin 2014/06/03 18:09:16 I'm a little confused by this comment. Why does, f
Joao da Silva 2014/06/05 17:10:32 Note that there is _the_ root node, and there are
+ bool CanBeModified(const BookmarkNode* node);
+
private:
// BaseBookmarkModelObserver:
virtual void BookmarkModelChanged() OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698