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

ui/views/cocoa/bridged_content_view.mm

Index: ui/views/cocoa/bridged_content_view.mm
diff --git a/ui/views/cocoa/bridged_content_view.mm b/ui/views/cocoa/bridged_content_view.mm
index 1623e32b8ede82ff2b578ad5500010dca09f991c..d666d850d4d519959bb129c30f7f2c44a468c7c8 100644
--- a/ui/views/cocoa/bridged_content_view.mm
+++ b/ui/views/cocoa/bridged_content_view.mm
@@ -555,6 +555,12 @@ ui::KeyEvent GetCharacterEventFromNSEvent(NSEvent* event) {
   hostedView_->GetWidget()->OnMouseEvent(&event);
 }
 
+// http://crbug.com/646792
+// Disable pendingExitEvent_ logic in BaseView.
+- (void)mouseDown:(NSEvent*)theEvent {

[snake, 2016/09/22 18:22:39]

What about this MacViews specific fix, for now?

[tapted, 2016/09/23 07:10:21]

The code looks good, but I tried commenting this out and can't tell the
difference in behaviour. You explained earlier that the MouseExit could be
ignored - I tried dragging from the bookmarks bar and showing menus, but it
seems to work fine without this. I'm probably missing something.

As well as saying what the override is doing (as you have), can you explain in
the comment why it's necessary? There was a chromium-dev post about this
recently -- see
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/MY4e_wJY6yY/cOVLt...

The CL description needs to be updated too.

[snake, 2016/09/26 10:28:48]

hm.. I will investigate this. I did remove this for now.

Done.

+  [self mouseEvent:theEvent];
+}
+
 // NSView implementation.
 
 - (BOOL)acceptsFirstResponder {