MacViews: Fix sending mouse exit event and releasing capture on D&D.

ui/views/cocoa/drag_drop_client_mac_unittest.mm

 // Copyright 2016 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/views/cocoa/drag_drop_client_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
+#import "base/mac/scoped_objc_class_swizzler.h"
 #include "base/mac/sdk_forward_declarations.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/threading/thread_task_runner_handle.h"
 #import "ui/views/cocoa/bridged_native_widget.h"
 #include "ui/views/test/widget_test.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/native_widget_mac.h"
 #include "ui/views/widget/widget.h"
 
 using base::ASCIIToUTF16;
 
+@interface NSView (DragSessionTestingDonor)
+@end
+
+@implementation NSView (DragSessionTestingDonor)
+- (NSDraggingSession*)cr_beginDraggingSessionWithItems:(NSArray*)items
+                                                 event:(NSEvent*)event
+                                                source:(id<NSDraggingSource>)
+                                                           source {
+  return nil;
+}
+@end
+
 // Mocks the NSDraggingInfo sent to the DragDropClientMac's DragUpdate() and
 // Drop() methods. Out of the required methods of the protocol, only
 // draggingLocation and draggingPasteboard are used.
 @interface MockDraggingInfo : NSObject<NSDraggingInfo> {
   NSPasteboard* pasteboard_;
 }
 
 @property BOOL animatesToDestination;
 @property NSInteger numberOfValidItemsForDrop;
 @property NSDraggingFormation draggingFormation;
@@ skipping 173 matching lines @@
   SetData(data);
 
   target_->set_formats(ui::OSExchangeData::STRING | ui::OSExchangeData::URL);
 
   // Check if the target receives the data from a drop and returns the expected
   // operation.
   EXPECT_EQ(DragUpdate(nil), NSDragOperationCopy);
   EXPECT_EQ(Drop(), NSDragOperationMove);
 }
 
+// Ensure that capture is released before the end of a drag and drop operation.
+TEST_F(DragDropClientMacTest, ReleaseCapture) {
+  // DragDropView doesn't actually capture the mouse, so explicitly acquire it
+  // to test that StartDragAndDrop() actually releases it.
+  // Although this is not an interactive UI test, acquiring capture should be OK
+  // since the runloop will exit before the system has any opportunity to
+  // capture anything.
+  bridge_->AcquireCapture();
+  EXPECT_TRUE(bridge_->HasCapture());
+
+  // Create the drop data
+  OSExchangeData data;
+  const base::string16& text = ASCIIToUTF16("text");
+  data.SetString(text);
+  SetData(data);
+
+  // There's no way to cleanly stop NSDraggingSession inside unit tests, so just
+  // don't start it at all.
+  base::mac::ScopedObjCClassSwizzler swizzle(
+      [NSView class], @selector(beginDraggingSessionWithItems:event:source:),
+      @selector(cr_beginDraggingSessionWithItems:event:source:));
+
+  // Immediately quit drag'n'drop, or we'll hang.
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&DragDropClientMac::EndDrag,
+                            base::Unretained(drag_drop_client())));
+
+  // It will call ReleaseCapture().
+  drag_drop_client()->StartDragAndDrop(
+      target_, data, 0, ui::DragDropTypes::DRAG_EVENT_SOURCE_MOUSE);
+
+  // The capture should be released.
+  EXPECT_FALSE(bridge_->HasCapture());
+}
+
 // Tests if the drag and drop target rejects the dropped data with the
 // incorrect format.
 TEST_F(DragDropClientMacTest, InvalidFormatDragDrop) {
   OSExchangeData data;
   const base::string16& text = ASCIIToUTF16("text");
   data.SetString(text);
   SetData(data);
 
   target_->set_formats(ui::OSExchangeData::URL);
 
@@ skipping 17 matching lines @@
 
   // Add valid data to the pasteboard and check to see if the target accepts
   // it.
   [pasteboard setString:@"text" forType:NSPasteboardTypeString];
   EXPECT_EQ(DragUpdate(pasteboard), NSDragOperationCopy);
   EXPECT_EQ(Drop(), NSDragOperationMove);
 }
 
 }  // namespace test
 }  // namespace views