Bug 71129

Summary: IndexedDB: IDBDatabase should have a close pending field.
Product: WebKit Reporter: jochen
Component: WebCore Misc.Assignee: Joshua Bell <jsbell>
Status: RESOLVED FIXED    
Severity: Normal CC: alecflett, dgrogan, hans, jsbell, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description jochen 2011-10-28 12:01:47 PDT
IDBDatabase should have a close pending field, and nobody should send versionchange events to it while it's closing (see also http://code.google.com/p/chromium/issues/detail?id=100645)
Comment 1 Joshua Bell 2012-06-28 11:17:48 PDT
Created attachment 149976 [details]
Patch
Comment 2 Joshua Bell 2012-06-28 11:18:57 PDT
alecflett@ - can you take a look?

tony@ - r?
Comment 3 Joshua Bell 2012-06-28 11:27:04 PDT
Note that most chunks of the new layout test passed before and are redundant with other tests, but I wanted everything that exercised the flag in one place.

The bits that previously failed were the version change and delete, which exercise the case described in the ChangeLog.
Comment 4 Alec Flett 2012-06-28 15:47:32 PDT
Comment on attachment 149976 [details]
Patch

LGTM
Comment 5 WebKit Review Bot 2012-06-28 16:48:49 PDT
Comment on attachment 149976 [details]
Patch

Clearing flags on attachment: 149976

Committed r121488: <http://trac.webkit.org/changeset/121488>
Comment 6 WebKit Review Bot 2012-06-28 16:48:54 PDT
All reviewed patches have been landed.  Closing bug.