Avoid conversion of index to BookmarkNode pointer unnacessarily.

components/bookmarks/browser/bookmark_model.h

Index: components/bookmarks/browser/bookmark_model.h
diff --git a/components/bookmarks/browser/bookmark_model.h b/components/bookmarks/browser/bookmark_model.h
index 076b01b5c53b6c30c7f30afbda03d4952ee014f9..4adbf24e49e10590a1c148298166ee564cffb460 100644
--- a/components/bookmarks/browser/bookmark_model.h
+++ b/components/bookmarks/browser/bookmark_model.h
@@ -127,9 +127,9 @@ class BookmarkModel : public KeyedService {
   // state during their own initializer, such as the NTP.
   bool IsDoingExtensiveChanges() const { return extensive_changes_ > 0; }
 
-  // Removes the node at the given |index| from |parent|. Removing a folder node
-  // recursively removes all nodes. Observers are notified immediately.
-  void Remove(const BookmarkNode* parent, int index);
+  // Removes the child node. Removing a folder node recursively removes all
+  // nodes. Observers are notified immediately.
+  void Remove(const BookmarkNode* parent, const BookmarkNode* child);

[sky, 2015/04/29 15:26:27]

If you're going to clean this code up, there is no point in supplying parent.
You can get the parent from child.

 
   // Removes all the non-permanent bookmark nodes that are editable by the user.
   // Observers are only notified when all nodes have been removed. There is no