corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1942809 - Consolidate tabs.length check by moving it into _recordSavedTabGroupState. r=dwalker

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D234956
This commit is contained in:
Dão Gottwald
2025-01-21 15:23:48 +00:00
parent b65702aa46
commit 78725d1a6b
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
browser/components/sessionstore/SessionStore.sys.mjs
View File

@@ -2545,9 +2545,7 @@ var SessionStoreInternal = {
// Add saved tab group references to saved tab group state.
for (let tabGroupToSave of newlySavedTabGroups.values()) {
if (tabGroupToSave.tabs.length) {
this._recordSavedTabGroupState(tabGroupToSave);
}
this._recordSavedTabGroupState(tabGroupToSave);
}
},
@@ -7792,9 +7790,7 @@ var SessionStoreInternal = {
tabGroup.tabs,
tabGroup.ownerGlobal
);
if (tabGroupState.tabs.length) {
this._recordSavedTabGroupState(tabGroupState);
}
this._recordSavedTabGroupState(tabGroupState);
},
/**
@@ -7802,7 +7798,10 @@ var SessionStoreInternal = {
* @returns {void}
*/
_recordSavedTabGroupState(savedTabGroupState) {
if (this.getSavedTabGroup(savedTabGroupState.id)) {
if (
!savedTabGroupState.tabs.length ||
this.getSavedTabGroup(savedTabGroupState.id)
) {
return;
}
this._savedGroups.push(savedTabGroupState);