Changes the way we calculate hit rects in TabView to reduce...

chrome/browser/cocoa/tab_view.mm

 // Copyright (c) 2009 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.
 
 #include "chrome/browser/cocoa/tab_view.h"
 #include "chrome/browser/cocoa/tab_window_controller.h"
 
 @implementation TabView
 
 - (id)initWithFrame:(NSRect)frame {
@@ skipping 14 matching lines @@
 // hierarchy) first. We want to handle clicks and drags in this class and
 // leave the background button for display purposes only.
 - (BOOL)acceptsFirstMouse:(NSEvent *)theEvent {
   return YES;
 }
 
 // Determines which view a click in our frame actually hit. It's either this
 // view or our child close button.
 - (NSView *)hitTest:(NSPoint)aPoint {
   NSPoint viewPoint = [self convertPoint:aPoint fromView:[self superview]];
+  NSRect frame = [self frame];
+
+  // Reduce the width of the hit rect slightly to remove the overlap
+  // between adjacent tabs.  The drawing code in TabCell has the top
+  // corners of the tab inset by height*2/3, so we inset by half of
+  // that here.  This doesn't completely eliminate the overlap, but it
+  // works well enough.
+  NSRect hitRect = NSInsetRect(frame, frame.size.height / 3.0f, 0);
   if (NSPointInRect(viewPoint, [closeButton_ frame])) return closeButton_;
-  if (NSPointInRect(aPoint, [self frame])) return self;
+  if (NSPointInRect(aPoint, hitRect)) return self;
   return nil;
 }
 
 // Handle clicks and drags in this button. We get here because we have
 // overridden acceptsFirstMouse: and the click is within our bounds.
 // TODO(pinkerton/alcor): This routine needs *a lot* of work to marry Cole's
 // ideas of dragging cocoa views between windows and how the Browser and
 // TabStrip models want to manage tabs.
 - (void)mouseDown:(NSEvent *)theEvent {
   static const CGFloat kTearDistance = 36.0;
@@ skipping 248 matching lines @@
       TabWindowController* dropController = sourceController;
       [dropController dropTabView:[dropController selectedTabView]
                    fromController:nil];
     }
   }
 
   [sourceController removePlaceholder];
 }
 
 @end