Browse Source

fix: only notify mouse-up on mouse-up (#24240)

Shelley Vohr 4 years ago
parent
commit
acb64a2d3c
1 changed files with 4 additions and 4 deletions
  1. 4 4
      shell/browser/ui/tray_icon_cocoa.mm

+ 4 - 4
shell/browser/ui/tray_icon_cocoa.mm

@@ -131,10 +131,6 @@
 }
 
 - (void)handleClickNotifications:(NSEvent*)event {
-  trayIcon_->NotifyMouseUp(
-      gfx::ScreenPointFromNSPoint([event locationInWindow]),
-      ui::EventFlagsFromModifiers([event modifierFlags]));
-
   // If we are ignoring double click events, we should ignore the `clickCount`
   // value and immediately emit a click event.
   BOOL shouldBeHandledAsASingleClick =
@@ -172,6 +168,10 @@
 - (void)mouseUp:(NSEvent*)event {
   [[statusItem_ button] highlight:NO];
 
+  trayIcon_->NotifyMouseUp(
+      gfx::ScreenPointFromNSPoint([event locationInWindow]),
+      ui::EventFlagsFromModifiers([event modifierFlags]));
+
   [self handleClickNotifications:event];
 }