Closed
Bug 755515
Opened 13 years ago
Closed 13 years ago
Catch exceptions in error handlers/versionchange handlers
Categories
(Core :: Storage: IndexedDB, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: khuey, Assigned: khuey)
References
Details
Attachments
(1 file)
|
14.00 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
||
Comment 1•13 years ago
|
||
Please do explain :)
|
Assignee | |
Comment 2•13 years ago
|
||
We already did this for success events (including versionchange). I added code to catch exceptions from error events and some tests.
Attachment #636796 -
Flags: review?(bent.mozilla) → review+
Actually, I talked it over with Ben and I think we should check for exception before we check if the default action should be done.
Similar to how exceptions override return values.
|
|
Assignee | |
Comment 4•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in
before you can comment on or make changes to this bug.
Description
•