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

Bug 1952900 - improve resilience of tab group removal in browser_tab_groups_closed.js. r=jswinarton,sessionstore-reviewers,tabbrowser-reviewers

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D247115
This commit is contained in:
DJ
2025-05-01 20:47:40 +00:00
committed by dwalker@mozilla.com
parent 13221f07d4
commit 1a51ae6a74
4 changed files with 56 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
browser/components/sessionstore/test/browser_tab_groups_closed.js
View File

@@ -1,5 +1,11 @@
"use strict";
const lazy = {};
ChromeUtils.defineESModuleGetters(lazy, {
TabGroupTestUtils: "resource://testing-common/TabGroupTestUtils.sys.mjs",
});
const ORIG_STATE = SessionStore.getBrowserState();
add_task(async function test_closingTabGroupAddsClosedGroup() {
@@ -25,7 +31,7 @@ add_task(async function test_closingTabGroupAddsClosedGroup() {
"Window state starts with no closed groups"
);
await win.gBrowser.removeTabGroup(group);
await lazy.TabGroupTestUtils.removeTabGroup(group);
windowState = ss.getWindowState(win).windows[0];
Assert.equal(
@@ -66,7 +72,7 @@ add_task(async function test_closedTabGroupSkipsNotWorthSavingTabs() {
"Window state starts with no closed groups"
);
await win.gBrowser.removeTabGroup(group);
await lazy.TabGroupTestUtils.removeTabGroup(group);
windowState = ss.getWindowState(win).windows[0];
Assert.equal(
@@ -128,9 +134,7 @@ add_task(async function test_closedTabCountsRespectTabGroups() {
})
);
let removePromise = BrowserTestUtils.waitForEvent(group, "TabGroupRemoved");
win.gBrowser.removeTabGroup(group);
await removePromise;
await lazy.TabGroupTestUtils.removeTabGroup(group);
Assert.equal(
SessionStore.getClosedTabCount(),
@@ -169,7 +173,7 @@ add_task(async function test_purgingSessionHistoryClearsClosedTabGroups() {
await TabStateFlusher.flush(tab.linkedBrowser);
let group = win.gBrowser.addTabGroup([tab]);
await win.gBrowser.removeTabGroup(group);
await lazy.TabGroupTestUtils.removeTabGroup(group);
let windowState = ss.getWindowState(win).windows[0];
Assert.equal(