Bug 195575 – [Synthetic Click] Dispatch mouseout soon after mouseup

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[zalan]

to dismiss the hover state.

[Radar WebKit Bug Importer]

<rdar://problem/48784946>

[zalan]

Created attachment 364296 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 364296 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=364296&action=review

> Source/WebCore/ChangeLog:5
> +        <rdar://problem/48784946>

Use the other radar.

> Source/WebCore/page/ios/EventHandlerIOS.mm:518
> +void EventHandler::dispatchFakeMouseOut(const PlatformMouseEvent& platformMouseEvent)

We should use a standard term: fake or synthetic

> Source/WebCore/page/ios/EventHandlerIOS.mm:520
> +    updateMouseEventTargetNode(nullptr, platformMouseEvent, true);

Maybe an enum instead of 'true'

[zalan]

Created attachment 364324 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 364324 [details]
Patch

Clearing flags on attachment: 364324

Committed r242763: <https://trac.webkit.org/changeset/242763>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

This change may have caused layout test crashes on iOS Simulator:
https://build.webkit.org/results/Apple%20iOS%2012%20Simulator%20Release%20WK2%20(Tests)/r242763%20(3076)/fast/events/change-frame-focus-crash-log.txt

Application Specific Information:
CRASHING TEST: fast/events/can-click-element-on-page-with-active-pseudo-class-and-search-field.html
CoreSimulator 581.2 - Device: Managed 7 - Runtime: iOS 12.1 (16B91) - DeviceType: iPhone SE

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.WebKit              	0x000000010d958a8c WebKit::WebPage::completeSyntheticClick(WebCore::Node&, WebCore::FloatPoint const&, WTF::OptionSet<WebKit::WebEvent::Modifier>, WebCore::SyntheticClickType) + 706 (WebPageIOS.mm:643)
1   com.apple.WebKit              	0x000000010d9587a3 WebKit::WebPage::handleSyntheticClick(WebCore::Node&, WebCore::FloatPoint const&, WTF::OptionSet<WebKit::WebEvent::Modifier>) + 491 (WebPageIOS.mm:586)
2   com.apple.WebKit              	0x000000010d95b3c1 WebKit::WebPage::commitPotentialTap(WTF::OptionSet<WebKit::WebEvent::Modifier>, unsigned long long) + 327
3   com.apple.WebKit              	0x000000010da4256b WebKit::WebPage::didReceiveWebPageMessage(IPC::Connection&, IPC::Decoder&) + 6473 (WebPageMessageReceiver.cpp:422)
4   com.apple.WebKit              	0x000000010d67cb71 IPC::MessageReceiverMap::dispatchMessage(IPC::Connection&, IPC::Decoder&) + 127 (MessageReceiverMap.cpp:123)
5   com.apple.WebKit              	0x000000010d972706 WebKit::WebProcess::didReceiveMessage(IPC::Connection&, IPC::Decoder&) + 28 (WebProcess.cpp:729)
6   com.apple.WebKit              	0x000000010d66f622 IPC::Connection::dispatchMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >) + 108 (Connection.cpp:1012)
7   com.apple.WebKit              	0x000000010d672cb4 IPC::Connection::dispatchOneIncomingMessage() + 180 (Connection.cpp:1079)
8   JavaScriptCore                	0x0000000113ac61a7 WTF::RunLoop::performWork() + 231 (RunLoop.cpp:106)
9   JavaScriptCore                	0x0000000113ac6432 WTF::RunLoop::performWork(void*) + 34 (RunLoopCF.cpp:39)
10  com.apple.CoreFoundation      	0x000000010f6c9721 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17
11  com.apple.CoreFoundation      	0x000000010f6c8f93 __CFRunLoopDoSources0 + 243
12  com.apple.CoreFoundation      	0x000000010f6c363f __CFRunLoopRun + 1263
13  com.apple.CoreFoundation      	0x000000010f6c2e11 CFRunLoopRunSpecific + 625
14  com.apple.Foundation          	0x000000010d0e6322 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 277
15  com.apple.Foundation          	0x000000010d0e6492 -[NSRunLoop(NSRunLoop) run] + 76
16  libxpc.dylib                  	0x0000000110eb5812 _xpc_objc_main + 460
17  libxpc.dylib                  	0x0000000110eb7cbd xpc_main + 143
18  com.apple.WebKit              	0x000000010d767bfe WebKit::XPCServiceMain(int, char const**) + 403 (OSObjectPtr.h:69)
19  libdyld.dylib                 	0x0000000110ba2575 start + 1

[Ryan Haddad]

Flakiness dashboard data seems to confirm this suspicion. Rolling out this change because the crash is slowing down EWS.

[Ryan Haddad]

Reverted r242763 for reason:

Causes layout test crashes on iOS simulator

Committed r242777: <https://trac.webkit.org/changeset/242777>

[Ryan Haddad]

(In reply to Ryan Haddad from comment #8)
> Flakiness dashboard data seems to confirm this suspicion. Rolling out this
> change because the crash is slowing down EWS.
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fevents%2Fchange-frame-focus.html

[zalan]

Created attachment 364393 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 364393 [details]
Patch

Clearing flags on attachment: 364393

Committed r242798: <https://trac.webkit.org/changeset/242798>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Daniel Bates]

(In reply to WebKit Commit Bot from comment #12)
> Comment on attachment 364393 [details]
> Patch
> 
> Clearing flags on attachment: 364393
> 
> Committed r242798: <https://trac.webkit.org/changeset/242798>

This broke tapping to reveal context hyperlinks in Bugzilla patch review. See bug #195932.