Bug 251046 – Change code order in HTMLMetaElement::process()

• Component: DOM
• Status: RESOLVED FIXED

[Ahmad Saleem]

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!

[Tim Nguyen (:ntim)]

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`

[Ahmad Saleem]

(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.

[EWS]

Committed 259413@main (b66d5e5ea816): <https://commits.webkit.org/259413@main>

Reviewed commits have been landed. Closing PR #9080 and removing active labels.

[Radar WebKit Bug Importer]

<rdar://problem/104684501>