Closed Bug 1174954 Opened 9 years ago Closed 9 years ago

Remove the DOMError special-case in JS-implemented webidl

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla41
Tracking Flags:
Tracking Status
firefox41 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(3 files)

part 1. Stop throwing DOMErrors from JS-implemented webidl, since we can now throw content-side Errors instead
(deleted), patch
bholley
: review+
Details | Diff | Splinter Review
part 2. Remove the special-casing of DOMError in JS-implemented webidl code
(deleted), patch
bholley
: review+
Details | Diff | Splinter Review
part 3. Remove ReportJSExceptionFromJSImplementation, since it no longer does anything interesting
(deleted), patch
bholley
: review+
Details | Diff | Splinter Review
Now that bug 1173593 is fixed, we should be able to remove this hack and just have relevant Chrome callers throw an actual Error from the content compartment.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8623204 - Flags: review?(bobbyholley) → review+
Attachment #8623205 - Flags: review?(bobbyholley) → review+
Attachment #8623206 - Flags: review?(bobbyholley) → review+
https://hg.mozilla.org/mozilla-central/rev/66f5705f110d https://hg.mozilla.org/mozilla-central/rev/b6f23bd165db https://hg.mozilla.org/mozilla-central/rev/a4093b87ff57
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox41: affected → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: