Bug 198800

Summary: REGRESSION (r246103) [ Mojave+ WK1 ] Layout Test scrollbars/scrollbar-iframe-click-does-not-blur-content.html is timing out
Product: WebKit Reporter: Shawn Roberts <sroberts>
Component: Tools / TestsAssignee: Antoine Quint <graouts>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, graouts, thorton, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=198462
Attachments:
Description Flags
Patch thorton: review+

Description Shawn Roberts 2019-06-12 13:14:35 PDT
The following layout test is failing on Mojave+ WK1

scrollbars/scrollbar-iframe-click-does-not-blur-content.html

Probable cause:

After changes in r246103 the the above layout test is timing out every run on WK1 testers

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=scrollbars%2Fscrollbar-iframe-click-does-not-blur-content.html
Comment 1 Radar WebKit Bug Importer 2019-06-12 13:15:25 PDT
<rdar://problem/51679634>
Comment 2 Shawn Roberts 2019-06-12 13:22:17 PDT
Updated expectations in https://trac.webkit.org/changeset/246371/webkit
Comment 3 Antoine Quint 2019-06-13 02:58:48 PDT
We don't detect an NSScroller in a sub-frame due to the WebHTMLView's implementation of -[NSView hitTest:]. We need to use the real method when detecting which view we're trying to swizzle the event for.
Comment 4 Antoine Quint 2019-06-13 03:11:56 PDT
Created attachment 372042 [details]
Patch
Comment 5 Tim Horton 2019-06-13 10:36:34 PDT
Comment on attachment 372042 [details]
Patch

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

> Tools/ChangeLog:10
> +        which lets use use the default implementation and correctly returns an NSScroller in a sub-frame.

use use

> Tools/DumpRenderTree/mac/EventSendingController.mm:570
> +        view = [(WebHTMLView*)view _hitViewForEvent:event];

WebHTMLView *
Comment 6 Antoine Quint 2019-06-13 23:51:01 PDT
Committed r246429: <https://trac.webkit.org/changeset/246429>