AK: Don't iterate through entire String for String::ends_with

Author: https://github.com/shannonbooth
Commit: 23f3ecbe98
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/4504
Reviewed-by: https://github.com/trflynn89 ✅

AK/String.cpp

@@ -352,14 +352,19 @@ bool String::starts_with_bytes(StringView bytes, CaseSensitivity case_sensitivit
 
 bool String::ends_with(u32 code_point) const
 {
+    ASSERT(is_unicode(code_point));
+
     if (is_empty())
         return false;
 
-    u32 last_code_point = 0;
+    Array<u8, 4> code_point_as_utf8;
-    for (auto it = code_points().begin(); it != code_points().end(); ++it)
+    size_t i = 0;
-        last_code_point = *it;
 
-    return last_code_point == code_point;
+    size_t code_point_byte_length = UnicodeUtils::code_point_to_utf8(code_point, [&](auto byte) {
+        code_point_as_utf8[i++] = static_cast<u8>(byte);
+    });
+
+    return ends_with_bytes(StringView { code_point_as_utf8.data(), code_point_byte_length });
 }
 
 bool String::ends_with_bytes(StringView bytes, CaseSensitivity case_sensitivity) const