Mac. Ensure no more than one visibly selected item in each bookmark folder menu.

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

 // Copyright (c) 2011 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_folder_controller.h"
 
 #include "base/mac/mac_util.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_utils.h"
@@ skipping 192 matching lines @@
                 copy:(BOOL)copy;
 
 @end
 
 @interface BookmarkButton (BookmarkBarFolderMenuHighlighting)
 
 // Make the button's border frame always appear when |forceOn| is YES,
 // otherwise only border the button when the mouse is inside the button.
 - (void)forceButtonBorderToStayOnAlways:(BOOL)forceOn;
 
-// On 10.6 event dispatch for an NSButtonCell's
-// showsBorderOnlyWhileMouseInside seems broken if scrolling the
-// view that contains the button.  It appears that a mouseExited:
-// gets lost, so the button stays highlit forever.  We accomodate
-// here.
-- (void)toggleButtonBorderingWhileMouseInside;
 @end
 
 @implementation BookmarkButton (BookmarkBarFolderMenuHighlighting)
 
 - (void)forceButtonBorderToStayOnAlways:(BOOL)forceOn {
   [self setShowsBorderOnlyWhileMouseInside:!forceOn];
   [self setNeedsDisplay];
 }
 
-- (void)toggleButtonBorderingWhileMouseInside {
-  BOOL toggle = [self showsBorderOnlyWhileMouseInside];
-  [self setShowsBorderOnlyWhileMouseInside:!toggle];
-  [self setShowsBorderOnlyWhileMouseInside:toggle];
-}
-
 @end
 
 @implementation BookmarkBarFolderController
 
 @synthesize subFolderGrowthToRight = subFolderGrowthToRight_;
 
 - (id)initWithParentButton:(BookmarkButton*)button
           parentController:(BookmarkBarFolderController*)parentController
              barController:(BookmarkBarController*)barController {
   NSString* nibPath =
@@ skipping 614 matching lines @@
 
 // End a scrolling timer.  Can be called excessively with no harm.
 - (void)endScroll {
   if (scrollTimer_) {
     [scrollTimer_ invalidate];
     scrollTimer_ = nil;
     verticalScrollDelta_ = 0;
   }
 }
 
+- (int)indexOfButton:(BookmarkButton*)button {
+  if (button == nil)
+    return -1;
+  int index = [buttons_ indexOfObject:button];
+  return (index == NSNotFound) ? -1 : index;
+}
+
+- (BookmarkButton*)buttonAtIndex:(int)which {
+  if (which < 0 || which >= [self buttonCount])
+    return nil;
+  return [buttons_ objectAtIndex:which];
+}
+
+// Private, called by performOneScroll only.
+// If the button at index contains the mouse it will select it and return YES.
+// Otherwise returns NO.
+- (BOOL)selectButtonIfHoveredAtIndex:(int)index {
+  BookmarkButton *btn = [self buttonAtIndex:index];
+  if ([[btn cell] isMouseReallyInside]) {
+    buttonThatMouseIsIn_ = btn;
+    [self setSelectedButtonByIndex:index];
+    return YES;
+  }
+  return NO;
+}
+
 // Perform a single scroll of the specified amount.
 - (void)performOneScroll:(CGFloat)delta {
   if (delta == 0.0)
     return;
   CGFloat finalDelta = [self determineFinalScrollDelta:delta];
-  if (finalDelta > 0.0 || finalDelta < 0.0) {
-    if (buttonThatMouseIsIn_)
-      [buttonThatMouseIsIn_ toggleButtonBorderingWhileMouseInside];
-    NSRect windowFrame = [[self window] frame];
-    NSSize newSize = NSMakeSize(NSWidth(windowFrame), 0.0);
-    [self adjustWindowLeft:windowFrame.origin.x
-                      size:newSize
-               scrollingBy:finalDelta];
+  if (finalDelta == 0.0)
+    return;
+  int index = [self indexOfButton:buttonThatMouseIsIn_];
+  // Check for a current mouse-initiated selection.
+  BOOL maintainHoverSelection =
+      (buttonThatMouseIsIn_ &&
+      [[buttonThatMouseIsIn_ cell] isMouseReallyInside] &&
+      selectedIndex_ != -1 &&
+      index == selectedIndex_);
+  NSRect windowFrame = [[self window] frame];
+  NSSize newSize = NSMakeSize(NSWidth(windowFrame), 0.0);
+  [self adjustWindowLeft:windowFrame.origin.x
+                    size:newSize
+             scrollingBy:finalDelta];
+  // We have now scrolled.
+  if (!maintainHoverSelection)
+    return;
+  // Is mouse still in the same hovered button?
+  if ([[buttonThatMouseIsIn_ cell] isMouseReallyInside])
+    return;
+  // The finalDelta scroll direction will tell us us whether to search up or
+  // down the buttons array for the newly hovered button.
+  if (finalDelta < 0.0) { // Scrolled up, so search backwards for new hover.
+    index--;
+    while (index >= 0) {
+      if ([self selectButtonIfHoveredAtIndex:index])
+        return;
+      index--;
+    }
+  } else { // Scrolled down, so search forward for new hovered button.
+    index++;
+    int btnMax = [self buttonCount];
+    while (index < btnMax) {
+      if ([self selectButtonIfHoveredAtIndex:index])
+        return;
+      index++;
+    }
   }
 }
 
 - (CGFloat)determineFinalScrollDelta:(CGFloat)delta {
   if ((delta > 0.0 && ![scrollUpArrowView_ isHidden]) ||
       (delta < 0.0 && ![scrollDownArrowView_ isHidden])) {
     NSWindow* window = [self window];
     NSRect windowFrame = [window frame];
     NSScreen* screen = [window screen];
     NSPoint scrollPosition = [scrollView_ documentVisibleRect].origin;
@@ skipping 370 matching lines @@
   // If a "hover open" is pending when the bookmark bar folder is
   // closed, be sure it gets cancelled.
   [NSObject cancelPreviousPerformRequestsWithTarget:self];
 
   [self endScroll];  // Just in case we were scrolling.
   [barController_ childFolderWillClose:self];
   [self closeBookmarkFolder:self];
   [self autorelease];
 }
 
-- (int)indexOfButton:(BookmarkButton*)button {
-  int index = [buttons_ indexOfObject:button];
-  return (index == NSNotFound) ? -1 : index;
-}
-
-- (BookmarkButton*)buttonAtIndex:(int)which {
-  if (which < 0 || which >= [self buttonCount])
-    return nil;
-  return [buttons_ objectAtIndex:which];
-}
-
 #pragma mark BookmarkButtonDelegate Protocol
 
 - (void)fillPasteboard:(NSPasteboard*)pboard
        forDragOfButton:(BookmarkButton*)button {
   [[self folderTarget] fillPasteboard:pboard forDragOfButton:button];
 
   // Close our folder menu and submenus since we know we're going to be dragged.
   [self closeBookmarkFolder:self];
 }
 
@@ skipping 16 matching lines @@
   [self performSelector:@selector(openBookmarkFolderFromButtonAndCloseOldOne:)
              withObject:sender
              afterDelay:bookmarks::kHoverOpenDelay
                 inModes:[NSArray arrayWithObject:NSRunLoopCommonModes]];
 }
 
 // Called from the BookmarkButton
 - (void)mouseExitedButton:(id)sender event:(NSEvent*)event {
   if (buttonThatMouseIsIn_ == sender)
     buttonThatMouseIsIn_ = nil;
+    [self setSelectedButtonByIndex:-1];
 
   // Stop any timer about opening a new hover-open folder.
 
   // Since a performSelector:withDelay: on self retains self, it is
   // possible that a cancelPreviousPerformRequestsWithTarget: reduces
   // the refcount to 0, releasing us.  That's a bad thing to do while
   // this object (or others it may own) is in the event chain.  Thus
   // we have a retain/autorelease.
   [self retain];
   [NSObject cancelPreviousPerformRequestsWithTarget:self];
@@ skipping 171 matching lines @@
 
   [self performOneScroll:delta];
 }
 
 // All changes to selectedness of buttons (aka fake menu items) ends up
 // calling this method to actually flip the state of items.
 // Needs to handle -1 as the invalid index (when nothing is selected) and
 // greater than range values too.
 - (void)setStateOfButtonByIndex:(int)index
                           state:(bool)state {
- if (index < 0 || index > ([self buttonCount] -1))
-   return;
-
-  [[buttons_ objectAtIndex:index] highlight:state];
+  if (index >= 0 && index < [self buttonCount])
+    [[buttons_ objectAtIndex:index] highlight:state];
 }
 
 // Selects the required button and deselects the previously selected one.
 // An index of -1 means no selection.
 - (void)setSelectedButtonByIndex:(int)index {
   if (index == selectedIndex_)
     return;
 
   [self setStateOfButtonByIndex:selectedIndex_ state:NO];
   [self setStateOfButtonByIndex:index state:YES];
@@ skipping 432 matching lines @@
 
 - (void)setIgnoreAnimations:(BOOL)ignore {
   ignoreAnimations_ = ignore;
 }
 
 - (BookmarkButton*)buttonThatMouseIsIn {
   return buttonThatMouseIsIn_;
 }
 
 @end  // BookmarkBarFolderController