gecko-profiler-interfaces.js shouldn't specify the "responsiveness" feature by default
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P3)
Tracking
(Not tracked)
People
(Reporter: julienw, Unassigned)
Details
Attachments
(1 obsolete file)
The "responsiveness" feature has been removed recently, but is still specified in gecko-profiler-interface.js. As a result this gives some unneeded logs in some tests.
Reporter | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Comment 2•5 years ago
|
||
The priority flag is not set for this bug.
:julienw, could you have a look please?
For more information, please visit auto_nag documentation.
Reporter | ||
Updated•5 years ago
|
Comment 3•5 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:julienw, could you have a look please?
For more information, please visit auto_nag documentation.
Reporter | ||
Comment 4•5 years ago
|
||
yes, I know :-)
Reporter | ||
Updated•5 years ago
|
Comment 5•5 years ago
|
||
Because this bug's Severity has not been changed from the default since it was filed, and it's Priority is P3
(Backlog,) indicating it has been triaged, the bug's Severity is being updated to S3
(normal.)
Comment 6•4 years ago
|
||
The phabricator review is still showing up in my bug review dashboards. Is the patch here still valid?
Updated•4 years ago
|
Reporter | ||
Comment 7•4 years ago
|
||
I believe it is, just it was difficult to context switch back to this. I'll make sure to land this soon.
Updated•2 years ago
|
Reporter | ||
Comment 9•2 years ago
|
||
The patch in this bug is too complex for what we want to do here. We only want to remove the "responsiveness" feature from the codebase. So I marked it abandoned. I also don't expect to work on this soon, but I believe this is still something we want to do. We may wait until Bug 1603246 to remove all references in one go (see also bug 1594566).
Description
•