Bug 226589 – Make WebIDBServer use WorkQueue instead of Thread

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Sihui Liu]

...

[Sihui Liu]

Created attachment 430480 [details]
Patch

[Chris Dumez]

Comment on attachment 430480 [details]
Patch

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

> Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp:67
> +        auto origins = m_server->getOrigins();

Why?

> Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp:-394
> -    tryClose();

Why don't we call this anymore? This is the kind of thing function-level changelogs are for.

[Sihui Liu]

Comment on attachment 430480 [details]
Patch

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

>> Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp:67
>> +        auto origins = m_server->getOrigins();
> 
> Why?

Oops should remove

>> Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp:-394
>> -    tryClose();
> 
> Why don't we call this anymore? This is the kind of thing function-level changelogs are for.

This was added to aggressively destroy WebIDBServer to ensure thread exits. It's possible a new connection is immediately created (web page is reloaded after web process crashes) after we close WebIDBServer, and we will create a new WebIDBServer. In this case both servers may access the same database on different work queues, unless we wait until the old WebIDBServer is destroyed before creating new one. To make things easy, we can just keep the WebIDBServer until session is destroyed.

Will add to changelog.

[Sihui Liu]

Created attachment 430498 [details]
Patch for landing

[EWS]

Tools/Scripts/svn-apply failed to apply attachment 430498 [details] to trunk.
Please resolve the conflicts and upload a new patch.

[Sihui Liu]

Created attachment 430546 [details]
Patch for landing

[EWS]

Committed r278449 (238469@main): <https://commits.webkit.org/238469@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/78856484>