Audit JS that sets `open=true` on anchors to see if that's still necessary after bug 1694285 lands
Categories
(Firefox :: Toolbars and Customization, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: mconley, Assigned: dao)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-cleanups])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
In bug 1694285, I made it so that open=true
is set on the nearest toolbarbutton on an anchorNode (or the anchorNode if no toolbarbutton is found) by panel.js when an anchored panel is opened.
dao noted that there are a few places that try to do the same thing elsewhere in the front-end code that are now made redundant because of bug 1694285. See https://phabricator.services.mozilla.com/D109939#3575029.
This bug is for checking to see if those places are redundant and if they can be removed.
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Comment 3•4 years ago
|
||
Backed out changeset 8d8aa8575c1d (Bug 1701306) for causing multiple bc failures.
Backout link: https://hg.mozilla.org/integration/autoland/rev/f85284243a8147658e980a507ca9624948f68926
Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&tochange=f85284243a8147658e980a507ca9624948f68926&fromchange=5c6a0c43dfd88552c96b89c11ad0d67846f202bc&searchStr=bc
Assignee | ||
Updated•4 years ago
|
Comment 5•4 years ago
|
||
Backed out for causing browser-chrome failures in browser_page_action_menu
Backout link: https://hg.mozilla.org/integration/autoland/rev/98fbdf08cf7a66f0230c5cf280edaa94d4ea9d63
Updated•4 years ago
|
Assignee | ||
Comment 6•4 years ago
|
||
This is green on try: https://treeherder.mozilla.org/jobs?repo=try&revision=9d3601de7432feaa04ad5c7900ee3e68fec22cc0
I'm guessing something around browser_page_action_menu.js and proton changed in the meantime.
Comment 8•4 years ago
|
||
bugherder |
Description
•