First inactive tab to be moved into another window loses laziness
Categories
(Firefox :: Tabbed Browser, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: Oriol, Assigned: Oriol)
References
Details
Attachments
(1 file)
-
Let a window have 3 tabs, activate the 1st one.
-
Enable session restore, restart Firefox, focus that window.
-
Press Shift and click the 2nd tab so that tabs 1,2 are multiselected with the 1st active.
-
Open browser console and check that the 2nd tab is lazy:
gBrowser.tabs[1].linkedPanel === ""
-
Right-click a multiselected tab, chose "Move Tabs > Move to New Window"
-
Close browser console, focus the new window, open browser console and enter
gBrowser.tabs[1].linkedPanel === ""
-
It should still be
true
but now it'sfalse
. The tab is no longer lazy.
Bug 1521346 fixes this problem except for the first inactive tab in the multiselection. That one is moved using window.openDialog instead of gBrowser.adoptTab, so it's still unlazified. Should be fixed here.
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
I guess the problem is that when the first tab is moved into the new window, at that point it's the active tab there, so it's loaded.
Moving the active tab of the old window first would avoid this problem, however this will change the selected tab in the old window, maybe to a lazy multiselected tab that is going to be moved afterwards.
So I guess a mitigation would be to check if, among the tabs to be moved, there is an inactive one which is not lazy, and move it first.
If all the inactive tabs are lazy, then the only option seems to open the new window with a blank tab, move the tabs, and close the blank tab.
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 2•2 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/51ffa59f1488
Preserve lazyness of the first inactive tab moved into a new window. r=jaws
Comment 4•2 years ago
|
||
Backed out for WR failures on outline-013.html
Backout link: https://hg.mozilla.org/integration/autoland/rev/6b8e4909d303024cef8f404fc1d2ae4f3ecce055
Push link: https://hg.mozilla.org/integration/autoland/rev/51ffa59f14882531251a889b57ed73530c8997ac
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=231176971&repo=autoland&lineNumber=35536
Pushed by nbeleuzu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab719ccd5501 Preserve lazyness of the first inactive tab moved into a new window. r=jaws CLOSED TREE
Comment 6•2 years ago
|
||
bugherder |
Assignee | ||
Comment 7•2 years ago
|
||
The failure was caused by bug 1531025.
Description
•