Bug 1877574 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Also small note to check, from review https://phabricator.services.mozilla.com/D207408#7185455
```
 #positionContentMarkersExtension is called twice. Here are the 2 different traces:

first trace:

#positionContentMarkersExtension (resource://devtools/client/shared/sourceeditor/editor.js#867)
setPositionContentMarker (resource://devtools/client/shared/sourceeditor/editor.js#934)
componentDidUpdate (resource://devtools/client/debugger/src/components/Editor/ColumnBreakpoints.js#118)
[React/Redux]
addBreakpoint (resource://devtools/client/debugger/src/actions/breakpoints/modify.js#141)

second trace:

#positionContentMarkersExtension (resource://devtools/client/shared/sourceeditor/editor.js#867)
setPositionContentMarker (resource://devtools/client/shared/sourceeditor/editor.js#934)
componentDidUpdate (resource://devtools/client/debugger/src/components/Editor/ColumnBreakpoints.js#118)
[React/Redux]
Editor (resource://devtools/client/debugger/src/components/Editor/index.js#127)
extensions (resource://devtools/client/shared/sourceeditor/editor.js#679)
measure (resource://devtools/client/shared/sourceeditor/codemirror6/codemirror6.bundle.mjs#1)
measureScheduled (resource://devtools/client/shared/sourceeditor/codemirror6/codemirror6.bundle.mjs#1)

In this second trace, the update seems to be triggered by this.#updateListener(v).
```
Revision 1 by
on 
Also small note to confirm after this fix, from review https://phabricator.services.mozilla.com/D207408#7185455
```
 #positionContentMarkersExtension is called twice. Here are the 2 different traces:

first trace:

#positionContentMarkersExtension (resource://devtools/client/shared/sourceeditor/editor.js#867)
setPositionContentMarker (resource://devtools/client/shared/sourceeditor/editor.js#934)
componentDidUpdate (resource://devtools/client/debugger/src/components/Editor/ColumnBreakpoints.js#118)
[React/Redux]
addBreakpoint (resource://devtools/client/debugger/src/actions/breakpoints/modify.js#141)

second trace:

#positionContentMarkersExtension (resource://devtools/client/shared/sourceeditor/editor.js#867)
setPositionContentMarker (resource://devtools/client/shared/sourceeditor/editor.js#934)
componentDidUpdate (resource://devtools/client/debugger/src/components/Editor/ColumnBreakpoints.js#118)
[React/Redux]
Editor (resource://devtools/client/debugger/src/components/Editor/index.js#127)
extensions (resource://devtools/client/shared/sourceeditor/editor.js#679)
measure (resource://devtools/client/shared/sourceeditor/codemirror6/codemirror6.bundle.mjs#1)
measureScheduled (resource://devtools/client/shared/sourceeditor/codemirror6/codemirror6.bundle.mjs#1)

In this second trace, the update seems to be triggered by this.#updateListener(v).
```

Back to Bug 1877574 Comment 2