RESOLVED FIXED274480
Update PublicSuffixStore::topPrivatelyControlledDomain() to take in a StringView 
https://bugs.webkit.org/show_bug.cgi?id=274480
Summary Update PublicSuffixStore::topPrivatelyControlledDomain() to take in a StringView
Chris Dumez
Reported 2024-05-21 12:58:22 PDT
Update PublicSuffixStore::topPrivatelyControlledDomain() to take in a StringView instead of a String. Most call sites have a StringView and we don't ever need to construct a String if the host is already in the cache.
Attachments
Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2024-05-21 12:59:53 PDT
Pull request: https://github.com/WebKit/WebKit/pull/28866
EWS
Comment 2 2024-05-21 17:58:03 PDT
Committed 279092@main (3a59245dd5ef): <https://commits.webkit.org/279092@main> Reviewed commits have been landed. Closing PR #28866 and removing active labels.
Radar WebKit Bug Importer
Comment 3 2024-05-21 17:59:15 PDT
<rdar://problem/128507635>
Note You need to log in before you can comment on or make changes to this bug.