[webkit-changes] [WebKit/WebKit] 50f43b: REGRESSION (Safari 17.2): Does not respect mousemo...

Abrar Rahman Protyasha noreply at github.com
Thu Feb 22 00:10:50 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 50f43b731d2067d9ecc9be6d21ac8cbdefcd6e9e
      https://github.com/WebKit/WebKit/commit/50f43b731d2067d9ecc9be6d21ac8cbdefcd6e9e
  Author: Abrar Rahman Protyasha <a_protyasha at apple.com>
  Date:   2024-02-22 (Thu, 22 Feb 2024)

  Changed paths:
    M LayoutTests/fast/events/cancel-mousedown-and-drag-from-frame-to-other-frame.html
    M LayoutTests/fast/events/cancel-mousedown-and-drag-to-frame-expected.txt
    M LayoutTests/fast/events/cancel-mousedown-and-drag-to-frame.html
    M LayoutTests/fast/events/resources/mouse-drag-from-frame-target-subframe.html
    M Source/WebCore/page/EventHandler.cpp

  Log Message:
  -----------
  REGRESSION (Safari 17.2): Does not respect mousemove events in an iframe when the mouse is clicked from outside of an iframe
https://bugs.webkit.org/show_bug.cgi?id=269886
rdar://120540148

Reviewed by Wenson Hsieh.

In 269246 at main, we introduced a behavior change where initiating a drag
gesture from a cancelled mousedown event causes all mousemove events to
be routed to the originating frame till the drag gesture terminates with
a corresponding mouseup event. This was a web compatibility regression
since users expect to receive mouse events in the inner frame when we
are dragging the pointer over it.

Instead, this patch limits this mouse event target capturing behavior to
subframes only, and not the outer frame. This patch makes mouse event
targets independent of mousedown event cancellation when the pointer is
dragged out of a subframe. This change in behavior aligns us with other
major browser vendors' behavior.

Finally, we update cancel-mousedown-and-drag-to-frame.html since the new
behavior is essentially the opposite of what the test had been
expecting.

* LayoutTests/fast/events/cancel-mousedown-and-drag-from-frame-to-other-frame.html:
* LayoutTests/fast/events/cancel-mousedown-and-drag-to-frame-expected.txt:
* LayoutTests/fast/events/cancel-mousedown-and-drag-to-frame.html:
* LayoutTests/fast/events/resources/mouse-drag-from-frame-target-subframe.html:
* Source/WebCore/page/EventHandler.cpp:
(WebCore::EventHandler::handleMousePressEvent):

Canonical link: https://commits.webkit.org/275157@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list