Bug 224454 - REGRESSION (r275792): [ iOS wk2 ] TestWebKitAPI.IndexedDB.DatabaseProcessKill is a flakey timeout
Summary: REGRESSION (r275792): [ iOS wk2 ] TestWebKitAPI.IndexedDB.DatabaseProcessKill...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-04-12 14:37 PDT by Robert Jenner
Modified: 2021-04-13 09:54 PDT (History)
6 users (show)

See Also:


Attachments
Patch (5.18 KB, patch)
2021-04-12 20:12 PDT, Simon Fraser (smfr)
beidson: review+
ews-feeder: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Jenner 2021-04-12 14:37:54 PDT
TestWebKitAPI.IndexedDB.DatabaseProcessKill

is a flakey timeout on iOS wk2. 

HISTORY:
https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.IndexedDB.DatabaseProcessKill

Link to stdio file where timeout is noted:
https://build.webkit.org/#/builders/28/builds/910/steps/11/logs/stdio

It is flakey, but it does timeout more than it passes.
Comment 1 Radar WebKit Bug Importer 2021-04-12 14:38:11 PDT
<rdar://problem/76558602>
Comment 2 Robert Jenner 2021-04-12 16:01:06 PDT
This appears to be trivially reproducible with a release build, but does not appear to affect debug or production builds. As such, I did reproduce the failure at iOS Release ToT using the following test:

run-api-tests --ios-simulator TestWebKitAPI.IndexedDB.DatabaseProcessKill --child-process=50

Note that it is only occurring on release, and will pass on Production and Debug builds. I was also able to verify a regress point. The timeouts occur in r275792, and did not occur in 275791.

It appears this changeset caused the timeouts:

https://trac.webkit.org/changeset/275792/webkit
Comment 3 Simon Fraser (smfr) 2021-04-12 20:12:54 PDT
Created attachment 425825 [details]
Patch
Comment 4 Sihui Liu 2021-04-12 22:08:41 PDT
Comment on attachment 425825 [details]
Patch

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

> Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBDatabaseProcessKill.mm:85
>      [webView _test_waitForDidFinishNavigation];

Why not just replace this with TestWebKitAPI::Util::run(&receivedScriptMessage)?
Comment 5 Simon Fraser (smfr) 2021-04-13 08:57:39 PDT
Comment on attachment 425825 [details]
Patch

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

>> Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBDatabaseProcessKill.mm:85
>>      [webView _test_waitForDidFinishNavigation];
> 
> Why not just replace this with TestWebKitAPI::Util::run(&receivedScriptMessage)?

Maybe it could just be removed. I'll test.
Comment 6 Simon Fraser (smfr) 2021-04-13 09:54:48 PDT
https://trac.webkit.org/changeset/275890/webkit