Close native tab group by middle click

Author: piroor

webextensions/common/Tab.js

@@ -2090,11 +2090,11 @@ export default class Tab {
   }
 
   get sanitized() {
-    if (!this.tab)
+    if (!this.raw)
       return {};
 
     const sanitized = {
-      ...this.tab,
+      ...this.raw,
       '$possibleInitialUrl': null,
       '$TST': null,
       '$exportedForAPI': null,

webextensions/common/tree-behavior.js

@@ -117,6 +117,9 @@ export function getParentTabOperationBehavior(tab, { context, byInternalOperatio
 
 export function getClosingTabsFromParent(tab, removeInfo = {}) {
   log('getClosingTabsFromParent: ', tab, removeInfo);
+  if (tab?.$TST.isNativeTabGroup) {
+    return tab.$TST.descendants;
+  }
   const closeParentBehavior = getParentTabOperationBehavior(tab, {
     ...removeInfo,
     context: Constants.kPARENT_TAB_OPERATION_CONTEXT_CLOSE,

webextensions/sidebar/mouse-event-listener.js

@@ -35,6 +35,7 @@ import {
   countMatched,
   configs,
   shouldApplyAnimation,
+  mapAndFilter,
   isMacOS,
 } from '/common/common.js';
 import * as ApiTabs from '/common/api-tabs.js';
@@ -719,17 +720,20 @@ async function handleDefaultMouseUpOnTab({ lastMousedown, tab, event } = {}) {
   }
 
   if (lastMousedown.detail.isMiddleClick) { // Ctrl-click doesn't close tab on Firefox's tab bar!
-    log(`onMouseUp: middle click on the tab ${tab.id}: `, lastMousedown.detail.targetType);
+    log(`onMouseUp: middle click on the tab ${tab.id}: targetType = `, lastMousedown.detail.targetType);
     if (lastMousedown.detail.targetType != 'tab') // ignore middle click on blank area
       return false;
     const tabs = TreeBehavior.getClosingTabsFromParent(tab, {
       byInternalOperation: true
     });
-    Sidebar.confirmToCloseTabs(tabs.map(tab => tab.$TST.sanitized))
+    log('tabs: ', tabs);
+    const sanitizedTabsToClose = mapAndFilter(tabs, tab => tab.$TST.isNativeTabGroup ? undefined : tab.$TST.sanitized);
+    log('sanitizedTabsToClose: ', sanitizedTabsToClose);
+    Sidebar.confirmToCloseTabs(sanitizedTabsToClose)
       .then(async confirmed => {
         if (!confirmed)
           return;
-        const tabIds = tabs.map(tab => tab.id);
+        const tabIds = sanitizedTabsToClose.map(tab => tab.id);
         await Scroll.tryLockPosition(tabIds, Scroll.LOCK_REASON_REMOVE);
         BackgroundConnection.sendMessage({
           type:   Constants.kCOMMAND_REMOVE_TABS_BY_MOUSE_OPERATION,