Mac Panels interaction with Dock modifications. (step 1)

chrome/browser/ui/panels/panel_titlebar_view_cocoa.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/panels/panel_titlebar_view_cocoa.h"
 
 #include <Carbon/Carbon.h>  // kVK_Escape
 #import <Cocoa/Cocoa.h>
 
 #include "base/logging.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "chrome/browser/themes/theme_service.h"
 #import "chrome/browser/ui/cocoa/hover_image_button.h"
 #import "chrome/browser/ui/cocoa/nsview_additions.h"
 #import "chrome/browser/ui/cocoa/themed_window.h"
 #import "chrome/browser/ui/cocoa/tracking_area.h"
 #import "chrome/browser/ui/panels/panel_window_controller_cocoa.h"
 #include "grit/theme_resources_standard.h"
 #import "third_party/GTM/AppKit/GTMNSBezierPath+RoundRect.h"
 #include "ui/gfx/mac/nsimage_cache.h"
 #include "ui/gfx/scoped_ns_graphics_context_save_gstate_mac.h"
 
 const int kRoundedCornerSize = 3;
 const int kButtonPadding = 8;
 const int kIconAndTextPadding = 5;
 
+// Distance that user needs to move the mouse in order to start the drag.
+// Threshold is needed to differentiate drags from attempts to click the
+// titlebar with a twitch of the mouse pointer.
+const int kDragThreshold = 3;
+
 // Used to implement TestingAPI
 static NSEvent* MakeMouseEvent(NSEventType type,
                                NSPoint point,
                                int clickCount) {
   return [NSEvent mouseEventWithType:type
                             location:point
                        modifierFlags:0
                            timestamp:0
                         windowNumber:0
                              context:nil
@@ skipping 273 matching lines @@
   [self setNeedsDisplay:YES];
 }
 
 - (void)didChangeMainWindow:(NSNotification*)notification {
   [self setNeedsDisplay:YES];
   [self checkMouseAndUpdateSettingsButtonVisibility];
 }
 
 - (void)mouseDown:(NSEvent*)event {
   dragState_ = PANEL_DRAG_CAN_START;
+  dragStartLocation_ = [event locationInWindow];
 }
 
 - (void)mouseUp:(NSEvent*)event {
   DCHECK(dragState_ != PANEL_DRAG_IN_PROGRESS);
 
   if ([event clickCount] == 1)
     [controller_ onTitlebarMouseClicked];
 }
 
+- (BOOL)exceedsDragThreshold:(NSPoint)mouseLocation {
+  float deltaX = dragStartLocation_.x - mouseLocation.x;
+  float deltaY = dragStartLocation_.y - mouseLocation.y;
+  return deltaX > kDragThreshold || deltaY > kDragThreshold;

[jennb, 2011/11/18 00:36:55]

So much simpler! Shaves a few cycles, too. :-D

+}
+
 - (void)mouseDragged:(NSEvent*)event {
   // In addition to events needed to control the drag operation, fetch the right
   // mouse click events and key down events and ignore them, to prevent their
   // accumulation in the queue and "playing out" when the mouse is released.
   const NSUInteger mask =
       NSLeftMouseUpMask | NSLeftMouseDraggedMask | NSKeyUpMask |
       NSRightMouseDownMask | NSKeyDownMask ;
   BOOL keepGoing = YES;
 
   while (keepGoing) {
     base::mac::ScopedNSAutoreleasePool autorelease_pool;
 
     NSEvent* event = [NSApp nextEventMatchingMask:mask
                                         untilDate:[NSDate distantFuture]
                                            inMode:NSDefaultRunLoopMode
                                           dequeue:YES];
 
     switch ([event type]) {
       case NSLeftMouseDragged:
-        if (dragState_ == PANEL_DRAG_CAN_START)
+        if (dragState_ == PANEL_DRAG_CAN_START) {
+          if (![self exceedsDragThreshold:[event locationInWindow]])
+            return;  // Don't start real drag yet.
           [self startDrag];
+        }
         [self dragWithDeltaX:[event deltaX]];
         break;
 
       case NSKeyUp:
         if ([event keyCode] == kVK_Escape) {
           [self endDrag:YES];
           keepGoing = NO;
         }
         break;
 
       case NSLeftMouseUp:
-        [self endDrag:NO];
+        if (dragState_ == PANEL_DRAG_CAN_START)
+          [self mouseUp:event];  // Drag didn't really start, minimize instead.
+        else
+          [self endDrag:NO];
         keepGoing = NO;
         break;
 
       case NSRightMouseDownMask:
         break;
 
       default:
         // Dequeue and ignore other mouse and key events so the Chrome context
         // menu does not come after right click on a page during Panel
         // rearrangement, or the keystrokes are not 'accumulated' and entered
@@ skipping 88 matching lines @@
 }
 
 - (void)checkMouseAndUpdateSettingsButtonVisibility {
   BOOL mouseOverWindow = NSPointInRect([NSEvent mouseLocation],
                                        [[self window] frame]);
   [self updateSettingsButtonVisibility:mouseOverWindow];
 }
 
 @end