Skip to content

Commit 669f747

Browse files
committed
fix: cleanup pervious tab urls map
- rename `removeTab` to `cleanupTabs`
1 parent 125a3b8 commit 669f747

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

src/background.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,18 +8,21 @@ let enabledTab = new Set<number>();
88
let tabData = new Map<number, Record<string, UpdatePayload['data']>>();
99
let pervTabUrl = new Map<number, string>();
1010

11-
const removeTab = (tabId: number) => {
11+
const cleanupCache = (tabId: number) => {
1212
if (enabledTab.has(tabId)) {
1313
enabledTab.delete(tabId);
1414
}
1515
if (tabData.has(tabId)) {
1616
tabData.delete(tabId);
1717
}
18+
if (pervTabUrl.has(tabId)) {
19+
pervTabUrl.delete(tabId);
20+
}
1821
};
1922

2023
chrome.tabs.onUpdated.addListener((tabId, _, tab) => {
2124
if (pervTabUrl.get(tabId)! !== tab.url) {
22-
removeTab(tabId);
25+
cleanupCache(tabId);
2326
}
2427
pervTabUrl.set(tabId, tab.url!);
2528
addExtensionMessageListener((message) => {
@@ -49,7 +52,7 @@ chrome.tabs.onUpdated.addListener((tabId, _, tab) => {
4952
});
5053
});
5154

52-
chrome.tabs.onRemoved.addListener((tabId) => removeTab(tabId));
55+
chrome.tabs.onRemoved.addListener((tabId) => cleanupCache(tabId));
5356

5457
addExtensionOneTimeMessageListener((request, _, sendResponse) => {
5558
switch (request.type) {

0 commit comments

Comments
 (0)