Bug 53945

Summary: openDatabaseSync() stop responding after too many call
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: WebCore Misc.Assignee: Benjamin Poulain <benjamin>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, abecsi, benjamin, commit-queue, eric, ossy, webkit.review.bot
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 54802    
Attachments:
Description Flags
Patch
none
Patch none

Description Benjamin Poulain 2011-02-07 14:37:03 PST
When using openDatabaseSync() repeatedly from a worker, it stops responding, not creating the database, neither calling the callback function.

To reproduce: 
./Tools/Scripts/run-webkit-tests --release --qt fast/workers/storage/open-database-inputs-sync.html --iterations 640

This is the reason of some flaky test of Qt. When a lot of tests using openDatabaseSync() end up in the same run, we get failures.

I have not yet tested on other WebKit ports than Qt.
Comment 1 Benjamin Poulain 2011-02-07 16:41:40 PST
The tests are simply going over quota. Every test allocate a database of estimatedSize 1024. After a few hundred test, the quota for origin is exceeded.

The test runner should probably drop the databases between tests.
Comment 2 Benjamin Poulain 2011-02-08 02:10:26 PST
Created attachment 81615 [details]
Patch

Current idea: delete all databases between tests so we can do as many tests as we want without hitting the quota.


I don't set in r? as I want to test on the mac port and get comments.
Comment 3 Andras Becsi 2011-02-08 02:16:36 PST
(In reply to comment #2)
> I don't set in r? as I want to test on the mac port and get comments.

If I remember correctly the EWS only test patches which have r? set.
Comment 4 Benjamin Poulain 2011-02-08 02:45:46 PST
More info, Ossy noticed we do not setup the quota size for Qt before dumpDatabaseQuota() is called. This should be investigated as well.
Comment 5 Benjamin Poulain 2011-02-08 04:13:12 PST
Created attachment 81619 [details]
Patch
Comment 6 WebKit Commit Bot 2011-02-08 06:01:24 PST
The commit-queue encountered the following flaky tests while processing attachment 81619 [details]:

http/tests/websocket/tests/handshake-error.html bug 53851 (author: abarth@webkit.org)
The commit-queue is continuing to process your patch.
Comment 7 WebKit Commit Bot 2011-02-08 06:04:39 PST
Comment on attachment 81619 [details]
Patch

Clearing flags on attachment: 81619

Committed r77926: <http://trac.webkit.org/changeset/77926>
Comment 8 WebKit Commit Bot 2011-02-08 06:04:43 PST
All reviewed patches have been landed.  Closing bug.
Comment 9 WebKit Review Bot 2011-02-08 07:07:55 PST
http://trac.webkit.org/changeset/77926 might have broken GTK Linux 32-bit Release and Qt Linux Release