buffer: use private properties for brand checks in File

PR-URL: #47154
Refs: #46904
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Author: KhafraDev

lib/internal/file.js

@@ -21,7 +21,6 @@ const {
 
 const {
   codes: {
-    ERR_INVALID_THIS,
     ERR_MISSING_ARGS,
   },
 } = require('internal/errors');
@@ -64,18 +63,10 @@ class File extends Blob {
   }
 
   get name() {
-    if (!this || !(#name in this)) {
-      throw new ERR_INVALID_THIS('File');
-    }
-
     return this.#name;
   }
 
   get lastModified() {
-    if (!this || !(#name in this)) {
-      throw new ERR_INVALID_THIS('File');
-    }
-
     return this.#lastModified;
   }
 

test/parallel/test-file.js

@@ -146,10 +146,15 @@ const { inspect } = require('util');
 
 {
   const getter = Object.getOwnPropertyDescriptor(File.prototype, 'name').get;
-  assert.throws(
-    () => getter.call(undefined), // eslint-disable-line no-useless-call
-    {
-      code: 'ERR_INVALID_THIS',
-    }
-  );
+
+  [
+    undefined,
+    null,
+    true,
+  ].forEach((invalidThis) => {
+    assert.throws(
+      () => getter.call(invalidThis),
+      TypeError
+    );
+  });
 }