[Webkit-unassigned] [Bug 251046] New: Change code order in HTMLMetaElement::process()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 23 15:33:40 PST 2023


            Bug ID: 251046
           Summary: Change code order in HTMLMetaElement::process()
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Media
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com

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.


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/20230123/49064d21/attachment.htm>

More information about the webkit-unassigned mailing list