Closed
Bug 2011839
Opened 2 days ago
Closed 10 hours ago
getAllDuplicateTabsToClose is quadratic on the number of tabs
Categories
(Firefox :: Tabbed Browser, defect, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
149 Branch
| Tracking | Status | |
|---|---|---|
| firefox149 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 2 open bugs)
Details
(Keywords: perf)
Attachments
(2 files)
See bug 1820171 comment 14 for a profile that shows massive time spent there just doing url comparisons.
| Assignee | ||
Comment 1•2 days ago
|
||
No behavior change.
Updated•2 days ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•2 days ago
|
||
On by default for quite a while.
Updated•2 days ago
|
Pushed by ealvarez@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/880bde06807a
https://hg.mozilla.org/integration/autoland/rev/db74d15e5f2a
Make getAllDuplicateTabsToClose faster with tons of tabs. r=dao,chutten,tabbrowser-reviewers
https://github.com/mozilla-firefox/firefox/commit/409e9501b8a9
https://hg.mozilla.org/integration/autoland/rev/4ea392bd4ea9
Remove duplicate tabs pref. r=tabbrowser-reviewers,jsudiaman
Comment 4•10 hours ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/db74d15e5f2a
https://hg.mozilla.org/mozilla-central/rev/4ea392bd4ea9
Status: ASSIGNED → RESOLVED
Closed: 10 hours ago
status-firefox149:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•