Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm

Issue 2324593002: [Mac] Fix for security indicator drag&drop navigation (Closed)
Patch Set: Cleaned up Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm b/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
index 3b227f8bdecad182ef65e690c9565ee36011aa79..2c91df8e3542eead047659d50034b4bd58a07208 100644
--- a/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
+++ b/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
@@ -518,7 +518,11 @@ const CGFloat kAnimationDuration = 0.2;
// (URLDropTarget protocol)
- (NSDragOperation)draggingUpdated:(id<NSDraggingInfo>)sender {
- return [dropHandler_ draggingUpdated:sender];
+ bool canDropAtLocation =
+ [[self cell] canDropAtLocationInWindow:[sender draggingLocation]
+ ofView:self];
+ return canDropAtLocation ? [dropHandler_ draggingUpdated:sender]
shrike 2016/09/16 18:28:41 I think you need to add canDropAtLocation logic to
spqchan 2016/09/20 21:00:00 Done.
+ : NSDragOperationNone;
}
// (URLDropTarget protocol)

Powered by Google App Engine
This is Rietveld 408576698