RESOLVED FIXED274433
PublicSuffixStore::platformTopPrivatelyControlledDomain() should take in a StringView 
https://bugs.webkit.org/show_bug.cgi?id=274433
Summary PublicSuffixStore::platformTopPrivatelyControlledDomain() should take in a St...
Chris Dumez
Reported 2024-05-20 19:54:14 PDT
PublicSuffixStore::platformTopPrivatelyControlledDomain() should take in a StringView instead of a String. It keeps calling substring(), which is a lot more efficient on a StringView than a String.
Attachments
Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2024-05-20 19:55:42 PDT
Pull request: https://github.com/WebKit/WebKit/pull/28821
EWS
Comment 2 2024-05-20 22:44:00 PDT
Committed 279035@main (9dce2ade92d5): <https://commits.webkit.org/279035@main> Reviewed commits have been landed. Closing PR #28821 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.