LibJS: Update order of steps in CanonicalNumericIndexString

This is an editorial change in the ECMA-262 spec.

See: https://github.com/tc39/ecma262/commit/be5db32

1 files changed, 5 insertions, 5 deletions

diff --git a/Userland/Libraries/LibJS/Runtime/AbstractOperations.cpp b/Userland/Libraries/LibJS/Runtime/AbstractOperations.cpp
index fdc034aca0..55d28a4500 100644
--- a/Userland/Libraries/LibJS/Runtime/AbstractOperations.cpp
+++ b/Userland/Libraries/LibJS/Runtime/AbstractOperations.cpp
@@ -1178,12 +1178,12 @@ CanonicalIndex canonical_numeric_index_string(PropertyKey const& property_key, C
     if (endptr != argument.characters() + argument.length())
         return CanonicalIndex(CanonicalIndex::Type::Undefined, 0);
 
-    // 3. If SameValue(! ToString(n), argument) is false, return undefined.
-    if (n.to_string_without_side_effects() != argument)
-        return CanonicalIndex(CanonicalIndex::Type::Undefined, 0);
+    // 3. If SameValue(! ToString(n), argument) is true, return n.
+    if (n.to_string_without_side_effects() == argument)
+        return CanonicalIndex(CanonicalIndex::Type::Numeric, 0);
 
-    // 4. Return n.
-    return CanonicalIndex(CanonicalIndex::Type::Numeric, 0);
+    // 4. Return undefined.
+    return CanonicalIndex(CanonicalIndex::Type::Undefined, 0);
 }
 
 // 22.1.3.17.1 GetSubstitution ( matched, str, position, captures, namedCaptures, replacement ), https://tc39.es/ecma262/#sec-getsubstitution