Closed Bug 1643628 Opened 5 years ago Closed 1 year ago

When stepping the highlighted exception appears only after resuming

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stepan, Assigned: stepan)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

https://www.loom.com/share/4901a3b1d1f54691bb5c81a201365950

[Actual result]:

  • As shown in the screencast, the exception is highlighted only after resuming.

[Expected result]:

  • The exception is highlighted immediately after stepping over the expression that throws the exception.
Depends on: 1581708

Nicolas could this be potentially an issue with the 'resourceWatcher'? If not, we are getting the exception details in why.frameFinished.throw.preview in actions/pause/paused.js so maybe we could just addException from there to handle this case. Thanks.

Flags: needinfo?(nchevobbe)

I just checked and this isn't an issue with the resource watcher.
The console service does not emit the message until the debugger is resumed (so you don't get the error in the console until then too).

Maybe this could be something with how step over works? Would you know Jason?

Flags: needinfo?(nchevobbe) → needinfo?(jlaster)

Hmm, interesting. happy to dig into this if you get stuck stepan.

Flags: needinfo?(jlaster)

Highlights the exception immediately after stepping over the expression that throws the exception.
Depends on D78883

Assignee: nobody → stepanstava
Depends on: 1644250

The severity field is not set for this bug.
:jlast, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(jlaster)
Attachment #9155615 - Attachment description: Bug 1643628 - When stepping the highlighted exception appears only after resuming r=jlast → Bug 1643628 - When stepping the highlighted exception appears only after resuming r=davidwalsh
Severity: -- → S3
Priority: -- → P3
Depends on: 1659434
Flags: needinfo?(jlaster)

The bug assignee didn't login in Bugzilla in the last 7 months.
:bomsy, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: stepanstava → nobody
Flags: needinfo?(hmanilla)

This looks to be fixed now.

The highlighted exception now always appears (before, during and after stepping).
I confirmed using STR

  • Open https://preview-js-test.glitch.me/
  • Open debugger
  • Goto script2.js
  • Set breakpoint on line 2 inside function a()
  • Refresh the page
  • Step over a couple of times
  • Notice exception is highlighted before pause, during stepping and after
Status: UNCONFIRMED → RESOLVED
Closed: 1 year ago
Flags: needinfo?(hmanilla)
Resolution: --- → FIXED
Assignee: nobody → stepanstava
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: