Bug 274480

Summary:	Update PublicSuffixStore::topPrivatelyControlledDomain() to take in a StringView
Product:	WebKit	Reporter:	Chris Dumez <cdumez>
Component:	WebCore Misc.	Assignee:	Chris Dumez <cdumez>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

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.