RESOLVED FIXED 232166
[ iOS15 EWS ] TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequest tests are timing out 
https://bugs.webkit.org/show_bug.cgi?id=232166
Summary [ iOS15 EWS ] TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequest tests are t...
Eric Hutchison
Reported 2021-10-22 12:48:34 PDT
TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsNonAppInitiated TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsAppInitiated are timing out on iOS15 EWS. History: https://results.webkit.org/?suite=api-tests&suite=api-tests&test=TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsAppInitiated&test=TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsNonAppInitiated Build: https://ews-build.webkit.org/#/builders/9/builds/57175 STDIO: https://ews-build.webkit.org/api/v2/logs/15245550/raw Timeout TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsAppInitiated 2021-10-22 06:20:37.016 TestWebKitAPI[5427:88588] Writing analzed variants. TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsNonAppInitiated 2021-10-22 06:21:10.058 TestWebKitAPI[5441:88922] Writing analzed variants. worker/0 TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsAppInitiated Timeout 2021-10-22 06:21:10.058 TestWebKitAPI[5441:88922] Writing analzed variants. worker/0 TestWebKitAPI.AppPrivacyReport.LoadSimulatedRequestIsNonAppInitiated Timeout 2021-10-22 06:21:43.587 TestWebKitAPI[5453:89357] Writing analzed variants. Will attempt repro locally.
Attachments
Patch (1.99 KB, patch)
2021-10-29 16:39 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (1.81 KB, patch)
2021-11-01 16:24 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (1.82 KB, patch)
2021-11-02 09:04 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (1.94 KB, patch)
2021-11-02 12:43 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (2.55 KB, patch)
2021-11-03 11:36 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (3.22 KB, patch)
2021-11-04 17:17 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (3.05 KB, patch)
2021-11-05 11:04 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (1.93 KB, patch)
2021-11-05 16:30 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (1.93 KB, patch)
2021-11-05 16:37 PDT, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (2.47 KB, patch)
2021-11-08 06:50 PST, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Show Obsolete (9) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 2 2021-10-22 13:19:33 PDT
<rdar://problem/84560948>
Ryan Haddad
Comment 3 2021-10-22 15:33:49 PDT
I have not yet been able to reproduce this locally, and trunk bots aren't hitting it either. iOS EWS is consistently seeing these tests time out, so I've disabled them temporarily with r284721.
Alex Christensen
Comment 4 2021-10-29 09:48:46 PDT
Re-enabled in r285034 Will watch bots to verify that everything is ok.
Eric Hutchison
Comment 5 2021-10-29 16:14:30 PDT
Timeout started again as soon as tests were re-enabled.
Alex Christensen
Comment 6 2021-10-29 16:22:12 PDT
https://ews-build.webkit.org/#/builders/9 Sure enough
Alex Christensen
Comment 7 2021-10-29 16:24:54 PDT
Re-disabled in r285060
Kate Cheney
Comment 8 2021-10-29 16:39:43 PDT
Created attachment 442880 [details] Patch
Kate Cheney
Comment 9 2021-10-29 16:40:07 PDT
(In reply to Kate Cheney from comment #8) > Created attachment 442880 [details] > Patch some experimental logging to see if EWS will tell us where it is timing out.
Kate Cheney
Comment 10 2021-11-01 16:24:56 PDT
Created attachment 443032 [details] Patch
Kate Cheney
Comment 11 2021-11-02 09:04:07 PDT
Created attachment 443096 [details] Patch
Kate Cheney
Comment 12 2021-11-02 12:43:55 PDT
Created attachment 443123 [details] Patch
Kate Cheney
Comment 13 2021-11-03 11:36:06 PDT
Created attachment 443218 [details] Patch
Kate Cheney
Comment 14 2021-11-04 17:17:43 PDT
Created attachment 443359 [details] Patch
Kate Cheney
Comment 15 2021-11-04 17:18:06 PDT
(In reply to Kate Cheney from comment #14) > Created attachment 443359 [details] > Patch Test to see if the order of test is causing it to fail, I moved it to a new location.
Kate Cheney
Comment 16 2021-11-05 11:04:27 PDT
Created attachment 443419 [details] Patch
Kate Cheney
Comment 17 2021-11-05 15:30:03 PDT
(In reply to Kate Cheney from comment #16) > Created attachment 443419 [details] > Patch Ok, changing the HTML seems to have fixed it, so something about loading the img src in the HTML seems to be causing a timeout.
Chris Dumez
Comment 18 2021-11-05 15:33:18 PDT
Comment on attachment 443419 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=443419&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/AppPrivacyReport.mm:713 > + // NSString *HTML = @"<html><head></head><body><img src='https://apple.com/'></img></body></html>"; Is this really trying to connect to https://apple.com/ on the Web? If so, that not OK. Tests should not connect to the internet.
Kate Cheney
Comment 19 2021-11-05 16:30:37 PDT
Created attachment 443456 [details] Patch
Kate Cheney
Comment 20 2021-11-05 16:32:34 PDT
Talked with Chris offline, hopefully using 127.0.0.1 and about:blank will fix the issue if potentially this is caused by bad internet connection or firewall.
Kate Cheney
Comment 21 2021-11-05 16:37:24 PDT
Created attachment 443457 [details] Patch
Kate Cheney
Comment 22 2021-11-08 06:50:59 PST
Created attachment 443541 [details] Patch
Alex Christensen
Comment 23 2021-11-08 09:29:55 PST
Comment on attachment 443541 [details] Patch wow
EWS
Comment 24 2021-11-08 10:05:29 PST
Committed r285411 (243968@main): <https://commits.webkit.org/243968@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 443541 [details].
Note You need to log in before you can comment on or make changes to this bug.