Summary: | Add new worker-specific tests for the async DB | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric U. <ericu> | ||||||||
Component: | DOM | Assignee: | Dumitru Daniliuc <dumi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Enhancement | CC: | dumi, ericu, levin | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 34992 | ||||||||||
Bug Blocks: | 34990 | ||||||||||
Attachments: |
|
Description
Eric U.
2010-03-30 13:10:50 PDT
I'll take this bug. I've already added 2 tests that shut down workers while they access the database. So I'll work on adding some tests that access the same DB in the page and on multiple workers at the same time. Created attachment 69424 [details]
patch #1: access the same DB in workers and the page
Created attachment 69425 [details] patch #2: terminate workers in the middle of DB operations These tests were initially added in http://trac.webkit.org/changeset/64313. Then that patch got reverted in http://trac.webkit.org/changeset/64334, because of a bug. Then the bug was fixed, and the patch was re-landed in http://trac.webkit.org/changeset/64384, but apparently I forgot to re-submit the tests. Comment on attachment 69424 [details] patch #1: access the same DB in workers and the page View in context: https://bugs.webkit.org/attachment.cgi?id=69424&action=review > LayoutTests/fast/workers/storage/use-same-database-in-page-and-workers.html:49 > + log("Unexpected worker message: " + event.data); It seems like it would be better if it didn't have to time out on failure. Created attachment 69427 [details]
patch #1: access the same DB in workers and the page
fixed.
|