Make sure to update the BookmarkIndex on a URL Change

components/bookmarks/browser/bookmark_model.cc

Index: components/bookmarks/browser/bookmark_model.cc
diff --git a/components/bookmarks/browser/bookmark_model.cc b/components/bookmarks/browser/bookmark_model.cc
index 60e143caf4f38ccb184bca21ef82b37750874da4..f974d28237d7f1d7303860f6ebc2500fc9793757 100644
--- a/components/bookmarks/browser/bookmark_model.cc
+++ b/components/bookmarks/browser/bookmark_model.cc
@@ -372,7 +372,9 @@ void BookmarkModel::SetURL(const BookmarkNode* node, const GURL& url) {
   {
     base::AutoLock url_lock(url_lock_);
     RemoveNodeFromURLSet(mutable_node);

[sky, 2014/11/03 20:44:46]

I think it would be less error prone if RemoveNodeFromURLSet did this. The name
would have to be updated then, maybe RemoveNodeFromInternalMaps or something.

A similar AddNodeToInternalMaps would be nice.

+    index_->Remove(node);
     mutable_node->set_url(url);
+    index_->Add(node);
     nodes_ordered_by_url_set_.insert(mutable_node);
   }