Open
Bug 836298
Opened 12 years ago
Updated 2 years ago
Conditional pause on exceptions
Categories
(DevTools :: Debugger, enhancement, P5)
DevTools
Debugger
Tracking
(Not tracked)
NEW
People
(Reporter: rcampbell, Unassigned)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
This would be an augmentation of the Pause on Exceptions feature in the Debugger. We would like a way to break on type of exception. E.g., break on StackOverflow.
Reporter | ||
Updated•12 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Comment 1•12 years ago
|
||
We can use the conditional breakpoints mechanism, after bug 812172 is done.
Depends on: 812172
Comment 2•12 years ago
|
||
If there is possible conditionally, Is there a reason not to break on StackOverflow under normal "break on exception" conditions?
Comment 3•12 years ago
|
||
It works fine for me:
http://jsbin.com/eninec/1
Checking the "Pause on exceptions" option and clicking the button causes the debugger to pause with an InternalError exception.
Reporter | ||
Updated•12 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Blocks: dbg-exception
Updated•10 years ago
|
Summary: Conditional Break on Exception → Conditional break on exception
Updated•10 years ago
|
Summary: Conditional break on exception → Conditional break on exceptions
Updated•10 years ago
|
Summary: Conditional break on exceptions → Conditional pause on exceptions
Comment hidden (mozreview-request) |
Comment 5•8 years ago
|
||
^ patch that implements the feature, together with "$scriptUrl" and "$exception" variables.
Assignee: nobody → rob
Status: NEW → ASSIGNED
Updated•8 years ago
|
Attachment #8790239 -
Flags: review?(jlong)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 8•8 years ago
|
||
mozreview-review |
Comment on attachment 8790239 [details]
Bug 836298 - Add conditional pause on exception
https://reviewboard.mozilla.org/r/78154/#review77426
Hey Rob,
I hate to do this but we are not investing in the old debugger anymore. The old test suite particularly is in bad shape and doesn't run very well on our build infrastructure because it's too heavy. We should avoid changing the old debugger because right now we know that it works and it's something that people can fallback on if the new debugger is broken (by flipping a pref).
Sorry that you did all this work without knowing this. If you'd like to contribute to the new debugger, see here: https://github.com/devtools-html/debugger.html We're early on in the process, so it may be a little while before we are ready for an advanced feature like this (we don't even have popups yet), but feel free to file an issue and we can help mentor.
Attachment #8790239 -
Flags: review?(jlong) → review-
Updated•7 years ago
|
Assignee: rob → nobody
Status: ASSIGNED → NEW
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•5 years ago
|
Type: defect → enhancement
Priority: P3 → P5
Updated•5 years ago
|
Blocks: dbg-control
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•