fix: table select-all error when triggered twice with keyboard shortcut (#4415)

* fix(stately-selection): add keys match condition

* feat(aria-selection): add support for toggle state

* test(spectrum-table): double check selecting all via ctrl + A

* chore(aria-selection): rollback to select-all

Author: tianenpang

packages/@react-spectrum/table/test/Table.test.js

@@ -2347,6 +2347,13 @@ describe('TableView', function () {
         expect(onSelectionChange.mock.calls[0][0]).toEqual('all');
         checkRowSelection(rows.slice(1), true);
         checkSelectAll(tree, 'checked');
+
+        fireEvent.keyDown(getCell(tree, 'Bar 1'), {key: 'a', ctrlKey: true});
+
+        expect(onSelectionChange).toHaveBeenCalledTimes(1);
+        expect(onSelectionChange.mock.calls[0][0]).toEqual('all');
+        checkRowSelection(rows.slice(1), true);
+        checkSelectAll(tree, 'checked');
       });
 
       it('should deselect an item after selecting all', function () {

packages/@react-stately/selection/src/SelectionManager.ts

@@ -402,7 +402,7 @@ export class SelectionManager implements MultipleSelectionManager {
    * Selects all items in the collection.
    */
   selectAll() {
-    if (this.selectionMode === 'multiple') {
+    if (!this.isSelectAll && this.selectionMode === 'multiple') {
       this.state.setSelectedKeys('all');
     }
   }