Replace SAA non-fully active SecurityError with InvalidStateError

See discussion on the related pull request:
privacycg/storage-access#112

Bug: 1354256
Change-Id: Ie2ac7890ae1039feca28b63239f02e476070e813
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3926236
Reviewed-by: Dominic Farolino <dom@chromium.org>
Commit-Queue: Matt Reichhoff <mreichhoff@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1053043}

storage-access-api/requestStorageAccess.sub.window.js

@@ -48,12 +48,12 @@ if (topLevelDocument) {
     const description = "document.requestStorageAccess() call in a detached frame";
     // Can't use `promise_rejects_dom` here, since the error comes from the wrong global.
     return promise.then(t.unreached_func("Should have rejected: " + description), (e) => {
-      assert_equals(e.name, 'SecurityError', description);
+      assert_equals(e.name, 'InvalidStateError', description);
     });
   }, "[non-fully-active] document.requestStorageAccess() should not resolve when run in a detached frame");
 
   promise_test(t => {
-    return promise_rejects_dom(t, 'SecurityError', RunRequestStorageAccessViaDomParser(),
+    return promise_rejects_dom(t, 'InvalidStateError', RunRequestStorageAccessViaDomParser(),
      "document.requestStorageAccess() in a detached DOMParser result");
   }, "[non-fully-active] document.requestStorageAccess() should not resolve when run in a detached DOMParser document");