Closed
Bug 2049520
Opened 2 months ago
Closed 2 months ago
Extract TabContextMenu code from tabbrowser.js
Categories
(Firefox :: Tabbed Browser, task, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
154 Branch
| Tracking | Status | |
|---|---|---|
| firefox154 | --- | fixed |
People
(Reporter: sfoster, Assigned: sfoster)
References
Details
(Whiteboard: [fidefe-splitview])
Attachments
(1 file)
tabbrowser.js is already over 11,000 lines of code. The TabContextMenu stuff is about 1000 lines of that. Its a window global, but doesn't need to be declared in the same object scope as the rest of the code in that file and should be easy to extract. I propose to do that before adding yet more code in bug 2031718.
|
||
Updated•2 months ago
|
|
Assignee | |
Comment 1•2 months ago
|
||
|
||
Updated•2 months ago
|
Assignee: nobody → sfoster
Status: NEW → ASSIGNED
Pushed by sfoster@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/affefa5af9cd
https://hg.mozilla.org/integration/autoland/rev/832e9fbf7a6f
Move the definition of TabContextMenu out into its own file. r=tabbrowser-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox154:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 154 Branch
|
||
Updated•1 month ago
|
QA Whiteboard: [qa-triage-done-c155/b154]
You need to log in
before you can comment on or make changes to this bug.
Description
•