Closed
Bug 592801
Opened 14 years ago
Closed 14 years ago
IndexedDB: IDBRequest rework, make everything an event target
Categories
(Core :: Storage: IndexedDB, defect)
Core
Storage: IndexedDB
Tracking
()
RESOLVED
FIXED
People
(Reporter: bent.mozilla, Assigned: bent.mozilla)
References
Details
Attachments
(1 file, 1 obsolete file)
63.69 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•14 years ago
|
||
This makes everything an event listener, removes abort from IDBRequest, and simplifies event target creation.
Attachment #471217 -
Flags: review?(jonas)
Assignee | ||
Updated•14 years ago
|
Summary: IndexedDB: IDBRequest rework, make everything an event listener → IndexedDB: IDBRequest rework, make everything an event target
Attachment #471217 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 2•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/bb3b8e8f2b79
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•14 years ago
|
||
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
Assignee | ||
Comment 4•14 years ago
|
||
(In reply to comment #3) > http://hg.mozilla.org/mozilla-central/rev/c9f69a3d43af I meant http://hg.mozilla.org/mozilla-central/rev/6c0203305272
Assignee | ||
Comment 5•14 years ago
|
||
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.
Assignee | ||
Comment 6•14 years ago
|
||
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
Component: DOM → DOM: IndexedDB
You need to log in
before you can comment on or make changes to this bug.
Description
•