Hi Team, While going through Blink's commits, I came across another potential merge: Blink Commit - https://chromium.googlesource.com/chromium/blink/+/d905ef4a93adf27fc8f72cc333e28acf0e912cf6%5E%21/#F0 WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLMetaElement.cpp#163 ``` Check the name value earlier to avoid unnecessary comparisons. ``` This is to provide early return for 'name' & 'httpEquivValue' before doing other comparisons. So I think it is kind of optimization in process. Appreciate if someone can share input and if comment whether we can merge this without any test case. Thanks!
This is probably mergeable without testcase as it doesn't change behavior. Though some function names have changed: `fastGetAttribute` is `attributeWithoutSynchronization` I think `AtomicString` is just `AtomString`
(In reply to Tim Nguyen (:ntim) from comment #1) > This is probably mergeable without testcase as it doesn't change behavior. > > Though some function names have changed: > > `fastGetAttribute` is `attributeWithoutSynchronization` > > I think `AtomicString` is just `AtomString` Cool! Here is PR - https://github.com/WebKit/WebKit/pull/9080 Tested locally, it does compile and loading websites fine based on WebKit ToT and using --release.
Committed 259413@main (b66d5e5ea816): <https://commits.webkit.org/259413@main> Reviewed commits have been landed. Closing PR #9080 and removing active labels.
<rdar://problem/104684501>