Closed Bug 592801 Opened 9 years ago Closed 9 years ago

IndexedDB: IDBRequest rework, make everything an event target

Categories

(Core :: DOM: IndexedDB, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: bent.mozilla, Assigned: bent.mozilla)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Attached patch Patch, v1Splinter Review
This makes everything an event listener, removes abort from IDBRequest, and simplifies event target creation.
Attachment #471217 - Flags: review?(jonas)
Summary: IndexedDB: IDBRequest rework, make everything an event listener → IndexedDB: IDBRequest rework, make everything an event target
http://hg.mozilla.org/mozilla-central/rev/bb3b8e8f2b79
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Now that more things are cycle collected some threadsafety assertion (that I've still never been able to reproduce) is causing aborts. Pushed this to bulletproof everything. Hopefully it will stick.

http://hg.mozilla.org/mozilla-central/rev/c9f69a3d43af
Attached patch Backout patch, just in case (obsolete) — Splinter Review
Just in case (everything looks fine so far) here is a patch to back out this patch without needing to backout the others it landed with (UI for beta 6). Hoping this can just sit here.
Comment on attachment 473974 [details] [diff] [review]
Backout patch, just in case

Looks like this won't be necessary after all!
Attachment #473974 - Attachment is obsolete: true
Duplicate of this bug: 585471
You need to log in before you can comment on or make changes to this bug.