Closed Bug 1574192 Opened 4 months ago Closed 3 months ago

Add UI support for setting watchpoints

Categories

(DevTools :: Debugger, enhancement, P3)

enhancement

Tracking

(firefox71 fixed)

RESOLVED FIXED
Firefox 71
Tracking Status
firefox71 --- fixed

People

(Reporter: jlast, Assigned: bmiriam1230)

References

(Blocks 2 open bugs)

Details

(Keywords: dev-doc-needed, Whiteboard: [debugger-reserve])

Attachments

(1 obsolete file)

It should be possible to right click on a property and set or remove a watchpoint

  • "Pause on Get"
  • "Pause on Set"

The UI should show a breakpoint to the right of properties with a watchpoint

Priority: -- → P3
No longer blocks: 1574190
Blocks: dbg-71
Whiteboard: [debugger-mvp]
Assignee: nobody → bmiriam1230
Status: NEW → ASSIGNED
Whiteboard: [debugger-mvp] → [debugger-reserve]
Attachment #9091031 - Attachment description: Initial watchpoints front end commit → Bug 1574192 - Initial watchpoints front end commit. r=nchevobbe
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f26512d37077
Initial watchpoints front end commit. r=nchevobbe
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Attachment #9091031 - Attachment is obsolete: true

Hi there!

I am trying to document this, but I've looked all over the debugger in Fx 71 nightly, and can't figure out where to find this functionality. Can someone point me in the right direction, witha screenshot or other explanation? thanks!

Flags: needinfo?(jlaster)
Flags: needinfo?(bmiriam1230)

Sorry, tagging here has been unfortunate to suggest 71. DevEdition 71 still has the feature disabled (devtools.debugger.features.watchpoints), only 72 DE has it on by default (bug 1592718) and release is pending on QA sign off.

Hopefully it also helps to see the feature in action: https://www.dropbox.com/s/eeymzv3r3xb312c/demo-watchpoints.mp4?dl=0

Flags: needinfo?(jlaster)
Flags: needinfo?(bmiriam1230)

Harald, thanks for the video, that's helpful.

Chris, I'm moving this item into Fx72, in https://github.com/mdn/sprints/issues/2468

You need to log in before you can comment on or make changes to this bug.