[Mac] Reimplement HoverButton and HoverImageButton using NSButton.state.

ui/base/cocoa/hover_button.mm

 // Copyright (c) 2010 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 "ui/base/cocoa/hover_button.h"
 
 @implementation HoverButton
 
-@synthesize hoverState = hoverState_;
+@synthesize isMouseInside = isMouseInside_;
 
 - (id)initWithFrame:(NSRect)frameRect {
   if ((self = [super initWithFrame:frameRect])) {
     [self setTrackingEnabled:YES];
-    hoverState_ = kHoverStateNone;
     [self updateTrackingAreas];
   }
   return self;
 }
 
 - (void)awakeFromNib {
   [self setTrackingEnabled:YES];
-  self.hoverState = kHoverStateNone;
   [self updateTrackingAreas];
 }
 
 - (void)dealloc {
   [self setTrackingEnabled:NO];
   [super dealloc];
 }
 
+- (void)setIsMouseInside:(BOOL)isInside {
+  isMouseInside_ = isInside;
+  [self setNeedsDisplay:YES];
+}
+
+- (HoverState)hoverState {
+  if (self.state == NSOnState)
+    return kHoverStateMouseDown;
+  if (self.isMouseInside)
+    return kHoverStateMouseOver;
+  return kHoverStateNone;
+}
+
 - (void)mouseEntered:(NSEvent*)theEvent {
   if (trackingArea_.get())
-    self.hoverState = kHoverStateMouseOver;
+    self.isMouseInside = YES;
 }
 
 - (void)mouseExited:(NSEvent*)theEvent {
   if (trackingArea_.get())
-    self.hoverState = kHoverStateNone;
-}
-
-- (void)mouseDown:(NSEvent*)theEvent {
-  self.hoverState = kHoverStateMouseDown;
-  // The hover button needs to hold onto itself here for a bit.  Otherwise,
-  // it can be freed while |super mouseDown:| is in its loop, and the
-  // |checkImageState| call will crash.
-  // http://crbug.com/28220
-  scoped_nsobject<HoverButton> myself([self retain]);
-
-  [super mouseDown:theEvent];
-  // We need to check the image state after the mouseDown event loop finishes.
-  // It's possible that we won't get a mouseExited event if the button was
-  // moved under the mouse during tab resize, instead of the mouse moving over
-  // the button.
-  // http://crbug.com/31279
-  [self checkImageState];
+    self.isMouseInside = NO;
 }
 
 - (void)setTrackingEnabled:(BOOL)enabled {
   if (enabled) {
     trackingArea_.reset(
         [[CrTrackingArea alloc] initWithRect:NSZeroRect
                                      options:NSTrackingMouseEnteredAndExited |
                                              NSTrackingActiveAlways |
                                              NSTrackingInVisibleRect
                                        owner:self
@@ skipping 23 matching lines @@
   [self checkImageState];
 }
 
 - (void)checkImageState {
   if (!trackingArea_.get())
     return;
 
   // Update the button's state if the button has moved.
   NSPoint mouseLoc = [[self window] mouseLocationOutsideOfEventStream];
   mouseLoc = [self convertPoint:mouseLoc fromView:nil];
-  self.hoverState = NSPointInRect(mouseLoc, [self bounds]) ?
-      kHoverStateMouseOver : kHoverStateNone;
-}
-
-- (void)setHoverState:(HoverState)state {
-  hoverState_ = state;
-  [self setNeedsDisplay:YES];
+  self.isMouseInside = NSPointInRect(mouseLoc, [self bounds]);
 }
 
 @end