[Mac] Force invalidate bookmark bar layout on node change

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.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 "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.h"
 
 #include <stddef.h>
 
 #import "base/mac/bundle_locations.h"
 #import "base/mac/foundation_util.h"
@@ skipping 2116 matching lines @@
   // If a context menu is open, close it.
   [self cancelMenuTracking];
 
   const BookmarkNode* newNode = newParent->GetChild(newIndex);
   id<BookmarkButtonControllerProtocol> newController =
       [self controllerForNode:newParent];
   [newController addButtonForNode:newNode atIndex:newIndex];
   [self rebuildLayoutWithAnimated:NO];
 }
 
-// TODO(jrg): for now this is brute force.
 - (void)nodeChanged:(BookmarkModel*)model
                node:(const BookmarkNode*)node {
+  // Invalidate the layout if the changed node is visible. This ensures
+  // the button is updated if the button offsets don't change but the
+  // title does.
+  if (nodeIdToButtonMap_.find(node->id()) != nodeIdToButtonMap_.end())
+    layout_ = {};
   [self loaded:model];
 }
 
 - (void)nodeMoved:(BookmarkModel*)model
         oldParent:(const BookmarkNode*)oldParent oldIndex:(int)oldIndex
         newParent:(const BookmarkNode*)newParent newIndex:(int)newIndex {
   const BookmarkNode* movedNode = newParent->GetChild(newIndex);
   id<BookmarkButtonControllerProtocol> oldController =
       [self controllerForNode:oldParent];
   id<BookmarkButtonControllerProtocol> newController =
@@ skipping 531 matching lines @@
     return self;
   return [folderController_ controllerForNode:node];
 }
 
 // For testing.
 - (const BookmarkBarLayout&)currentLayout {
   return layout_;
 }
 
 @end