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.

.eslintrc.js

@@ -187,6 +187,10 @@ module.exports = {
         property: 'notEqual',
         message: 'Use `assert.notStrictEqual()` rather than `assert.notEqual()`.',
       },
+      {
+        property: 'webcrypto',
+        message: 'Use `globalThis.crypto`.',
+      },
       {
         property: '__defineGetter__',
         message: '__defineGetter__ is deprecated.',
@@ -220,12 +224,6 @@ module.exports = {
         selector: "CallExpression[callee.name='isNaN']",
         message: 'Use Number.isNaN() instead of the global isNaN() function.',
       },
-      {
-        // 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`.',
-      },
     ],
     'no-self-compare': 'error',
     'no-template-curly-in-string': 'error',

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);