BookmarkClient can add extra nodes to BookmarkModel.

components/bookmarks/browser/bookmark_model.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_MODEL_H_
 #define COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_MODEL_H_
 
 #include <map>
 #include <set>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/strings/string16.h"
 #include "base/synchronization/lock.h"
 #include "base/synchronization/waitable_event.h"
 #include "components/bookmarks/browser/bookmark_client.h"
 #include "components/bookmarks/browser/bookmark_node.h"
 #include "components/bookmarks/browser/bookmark_service.h"
 #include "ui/gfx/image/image.h"
 #include "url/gurl.h"
 
 class BookmarkExpandedStateTracker;
 class BookmarkIndex;
 class BookmarkModelObserver;
 struct BookmarkMatch;
+class ManagedBookmarksTracker;
 class PrefService;
 class ScopedGroupBookmarkActions;
 
 namespace base {
 class FilePath;
 class SequencedTaskRunner;
 }
 
 namespace bookmarks {
 class BookmarkLoadDetails;
@@ skipping 48 matching lines @@
 
   // Returns the 'bookmark bar' node. This is NULL until loaded.
   const BookmarkNode* bookmark_bar_node() const { return bookmark_bar_node_; }
 
   // Returns the 'other' node. This is NULL until loaded.
   const BookmarkNode* other_node() const { return other_node_; }
 
   // Returns the 'mobile' node. This is NULL until loaded.
   const BookmarkNode* mobile_node() const { return mobile_node_; }
 
+  // Returns the 'managed' node. This is NULL until loaded.
+  const BookmarkNode* managed_node() const { return managed_node_; }
+
   bool is_root_node(const BookmarkNode* node) const { return node == &root_; }
 
   // Returns whether the given |node| is one of the permanent nodes - root node,
-  // 'bookmark bar' node, 'other' node or 'mobile' node.
+  // 'bookmark bar' node, 'other' node,'mobile' node or 'managed' node.
   bool is_permanent_node(const BookmarkNode* node) const {
     return node == &root_ ||
            node == bookmark_bar_node_ ||
            node == other_node_ ||
-           node == mobile_node_;
+           node == mobile_node_ ||
+           node == managed_node_;
   }
 
+  // Returns true if |node| belongs to the |managed_node_| tree. Managed nodes
+  // are never editable, and managed folders can't get bookmarks added, removed

[sky, 2014/05/30 22:31:37]

Do we not allow applying policy changes during runtime?

[Joao da Silva, 2014/06/01 13:32:07]

We do, and the managed bookmarks will update when that happens. Clarified that
these bookmarks are not editable by the user.

+  // or moved.
+  bool IsManaged(const BookmarkNode* node) const;

[sky, 2014/05/30 22:31:37]

This name is mildly confusing. It sort of implies this is equivalent to node ==
managed_node_.

[Joao da Silva, 2014/06/01 13:32:07]

Right, naming is hard :-) How about "IsInManagedTree" or "InManagedTree"?

+
   // Returns the parent the last node was added to. This never returns NULL
   // (as long as the model is loaded).
   const BookmarkNode* GetParentForNewNodes();
 
   void AddObserver(BookmarkModelObserver* observer);
   void RemoveObserver(BookmarkModelObserver* observer);
 
   // Notifies the observers that an extensive set of changes is about to happen,
   // such as during import or sync, so they can delay any expensive UI updates
   // until it's finished.
@@ skipping 262 matching lines @@
   // Whether the initial set of data has been loaded.
   bool loaded_;
 
   // The root node. This contains the bookmark bar node, the 'other' node and
   // the mobile node as children.
   BookmarkNode root_;
 
   BookmarkPermanentNode* bookmark_bar_node_;
   BookmarkPermanentNode* other_node_;
   BookmarkPermanentNode* mobile_node_;
+  BookmarkPermanentNode* managed_node_;
 
   // The maximum ID assigned to the bookmark nodes in the model.
   int64 next_node_id_;
 
   // The observers.
   ObserverList<BookmarkModelObserver> observers_;
 
   // Set of nodes ordered by URL. This is not a map to avoid copying the
   // urls.
   // WARNING: |nodes_ordered_by_url_set_| is accessed on multiple threads. As
@@ skipping 13 matching lines @@
   // True if URLs are stored in the BookmarkIndex in addition to bookmark
   // titles.
   const bool index_urls_;
 
   base::WaitableEvent loaded_signal_;
 
   // See description of IsDoingExtensiveChanges above.
   int extensive_changes_;
 
   scoped_ptr<BookmarkExpandedStateTracker> expanded_state_tracker_;
+  scoped_ptr<ManagedBookmarksTracker> managed_bookmarks_tracker_;
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkModel);
 };
 
 #endif  // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_MODEL_H_