tools: move webcrypto into no-restricted-properties

Since eslint fixed eslint/eslint#16412 and we
are on eslint v8.57.0 so that we can take advantage of
no-restricted-properties rule for webcrypto.

eslint.config.mjs

@@ -9,7 +9,6 @@ import toolsConfig from './tools/eslint.config_partial.mjs';
 import {
   noRestrictedSyntaxCommonAll,
   noRestrictedSyntaxCommonLib,
-  noRestrictedSyntaxCommonTest,
   requireEslintTool,
   resolveEslintTool,
 } from './tools/eslint.config_utils.mjs';

test/parallel/test-global-webcrypto.js

@@ -7,7 +7,8 @@ if (!common.hasCrypto)
 const assert = require('assert');
 const crypto = require('crypto');
 
-/* eslint-disable no-restricted-syntax */
+/* eslint-disable no-restricted-properties, no-restricted-syntax */
+// TODO(zi): Remove no-restricted-syntax once we add no-restricted-properties to test/.eslintrc.yaml
 assert.strictEqual(globalThis.crypto, crypto.webcrypto);
 assert.strictEqual(Crypto, crypto.webcrypto.constructor);
 assert.strictEqual(SubtleCrypto, crypto.webcrypto.subtle.constructor);

tools/eslint.config_utils.mjs

@@ -24,12 +24,3 @@ export const noRestrictedSyntaxCommonLib = [
     message: '`setTimeout()` must be invoked with at least two arguments.',
   },
 ];
-
-export const noRestrictedSyntaxCommonTest = [
-  {
-    // TODO(@panva): move this to no-restricted-properties
-    // when https://github.com/eslint/eslint/issues/16412 is fixed.
-    selector: "Identifier[name='webcrypto']",
-    message: 'Use `globalThis.crypto`.',
-  },
-];