Have the ApplyEdit* bookmark utilities return the BookmarkNode pointer...

chrome/browser/bookmarks/bookmark_utils.cc

 // Copyright (c) 2006-2008 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.
 
 #include "chrome/browser/bookmarks/bookmark_utils.h"
 
 #include "app/drag_drop_types.h"
 #include "app/l10n_util.h"
 // TODO(port): Port these files.
 #if defined(OS_WIN)
@@ skipping 495 matching lines @@
                              const std::wstring& languages) {
   std::vector<std::wstring> words;
   QueryParser parser;
   parser.ExtractQueryWords(l10n_util::ToLower(text), &words);
   if (words.empty())
     return false;
 
   return (node->is_url() && DoesBookmarkContainWords(node, words, languages));
 }
 
-void ApplyEditsWithNoGroupChange(BookmarkModel* model,
-                                 const BookmarkNode* parent,
-                                 const BookmarkNode* node,
-                                 const std::wstring& new_title,
-                                 const GURL& new_url,
-                                 BookmarkEditor::Handler* handler) {
+const BookmarkNode* ApplyEditsWithNoGroupChange(BookmarkModel* model,
+    const BookmarkNode* parent, const BookmarkNode* node,
+    const std::wstring& new_title, const GURL& new_url,
+    BookmarkEditor::Handler* handler) {
   const BookmarkNode* old_parent = node ? node->GetParent() : NULL;
   const int old_index = old_parent ? old_parent->IndexOfChild(node) : -1;
 
   if (!node) {
     node =
         model->AddURL(parent, parent->GetChildCount(), new_title, new_url);
 
     if (handler)
       handler->NodeCreated(node);
-    return;
+    return node;
   }
 
   // If we're not showing the tree we only need to modify the node.
   if (old_index == -1) {
     NOTREACHED();
-    return;
+    return node;
   }
 
   if (new_url != node->GetURL()) {
-    model->AddURLWithCreationTime(old_parent, old_index, new_title,
-                                  new_url, node->date_added());
+    const BookmarkNode* new_node = model->AddURLWithCreationTime(old_parent,
+        old_index, new_title, new_url, node->date_added());
     model->Remove(old_parent, old_index + 1);
+    return new_node;
   } else {
     model->SetTitle(node, new_title);
   }
+  return node;
 }
 
-void ApplyEditsWithPossibleGroupChange(BookmarkModel* model,
-                                       const BookmarkNode* new_parent,
-                                       const BookmarkNode* node,
-                                       const std::wstring& new_title,
-                                       const GURL& new_url,
-                                       BookmarkEditor::Handler* handler) {
+const BookmarkNode* ApplyEditsWithPossibleGroupChange(BookmarkModel* model,
+    const BookmarkNode* new_parent, const BookmarkNode* node,
+    const std::wstring& new_title, const GURL& new_url,
+    BookmarkEditor::Handler* handler) {
   const BookmarkNode* old_parent = node ? node->GetParent() : NULL;
   const int old_index = old_parent ? old_parent->IndexOfChild(node) : -1;
-
+  const BookmarkNode* return_node = node;
   if (node) {
     Time date_added = node->date_added();
     if (new_parent == node->GetParent()) {
       // The parent is the same.
       if (new_url != node->GetURL()) {
         model->Remove(old_parent, old_index);
-        model->AddURLWithCreationTime(old_parent, old_index,
-                                      new_title, new_url, date_added);
+        return_node = model->AddURLWithCreationTime(old_parent, old_index,
+            new_title, new_url, date_added);
       } else {
         model->SetTitle(node, new_title);
       }
     } else if (new_url != node->GetURL()) {
       // The parent and URL changed.
       model->Remove(old_parent, old_index);
-      model->AddURLWithCreationTime(new_parent, new_parent->GetChildCount(),
-                                    new_title, new_url, date_added);
+      return_node = model->AddURLWithCreationTime(new_parent,
+          new_parent->GetChildCount(), new_title, new_url, date_added);
     } else {
       // The parent and title changed. Move the node and change the title.
       model->Move(node, new_parent, new_parent->GetChildCount());
       model->SetTitle(node, new_title);
     }
   } else {
     // We're adding a new URL.
-    node =
+    return_node =
         model->AddURL(new_parent, new_parent->GetChildCount(), new_title,
                       new_url);
     if (handler)
-      handler->NodeCreated(node);
+      handler->NodeCreated(return_node);
   }
+  return return_node;
 }
 
 // Formerly in BookmarkBarView
 void ToggleWhenVisible(Profile* profile) {
   PrefService* prefs = profile->GetPrefs();
   const bool always_show = !prefs->GetBoolean(prefs::kShowBookmarkBar);
 
   // The user changed when the bookmark bar is shown, update the preferences.
   prefs->SetBoolean(prefs::kShowBookmarkBar, always_show);
   prefs->ScheduleSavePersistentPrefs();
@@ skipping 19 matching lines @@
 
   // Formerly in BookmarkTableView
   prefs->RegisterIntegerPref(prefs::kBookmarkTableNameWidth1, -1);
   prefs->RegisterIntegerPref(prefs::kBookmarkTableURLWidth1, -1);
   prefs->RegisterIntegerPref(prefs::kBookmarkTableNameWidth2, -1);
   prefs->RegisterIntegerPref(prefs::kBookmarkTableURLWidth2, -1);
   prefs->RegisterIntegerPref(prefs::kBookmarkTablePathWidth, -1);
 }
 
 }  // namespace bookmark_utils