Bug 17609 - [IndexedDB] Events fired after transaction abort should not set active flag
Summary: [IndexedDB] Events fired after transaction abort should not set active flag
Status: RESOLVED FIXED
Alias: None
Product: WebAppsWG
Classification: Unclassified
Component: Indexed Database API (show other bugs)
Version: unspecified
Hardware: All All
: P2 normal
Target Milestone: ---
Assignee: This bug has no owner yet - up for the taking
QA Contact: public-webapps-bugzilla
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-26 20:48 UTC by Joshua Bell
Modified: 2012-07-15 09:20 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Bell 2012-06-26 20:48:46 UTC
At the point where a transaction is aborted, it's active flag becomes false - either implicitly (because the abort happens outside a callback) or explicitly due to a call to abort() ("...this method sets the transaction's active flag to false...")

In the steps for aborting a transaction, the any requests in the transaction's request list whose done flag is still false are run through sub-steps including:

"Fire an error event at the request. However when running these steps, ignore any request to run the steps for aborting a transaction as those steps are already running."

Which indicates the steps for "Fire an error event" are processed, which includes:

2. Set the active flag of transaction to true.

The clause in the steps for aborting a transaction should either be changed to read:

"Fire an error event at the request. However when running these steps, ignore any request to run the steps for aborting a transaction as those steps are already running, and do not set the active flag of the transaction."

Or the steps for aborting a transaction should drop the "fire an error event" reference and in-line the relevant steps instead (that'd be step 1 and parts of step 3)
Comment 1 Jonas Sicking 2012-07-15 09:20:38 UTC
http://dvcs.w3.org/hg/IndexedDB/rev/aab14b0951ca