Bug 210676 - REGRESSION (r260278): TestWebKitAPI.Fullscreen.Delegate is timing out on macOS bots
Summary: REGRESSION (r260278): TestWebKitAPI.Fullscreen.Delegate is timing out on macO...
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Peng Liu
URL:
Keywords: InRadar
: 211237 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-04-17 15:43 PDT by Ryan Haddad
Modified: 2020-06-12 12:23 PDT (History)
5 users (show)

See Also:


Attachments
Patch (7.22 KB, patch)
2020-04-22 15:55 PDT, Jer Noble
no flags Details | Formatted Diff | Diff
Patch (7.31 KB, patch)
2020-04-22 16:42 PDT, Jer Noble
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2020-04-17 15:43:40 PDT
TestWebKitAPI.Fullscreen.Delegate is consistently timing out on macOS Debug bots, and appears to be flaky on macOS release bots.

Timeout

    TestWebKitAPI.Fullscreen.Delegate
        _RegisterApplication(), FAILED TO establish the default connection to the WindowServer, _CGSDefaultConnection() is NULL.

https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.Fullscreen.Delegate
Comment 1 Radar WebKit Bug Importer 2020-04-17 15:44:00 PDT
<rdar://problem/61953702>
Comment 2 Jer Noble 2020-04-22 15:55:12 PDT
Created attachment 397281 [details]
Patch
Comment 3 Jer Noble 2020-04-22 16:42:25 PDT
Created attachment 397288 [details]
Patch
Comment 4 Daniel Bates 2020-04-22 21:00:25 PDT
Comment on attachment 397288 [details]
Patch

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

Patch looks good. A better patch would audit all of the unit tests and fix this up. For future thought, design to prevent this kind of mistake or detect it.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/FullscreenDelegate.mm:89
> +    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 100, 100) configuration:configuration.get()]);

This ok as-is. No change needed. Optimal solution would use auto.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/FullscreenLayoutConstraints.mm:59
> +    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 100, 100) configuration:configuration.get()]);

Ditto.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm:119
> +    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 640, 480) configuration:configuration.get()]);

Ditto.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/TopContentInset.mm:62
> +    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 100, 100) configuration:configuration.get()]);

Ditto.
Comment 5 Jer Noble 2020-04-23 16:44:02 PDT
(In reply to Daniel Bates from comment #4)
> Comment on attachment 397288 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=397288&action=review
> 
> Patch looks good. A better patch would audit all of the unit tests and fix
> this up. For future thought, design to prevent this kind of mistake or
> detect it.

Indeed. This was the result of a minor audit itself.

> > Tools/TestWebKitAPI/Tests/WebKitCocoa/FullscreenDelegate.mm:89
> > +    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 100, 100) configuration:configuration.get()]);
> 
> This ok as-is. No change needed. Optimal solution would use auto.


Okay, for the sake of expediency, I'm not going to update these all to use auto at this time. 

Thanks!
Comment 6 EWS 2020-04-23 16:45:20 PDT
Committed r260606: <https://trac.webkit.org/changeset/260606>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 397288 [details].
Comment 7 Ryan Haddad 2020-04-23 21:18:07 PDT
(In reply to EWS from comment #6)
> Committed r260606: <https://trac.webkit.org/changeset/260606>
Sadly, this has not resolved the timeouts on the bots.
Comment 8 Jer Noble 2020-04-30 13:17:11 PDT
*** Bug 211237 has been marked as a duplicate of this bug. ***
Comment 9 Ryan Haddad 2020-04-30 13:27:08 PDT
Reopening since it isn't fixed.
Comment 10 Ryan Haddad 2020-05-06 12:12:41 PDT
Disabled the test in https://trac.webkit.org/changeset/261243/webkit to get the bots to green.
Comment 11 Ryan Haddad 2020-06-12 12:23:11 PDT
(In reply to Ryan Haddad from comment #10)
> Disabled the test in https://trac.webkit.org/changeset/261243/webkit to get
> the bots to green.
Specifically, I disabled the Fullscreen.Alert test since it was causing the subsequent test to fail.