Bug 225372 – [ BigSur Release ] TestWebKitAPI.IndexedDB.KillWebProcessWithOpenConnection is a constant failure

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Robert Jenner]

TestWebKitAPI.IndexedDB.KillWebProcessWithOpenConnection

is a constantly failing API on BigSur Release.

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

 /Volumes/Data/worker/bigsur-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WebProcessKillIDBCleanup.mm:129
        Expected equality of these values:
          @"Open Succeeded"
            Which is: "Open Succeeded"
          string3.get()
            Which is: "Open Failed"

[Radar WebKit Bug Importer]

<rdar://problem/77532291>

[Sihui Liu]

Created attachment 428408 [details]
Patch

[Chris Dumez]

Comment on attachment 428408 [details]
Patch

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

> Tools/TestWebKitAPI/Tests/WebKitCocoa/KillWebProcessWithOpenConnection-1.html:3
> +var db;

Why this?

> Tools/TestWebKitAPI/Tests/WebKitCocoa/KillWebProcessWithOpenConnection-1.html:7
> +    db = event.target.result;

why not use var here?

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WebProcessKillIDBCleanup.mm:41
> +static bool readyToContinue;

You don't need a global here.

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WebProcessKillIDBCleanup.mm:111
> +    readyToContinue = false;

__block bool readyToContinue = false;

[Sihui Liu]

Comment on attachment 428408 [details]
Patch

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

>> Tools/TestWebKitAPI/Tests/WebKitCocoa/KillWebProcessWithOpenConnection-1.html:7
>> +    db = event.target.result;
> 
> why not use var here?

Left for testing. Will move it here.

>> Tools/TestWebKitAPI/Tests/WebKitCocoa/WebProcessKillIDBCleanup.mm:41
>> +static bool readyToContinue;
> 
> You don't need a global here.

Will remove.

[Sihui Liu]

Created attachment 428412 [details]
Patch for landing

[EWS]

Committed r277392 (237648@main): <https://commits.webkit.org/237648@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 428412 [details].

[Chris Dumez]

Open Failed: UnknownError: Error creating or migrating Records table in database

[Sihui Liu]

(In reply to Chris Dumez from comment #7)
> Open Failed: UnknownError: Error creating or migrating Records table in
> database

Yes, this is not fixed so I filed https://bugs.webkit.org/show_bug.cgi?id=226124. Can you take a look?

[Ryan Haddad]

Current output:

    TestWebKitAPI.IndexedDB.KillWebProcessWithOpenConnection
        
        /Volumes/Data/worker/bigsur-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WebProcessKillIDBCleanup.mm:134
        Expected equality of these values:
          @"Open Succeeded"
            Which is: "Open Succeeded"
          string3.get()
            Which is: "Open Failed: UnknownError: Error creating new Records table (%i) - %s5database is locked"

https://build.webkit.org/#/builders/105/builds/1474/steps/11/logs/stdio

[Ryan Haddad]

This appears to have been resolved by <https://commits.webkit.org/238469@main>.