url: improve isURLThis detection

PR-URL: #46866 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
nodejs · Mar 14, 2023 · b10fe58 · b10fe58
1 parent 08bf015
commit b10fe58
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -16,6 +16,7 @@ const {
   ObjectGetOwnPropertySymbols,
   ObjectGetPrototypeOf,
   ObjectKeys,
+  ObjectPrototypeHasOwnProperty,
   ReflectGetOwnPropertyDescriptor,
   ReflectOwnKeys,
   RegExpPrototypeSymbolReplace,
@@ -536,8 +537,7 @@ ObjectDefineProperties(URLSearchParams.prototype, {
 });
 
 function isURLThis(self) {
-  // TODO(@anonrig): Use ObjectPrototypeHasOwnProperty to avoid prototype look
-  return (self !== undefined && self !== null && self[context] !== undefined);
+  return self != null && ObjectPrototypeHasOwnProperty(self, context);
 }
 
 class URL {