Closed
Bug 755515
Opened 12 years ago
Closed 12 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•12 years ago
|
||
Please do explain :)
Assignee | ||
Comment 2•12 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•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f22f90356a87
Status: ASSIGNED → RESOLVED
Closed: 12 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
•