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

chrome/browser/bookmarks/bookmark_utils.h

 // 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.
 
 #ifndef CHROME_BROWSER_BOOKMARKS_BOOKMARK_UTILS_H_
 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_UTILS_H_
 
 #include <vector>
 
 #include "base/gfx/native_widget_types.h"
@@ skipping 124 matching lines @@
                                 const std::wstring& languages,
                                 std::vector<const BookmarkNode*>* nodes);
 
 // Returns true if |node|'s url or title contains the string |text|.
 // |languages| is user's accept-language setting to decode IDN.
 bool DoesBookmarkContainText(const BookmarkNode* node,
                              const std::wstring& text,
                              const std::wstring& languages);
 
 // Modifies a bookmark node (assuming that there's no magic that needs to be
-// done regarding moving from one folder to another).
-void ApplyEditsWithNoGroupChange(BookmarkModel* model,
-                                 const BookmarkNode* parent,
-                                 const BookmarkNode* node,
-                                 const std::wstring& new_title,
-                                 const GURL& new_url,
-                                 BookmarkEditor::Handler* handler);
+// done regarding moving from one folder to another).  If the URL changed or a
+// new node is explicitly being added, returns a pointer to the new node that
+// was created.  Otherwise the return value is identically |node|.
+const BookmarkNode* ApplyEditsWithNoGroupChange(
+    BookmarkModel* model,
+    const BookmarkNode* parent,
+    const BookmarkNode* node,
+    const std::wstring& new_title,
+    const GURL& new_url,
+    BookmarkEditor::Handler* handler);
 
 // Modifies a bookmark node assuming that the parent of the node may have
-// changed and the node will need to be removed and reinserted.
-void ApplyEditsWithPossibleGroupChange(BookmarkModel* model,
-                                       const BookmarkNode* new_parent,
-                                       const BookmarkNode* node,
-                                       const std::wstring& new_title,
-                                       const GURL& new_url,
-                                       BookmarkEditor::Handler* handler);
+// changed and the node will need to be removed and reinserted.  If the URL
+// changed or a new node is explicitly being added, returns a pointer to the
+// new node that was created.  Otherwise the return value is identically |node|.
+const BookmarkNode* ApplyEditsWithPossibleGroupChange(
+    BookmarkModel* model,
+    const BookmarkNode* new_parent,
+    const BookmarkNode* node,
+    const std::wstring& new_title,
+    const GURL& new_url,
+    BookmarkEditor::Handler* handler);
 
 // Toggles whether the bookmark bar is shown only on the new tab page or on
 // all tabs.  This is a preference modifier, not a visual modifier.
 void ToggleWhenVisible(Profile* profile);
 
 // Register local state prefs for bookmark bar view.
 void RegisterPrefs(PrefService* prefs);
 
 // Register user prefs for BookmarkBar, BookmarkView, ...
 void RegisterUserPrefs(PrefService* prefs);
 
 // Number of bookmarks we'll open before prompting the user to see if they
 // really want to open all.
 //
 // NOTE: treat this as a const. It is not const as various tests change the
 // value.
 extern int num_urls_before_prompting;
 
 }  // namespace bookmark_utils
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_UTILS_H_