Open Bug 836298 Opened 7 years ago Updated 7 months ago

Conditional pause on exceptions

Categories

(DevTools :: Debugger, enhancement, P5)

enhancement

Tracking

(Not tracked)

People

(Reporter: rcampbell, Unassigned)

References

(Blocks 1 open bug)

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.
OS: Mac OS X → All
Hardware: x86 → All
We can use the conditional breakpoints mechanism, after bug 812172 is done.
Depends on: 812172
If there is possible conditionally, Is there a reason not to break on StackOverflow under normal "break on exception" conditions?
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.
Priority: -- → P3
Summary: Conditional Break on Exception → Conditional break on exception
Summary: Conditional break on exception → Conditional break on exceptions
Summary: Conditional break on exceptions → Conditional pause on exceptions
^ patch that implements the feature, together with "$scriptUrl" and "$exception" variables.
Assignee: nobody → rob
Status: NEW → ASSIGNED
Attachment #8790239 - Flags: review?(jlong)
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-
Assignee: rob → nobody
Status: ASSIGNED → NEW
Product: Firefox → DevTools
Type: defect → enhancement
Priority: P3 → P5
Blocks: dbg-control
You need to log in before you can comment on or make changes to this bug.