[Webkit-unassigned] [Bug 254751] New: Avoid calling hasAttributeWithoutSynchronization() and then attributeWithoutSynchronization() for performance in HTMLAnchorElement.cpp
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Mar 30 10:25:04 PDT 2023
https://bugs.webkit.org/show_bug.cgi?id=254751
Bug ID: 254751
Summary: Avoid calling hasAttributeWithoutSynchronization() and
then attributeWithoutSynchronization() for performance
in HTMLAnchorElement.cpp
Product: WebKit
Version: WebKit Nightly Build
Hardware: Unspecified
OS: Unspecified
Status: NEW
Severity: Normal
Priority: P2
Component: Forms
Assignee: webkit-unassigned at lists.webkit.org
Reporter: ahmad.saleem792 at gmail.com
CC: akeerthi at apple.com, cdumez at apple.com,
rniwa at webkit.org, wenson_hsieh at apple.com
Hi Team,
While going through Blink's commit, I came across another potential commit (already testing locally).
Blink Commit (only partial needs to be done in HTMLAnchorElement.cpp) - https://chromium.googlesource.com/chromium/blink/+/f28d0fba0e523c1669c48354a416a091c5f062e0
WebKit Source - https://github.com/WebKit/WebKit/blob/fbbdeface10baf1a91eea6a903a622f5f2abb119/Source/WebCore/html/HTMLAnchorElement.cpp#L360
____
Locally:
A const auto& pingValue = attributeWithoutSynchronization(pingAttr);
M if (pingValue.isNull() || !document().settings().hyperlinkAuditingEnabled())
return;
M SpaceSplitString pingURLs(pingValue, SpaceSplitString::ShouldFoldCase::No);
_____
Just wanted to raise if we have any complications or we can go ahead and do this.
Thanks!
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230330/75071d10/attachment.htm>
More information about the webkit-unassigned
mailing list