Bug 191671 – IndexedDB.IndexedDBTempFileSize API test times out with process prewarming enabled

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Chris Dumez]

IndexedDB.IndexedDBTempFileSize API test times out with process prewarming enabled:
Timeout

    TestWebKitAPI.IndexedDB.IndexedDBTempFileSize
        _RegisterApplication(), FAILED TO establish the default connection to the WindowServer, _CGSDefaultConnection() is NULL.
        ERROR: SQLite database failed to checkpoint: database table is locked
        ./platform/sql/SQLiteDatabase.cpp(130) : bool WebCore::SQLiteDatabase::open(const WTF::String &, bool)
        _RegisterApplication(), FAILED TO establish the default connection to the WindowServer, _CGSDefaultConnection() is NULL.
        Attempted to create a NetworkLoad with a session (id=2) that does not exist.

Testing completed, Exit status: 3

[Chris Dumez]

I know what's going on and have tests. Will upload a fix tomorrow.

[Radar WebKit Bug Importer]

<rdar://problem/46086062>

[Chris Dumez]

Created attachment 354944 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 354944 [details]
Patch

Clearing flags on attachment: 354944

Committed r238250: <https://trac.webkit.org/changeset/238250>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Darin Adler]

Comment on attachment 354944 [details]
Patch

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

> Source/WebKit/UIProcess/WebProcessPool.cpp:571
> +    // Make sure the network process knowns about all the sessions that have been registered before it started.

Typo: "knowns" instead of "knows".

[Chris Dumez]

(In reply to Darin Adler from comment #6)
> Comment on attachment 354944 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=354944&action=review
> 
> > Source/WebKit/UIProcess/WebProcessPool.cpp:571
> > +    // Make sure the network process knowns about all the sessions that have been registered before it started.
> 
> Typo: "knowns" instead of "knows".

Fixed in <https://trac.webkit.org/changeset/238284>, thanks.