Duplication of breakpoints in the browser toolbox
Categories
(DevTools :: Debugger, defect, P2)
Tracking
(Not tracked)
People
(Reporter: bomsy, Assigned: bomsy)
References
(Depends on 1 open bug, Blocks 2 open bugs)
Details
(Whiteboard: dt-fission-future, dt-perf-stability-mvp)
Attachments
(3 obsolete files)
Prefs:
- fission.autostart => true
- devtools.target-switching.server.enabled => true
- fission.bfcacheInParent => true
I have STRs for duplicated breakpoints, but with the Browser Toolbox.
Here are my STRs:
- open Firefox on any page (eg the default about:newtab)
- open the Multiprocess Browser Toolbox
- in the MBT, open the Debugger, find the source resource://devtools/server/actors/resources/console-messages.js
- set a breakpoint at L217 (in
prepareConsoleMessageForRemote, const result = WebConsoleUtils.cloneObject(message);
) - close the MBT, and reopen it
ER: You should only see one breakpoint
AR: There are 3 identical breakpoints listed in the breakpoints panel. Every time you restart the MBT there is an additional breakpoint.
Assignee | ||
Comment 1•3 years ago
|
||
Depends on D128742
Assignee | ||
Comment 2•3 years ago
|
||
Depends on D128743
Updated•3 years ago
|
Assignee | ||
Comment 3•3 years ago
|
||
Comment 4•3 years ago
|
||
Comment on attachment 9250361 [details]
Bug 1734768 - [devtools] Stop showing duplicates
Revision D130968 was moved to bug 1740980. Setting attachment 9250361 [details] to obsolete.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 5•3 years ago
|
||
Bomsy, is anything blocking us from completing this bug?
Assignee | ||
Comment 6•3 years ago
|
||
(In reply to Jan Honza Odvarko [:Honza] (always need-info? me) from comment #5)
Bomsy, is anything blocking us from completing this bug?
The patch for making pending breakpoints unique is likely not the way we want to go with moving to breakpoints per url.
But the second patch a can actually be completed and landed as it useful in showing expose the thread info about the breakpoint.
This will make it less confusing, at least till we fix the duplication it.
I'll update both patches.
Thanks
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Comment 7•3 years ago
|
||
Comment on attachment 9250185 [details]
Bug 1734768 - [devtools] Show the thread info in breakpoint list tooltip
Revision D130866 was moved to bug 1755960. Setting attachment 9250185 [details] to obsolete.
Assignee | ||
Updated•3 years ago
|
Comment 9•3 years ago
|
||
Additional STRs can be found in Bug 1759975 comment #0 and Bug 1759975 comment #1
Description
•