[Mac] Force invalidate bookmark bar layout on node change

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller_unittest.mm

 // Copyright (c) 2012 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.
 
 #import <Cocoa/Cocoa.h>
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/command_line.h"
 #include "base/mac/mac_util.h"
@@ skipping 702 matching lines @@
   layout = [bar_ layoutFromCurrentState];
   EXPECT_FALSE(layout.IsAppsButtonVisible());
 
   // And try showing it via policy too.
   prefs->SetManagedPref(bookmarks::prefs::kShowAppsShortcutInBookmarkBar,
                         base::MakeUnique<base::Value>(true));
   layout = [bar_ layoutFromCurrentState];
   EXPECT_TRUE(layout.IsAppsButtonVisible());
 }
 
+// This tests a formerly-pathological case where if there was a single
+// bookmark on the bar and it was renamed, the button wouldn't update
+// since the offset and number of buttons stayed the same.
+// See crbug.com/724201
+TEST_F(BookmarkBarControllerTest, RenameBookmark) {
+  BookmarkModel* model = BookmarkModelFactory::GetForBrowserContext(profile());
+  const BookmarkNode* barNode = model->bookmark_bar_node();
+  const BookmarkNode* node = model->AddURL(barNode, 0, ASCIIToUTF16("Lorem"),
+                                           GURL("http://example.com"));
+  EXPECT_EQ([[bar_ buttons] count], 1U);
+  EXPECT_NSEQ([[[bar_ buttons] firstObject] title], @"Lorem");
+
+  model->SetTitle(node, ASCIIToUTF16("Ipsum"));
+  [bar_ rebuildLayoutWithAnimated:NO];
+  EXPECT_EQ([[bar_ buttons] count], 1U);
+  EXPECT_NSEQ([[[bar_ buttons] firstObject] title], @"Ipsum");
+}
+
 TEST_F(BookmarkBarControllerTest, NoItemsResizing) {
   bookmarks::BookmarkBarLayout layout = [bar_ layoutFromCurrentState];
   EXPECT_TRUE(layout.IsNoItemTextFieldVisible() &&
               layout.IsImportBookmarksButtonVisible());
   CGFloat oldOffset = layout.import_bookmarks_button_offset;
   CGFloat oldWidth = layout.import_bookmarks_button_width;
   CGRect viewFrame = [[bar_ view] frame];
   CGFloat originalViewWidth = NSWidth(viewFrame);
   // Assert that the import bookmarks button fits.
   EXPECT_GT(originalViewWidth, oldOffset + oldWidth);
@@ skipping 1443 matching lines @@
                              "2f3b ] 4b ");
   actual = bookmarks::test::ModelStringFromNode(root);
   EXPECT_EQ(expected, actual);
 
   // Verify that the other bookmark folder can't be deleted.
   BookmarkButton *otherButton = [bar_ otherBookmarksButton];
   EXPECT_FALSE([bar_ canDragBookmarkButtonToTrash:otherButton]);
 }
 
 }  // namespace