Closed
Bug 2021639
Opened 6 months ago
Closed 6 months ago
Use private class fields in devtools/client/netmonitor/src/api.js
Categories
(DevTools :: Netmonitor, task)
DevTools
Netmonitor
Tracking
(firefox150 fixed)
RESOLVED
FIXED
150 Branch
| Tracking | Status | |
|---|---|---|
| firefox150 | --- | fixed |
People
(Reporter: cvl123abc, Assigned: cvl123abc)
References
(Depends on 1 open bug)
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/144.0.0.0 Safari/537.36
Steps to reproduce:
Reviewed devtools/client/netmonitor/src/api.js and identified underscore-prefixed properties and methods that are used as private class members.
Actual results:
The file currently uses underscore-prefixed properties and methods to indicate privacy.
Expected results:
Use JavaScript private class fields and methods (#) instead of underscore-prefixed properties.
| Assignee | ||
Comment 1•6 months ago
|
||
I’m working on this as part of the ongoing private fields migration (meta bug 2011387).
| Assignee | ||
Comment 2•6 months ago
|
||
Updated•6 months ago
|
Assignee: nobody → cvl123abc
Attachment #9550791 -
Attachment description: WIP: Bug 2021639 - Use private class fields in devtools/client/netmonitor/src/api.js. → Bug 2021639 - Use private class fields in devtools/client/netmonitor/src/api.js. r=nchevobbe
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Pushed by nchevobbe@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/ef688059c83c
https://hg.mozilla.org/integration/autoland/rev/f135bc1e98c7
Use private class fields in devtools/client/netmonitor/src/api.js. r=nchevobbe,devtools-reviewers
Comment 4•6 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
status-firefox150:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 150 Branch
Comment 5•6 months ago
|
||
| bugherder | ||
Updated•6 months ago
|
QA Whiteboard: [qa-triage-done-c151/b150]
You need to log in
before you can comment on or make changes to this bug.
Description
•