Closed Bug 1581958 Opened 1 year ago Closed 1 year ago

Add a new "Break on..." submenu for the watchpoints context menu


(DevTools :: Debugger, enhancement, P3)

71 Branch


(firefox71 fixed)

Firefox 71
Tracking Status
firefox71 --- fixed


(Reporter: bmiriam1230, Assigned: bmiriam1230)




(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.100 Safari/537.36

Steps to reproduce:

Right click on an item in the Scopes pane to pull up the watchpoints context menu.

Actual results:

Currently there are two items: "Pause on get" and "Pause on set."

Expected results:

The watchpoint menu should have a "Break on..." menu item which creates the submenu items "Get" and "Set."

Blocks: dbg-watchpoints-m1
No longer blocks: dbg-watchpoints
Priority: -- → P3
Pushed by
Add sub-menu to watchpoints context menu r=jlast

Backed out changeset 4827f64f1e18 (bug 1581958) for failing at browser_misused_characters_in_strings.js on a CLOSED TREE.

Backout link:

Push with failures:

Log link:

Log snippet:
task 2019-09-19T20:45:13.251Z] 20:45:13 INFO - TEST-START | browser/base/content/test/static/browser_misused_characters_in_strings.js
[task 2019-09-19T20:45:13.667Z] 20:45:13 INFO - TEST-INFO | started process screencapture
[task 2019-09-19T20:45:13.836Z] 20:45:13 INFO - TEST-INFO | screencapture: exit 0
[task 2019-09-19T20:45:13.836Z] 20:45:13 INFO - Buffered messages logged at 20:45:13
[task 2019-09-19T20:45:13.837Z] 20:45:13 INFO - Entering test bound checkAllTheProperties
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - TEST-PASS | browser/base/content/test/static/browser_misused_characters_in_strings.js | Found 147 .properties files to scan for misused characters -
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - Buffered messages finished
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_misused_characters_in_strings.js | jar:file:///Users/cltbld/tasks/task_1568925716/build/application/!/chrome/en-US/locale/en-US/devtools/client/ with key=watchpoints.submenu has a misused ellipsis. Strings with an ellipsis should use the Unicode … character instead of three periods. -
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - Stack trace:
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochikit/content/browser-test.js:test_ok:1580
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_misused_characters_in_strings.js:testForError:145
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_misused_characters_in_strings.js:testForErrors:182
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_misused_characters_in_strings.js:checkAllTheProperties:222
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1350
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1385
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1213
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:805
[task 2019-09-19T20:45:13.838Z] 20:45:13 INFO - Leaving test bound checkAllTheProperties

Flags: needinfo?(bmiriam1230)


Got it, thanks. I pushed a change to fix this and it should work now.

Flags: needinfo?(bmiriam1230)
Pushed by
Add sub-menu to watchpoints context menu r=jlast
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Assignee: nobody → bmiriam1230
QA Whiteboard: [qa-71b-p2]
You need to log in before you can comment on or make changes to this bug.