diff --git a/patches/chromium/render_widget_host_view_mac.patch b/patches/chromium/render_widget_host_view_mac.patch index ec57de9210656..8ea35e8158912 100644 --- a/patches/chromium/render_widget_host_view_mac.patch +++ b/patches/chromium/render_widget_host_view_mac.patch @@ -10,10 +10,10 @@ kinds of utility windows. Similarly for `disableAutoHideCursor`. Additionally, disables usage of some private APIs in MAS builds. diff --git a/content/app_shim_remote_cocoa/render_widget_host_view_cocoa.mm b/content/app_shim_remote_cocoa/render_widget_host_view_cocoa.mm -index f872e4ec47c157288980caafa11104af2d205abb..8cf97e1a4aa6ed95bbc6427877ec96c6200dc7f2 100644 +index f872e4ec47c157288980caafa11104af2d205abb..4f5c251da5904e1e478f3420a571b2edd8427486 100644 --- a/content/app_shim_remote_cocoa/render_widget_host_view_cocoa.mm +++ b/content/app_shim_remote_cocoa/render_widget_host_view_cocoa.mm -@@ -154,6 +154,11 @@ void ExtractUnderlines(NSAttributedString* string, +@@ -154,6 +154,15 @@ void ExtractUnderlines(NSAttributedString* string, } // namespace @@ -21,11 +21,15 @@ index f872e4ec47c157288980caafa11104af2d205abb..8cf97e1a4aa6ed95bbc6427877ec96c6 +- (BOOL)acceptsFirstMouse; +- (BOOL)disableAutoHideCursor; +@end ++ ++@interface NSView (ElectronCustomMethods) ++- (BOOL)shouldIgnoreMouseEvent; ++@end + // These are not documented, so use only after checking -respondsToSelector:. @interface NSApplication (UndocumentedSpeechMethods) - (void)speakString:(NSString*)string; -@@ -573,6 +578,9 @@ void ExtractUnderlines(NSAttributedString* string, +@@ -573,6 +582,9 @@ void ExtractUnderlines(NSAttributedString* string, } - (BOOL)acceptsFirstMouse:(NSEvent*)theEvent { @@ -35,7 +39,16 @@ index f872e4ec47c157288980caafa11104af2d205abb..8cf97e1a4aa6ed95bbc6427877ec96c6 return [self acceptsMouseEventsWhenInactive]; } -@@ -990,6 +998,10 @@ void ExtractUnderlines(NSAttributedString* string, +@@ -648,6 +660,8 @@ void ExtractUnderlines(NSAttributedString* string, + // its parent view. + BOOL hitSelf = NO; + while (view) { ++ if ([view respondsToSelector:@selector(shouldIgnoreMouseEvent)] && ![view shouldIgnoreMouseEvent]) ++ return NO; + if (view == self) + hitSelf = YES; + if ([view isKindOfClass:[self class]] && ![view isEqual:self] && +@@ -990,6 +1004,10 @@ void ExtractUnderlines(NSAttributedString* string, eventType == NSKeyDown && !(modifierFlags & NSCommandKeyMask); @@ -46,7 +59,7 @@ index f872e4ec47c157288980caafa11104af2d205abb..8cf97e1a4aa6ed95bbc6427877ec96c6 // We only handle key down events and just simply forward other events. if (eventType != NSKeyDown) { _hostHelper->ForwardKeyboardEvent(event, latency_info); -@@ -1766,9 +1778,11 @@ void ExtractUnderlines(NSAttributedString* string, +@@ -1766,9 +1784,11 @@ void ExtractUnderlines(NSAttributedString* string, // Since this implementation doesn't have to wait any IPC calls, this doesn't // make any key-typing jank. --hbono 7/23/09 // @@ -58,7 +71,7 @@ index f872e4ec47c157288980caafa11104af2d205abb..8cf97e1a4aa6ed95bbc6427877ec96c6 - (NSArray*)validAttributesForMarkedText { // This code is just copied from WebKit except renaming variables. -@@ -1777,7 +1791,10 @@ extern NSString* NSTextInputReplacementRangeAttributeName; +@@ -1777,7 +1797,10 @@ extern NSString* NSTextInputReplacementRangeAttributeName; initWithObjects:NSUnderlineStyleAttributeName, NSUnderlineColorAttributeName, NSMarkedClauseSegmentAttributeName, diff --git a/shell/browser/native_browser_view_mac.mm b/shell/browser/native_browser_view_mac.mm index 808d29856bcd2..76f7acf54bc48 100644 --- a/shell/browser/native_browser_view_mac.mm +++ b/shell/browser/native_browser_view_mac.mm @@ -34,10 +34,16 @@ - (BOOL)mouseDownCanMoveWindow { return NO; } -- (NSView*)hitTest:(NSPoint)aPoint { - // Pass-through events that don't hit one of the exclusion zones +- (BOOL)shouldIgnoreMouseEvent { + NSEventType type = [[NSApp currentEvent] type]; + return type != NSEventTypeLeftMouseDragged && + type != NSEventTypeLeftMouseDown; +} + +- (NSView*)hitTest:(NSPoint)point { + // Pass-through events that hit one of the exclusion zones for (NSView* exclusion_zones in [self subviews]) { - if ([exclusion_zones hitTest:aPoint]) + if ([exclusion_zones hitTest:point]) return nil; } @@ -45,6 +51,8 @@ - (NSView*)hitTest:(NSPoint)aPoint { } - (void)mouseDown:(NSEvent*)event { + [super mouseDown:event]; + if ([self.window respondsToSelector:@selector(performWindowDragWithEvent)]) { // According to Google, using performWindowDragWithEvent: // does not generate a NSWindowWillMoveNotification. Hence post one. @@ -65,7 +73,7 @@ - (void)mouseDown:(NSEvent*)event { self.initialLocation = [event locationInWindow]; } -- (void)mouseDragged:(NSEvent*)theEvent { +- (void)mouseDragged:(NSEvent*)event { if ([self.window respondsToSelector:@selector(performWindowDragWithEvent)]) { return; } @@ -125,15 +133,13 @@ - (void)mouseDragged:(NSEvent*)theEvent { [self.window setFrameOrigin:newOrigin]; } -// Debugging tips: -// Uncomment the following four lines to color DragRegionView bright red -// #ifdef DEBUG_DRAG_REGIONS -// - (void)drawRect:(NSRect)aRect -// { -// [[NSColor redColor] set]; -// NSRectFill([self bounds]); -// } -// #endif +// For debugging purposes only. +- (void)drawRect:(NSRect)aRect { + if (getenv("ELECTRON_DEBUG_DRAG_REGIONS")) { + [[[NSColor greenColor] colorWithAlphaComponent:0.5] set]; + NSRectFill([self bounds]); + } +} @end @@ -146,15 +152,13 @@ - (BOOL)mouseDownCanMoveWindow { return NO; } -// Debugging tips: -// Uncomment the following four lines to color ExcludeDragRegionView bright red -// #ifdef DEBUG_DRAG_REGIONS -// - (void)drawRect:(NSRect)aRect -// { -// [[NSColor greenColor] set]; -// NSRectFill([self bounds]); -// } -// #endif +// For debugging purposes only. +- (void)drawRect:(NSRect)aRect { + if (getenv("ELECTRON_DEBUG_DRAG_REGIONS")) { + [[[NSColor redColor] colorWithAlphaComponent:0.5] set]; + NSRectFill([self bounds]); + } +} @end