Closed
Bug 1416250
Opened 8 years ago
Closed 8 years ago
Remove conditional catch consumers in dom/.
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla59
People
(Reporter: arai, Assigned: arai)
References
Details
Attachments
(1 file)
|
8.44 KB,
patch
|
billm
:
review+
|
Details | Diff | Splinter Review |
non-standard conditional catch syntax is going to be removed in bug 1228841.
there are 2 cases for replacement.
case 1: only conditional catch
from:
try {
A
} catch (e if cond) {
B
}
to
try {
A
} catch (e) {
if (!cond) {
throw e;
}
B
}
case 2: conditional catch and unconditional catch
from:
try {
A
} catch (e if cond) {
B
} catch {
C
}
to
try {
A
} catch (e) {
if (cond) {
B
} else {
C
}
}
Updated•8 years ago
|
Priority: -- → P3
| Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8928550 -
Flags: review?(wmccloskey)
Attachment #8928550 -
Flags: review?(wmccloskey) → review+
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/74ced70141b3
Remove conditional catch consumers in dom/. r=billm
| Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/74ced70141b30f7485a79ac69de4c64f8c462536
Bug 1416250 - Remove conditional catch consumers in dom/. r=billm
Comment 4•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Updated•8 years ago
|
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•