<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>236571</bug_id>
          
          <creation_ts>2022-02-13 14:16:06 -0800</creation_ts>
          <short_desc>Consistenly use hasTagName instead of is&lt;&gt; in HTMLStackItem testing functions</short_desc>
          <delta_ts>2022-02-14 16:02:17 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>DOM</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Cameron McCormack (:heycam)">heycam</reporter>
          <assigned_to name="Cameron McCormack (:heycam)">heycam</assigned_to>
          <cc>cdumez</cc>
    
    <cc>changseok</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>sam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1841074</commentid>
    <comment_count>0</comment_count>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2022-02-13 14:16:06 -0800</bug_when>
    <thetext>No reason to use is&lt;&gt; for a couple of element names but hasTagName for all others.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841075</commentid>
    <comment_count>1</comment_count>
      <attachid>451843</attachid>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2022-02-13 14:19:04 -0800</bug_when>
    <thetext>Created attachment 451843
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841391</commentid>
    <comment_count>2</comment_count>
      <attachid>451843</attachid>
    <who name="Sam Weinig">sam</who>
    <bug_when>2022-02-14 13:26:42 -0800</bug_when>
    <thetext>Comment on attachment 451843
Patch

Looks like this was done intentionally way back in http://trac.webkit.org/changeset/152440/webkit, but also seems like it was not the right choice.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841456</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-02-14 16:01:17 -0800</bug_when>
    <thetext>Committed r289777 (247245@main): &lt;https://commits.webkit.org/247245@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 451843.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841458</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-02-14 16:02:17 -0800</bug_when>
    <thetext>&lt;rdar://problem/88933767&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>451843</attachid>
            <date>2022-02-13 14:19:04 -0800</date>
            <delta_ts>2022-02-14 16:01:20 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-236571-20220214091903.patch</filename>
            <type>text/plain</type>
            <size>2254</size>
            <attacher name="Cameron McCormack (:heycam)">heycam</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg5Njg5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYjlhYjQ5MjZkZDcxMzg4
MWQzNjY1NGZjZTE5Yzk0N2UyOTNjMWI0MS4uNzk0NGU4NGY2OWM3Mzc3OGU4MDIxYTEzNTBkYzFl
MDY5ZDkyOWNlNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDIyLTAyLTEzICBDYW1l
cm9uIE1jQ29ybWFjayAgPGhleWNhbUBhcHBsZS5jb20+CisKKyAgICAgICAgQ29uc2lzdGVubHkg
dXNlIGhhc1RhZ05hbWUgaW5zdGVhZCBvZiBpczw+IGluIEhUTUxTdGFja0l0ZW0gdGVzdGluZyBm
dW5jdGlvbnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk
PTIzNjU3MQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IE5vIHJlYXNvbiB0byB1c2UgaXM8PiBmb3IgYSBjb3VwbGUgb2YgZWxlbWVudCBuYW1lcyBidXQg
aGFzVGFnTmFtZSBmb3IKKyAgICAgICAgYWxsIG90aGVycy4KKworICAgICAgICAqIGh0bWwvcGFy
c2VyL0hUTUxFbGVtZW50U3RhY2suY3BwOgorICAgICAgICAoV2ViQ29yZTo6SFRNTE5hbWVzOjpp
c1Njb3BlTWFya2VyKToKKyAgICAgICAgKFdlYkNvcmU6OkhUTUxOYW1lczo6aXNUYWJsZVNjb3Bl
TWFya2VyKToKKyAgICAgICAgKFdlYkNvcmU6OkhUTUxOYW1lczo6aXNTZWxlY3RTY29wZU1hcmtl
cik6CisKIDIwMjItMDItMTMgIENhbWVyb24gTWNDb3JtYWNrICA8aGV5Y2FtQGFwcGxlLmNvbT4K
IAogICAgICAgICBDYWNoZSBhbiBlbnRpcmUgYXR0cmlidXRlIFF1YWxpZmllZE5hbWUgd2hlbiBw
YXJzaW5nIEhUTUwsIG5vdCBqdXN0IGl0cyBsb2NhbCBuYW1lIEF0b21TdHJpbmcKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJDb3JlL2h0bWwvcGFyc2VyL0hUTUxFbGVtZW50U3RhY2suY3BwIGIvU291
cmNlL1dlYkNvcmUvaHRtbC9wYXJzZXIvSFRNTEVsZW1lbnRTdGFjay5jcHAKaW5kZXggNmVmMTcx
MmI5ZDYzYjcxYWNiN2MxNjlmNjJiNDljODJmMzEwNTgxNC4uMjI5ZDU0ZjdkMGM3NDk4M2VjOGYy
YjJiNjhmYjExNTQ2MWE3OWI3MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvaHRtbC9wYXJz
ZXIvSFRNTEVsZW1lbnRTdGFjay5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvaHRtbC9wYXJzZXIv
SFRNTEVsZW1lbnRTdGFjay5jcHAKQEAgLTQ5LDcgKzQ5LDcgQEAgaW5saW5lIGJvb2wgaXNTY29w
ZU1hcmtlcihIVE1MU3RhY2tJdGVtJiBpdGVtKQogICAgICAgICB8fCBpdGVtLmhhc1RhZ05hbWUo
Y2FwdGlvblRhZykKICAgICAgICAgfHwgaXRlbS5oYXNUYWdOYW1lKG1hcnF1ZWVUYWcpCiAgICAg
ICAgIHx8IGl0ZW0uaGFzVGFnTmFtZShvYmplY3RUYWcpCi0gICAgICAgIHx8IGlzPEhUTUxUYWJs
ZUVsZW1lbnQ+KGl0ZW0ubm9kZSgpKQorICAgICAgICB8fCBpdGVtLmhhc1RhZ05hbWUodGFibGVU
YWcpCiAgICAgICAgIHx8IGl0ZW0uaGFzVGFnTmFtZSh0ZFRhZykKICAgICAgICAgfHwgaXRlbS5o
YXNUYWdOYW1lKHRoVGFnKQogICAgICAgICB8fCBpdGVtLmhhc1RhZ05hbWUoTWF0aE1MTmFtZXM6
Om1pVGFnKQpAQCAtNzQsNyArNzQsNyBAQCBpbmxpbmUgYm9vbCBpc0xpc3RJdGVtU2NvcGVNYXJr
ZXIoSFRNTFN0YWNrSXRlbSYgaXRlbSkKIAogaW5saW5lIGJvb2wgaXNUYWJsZVNjb3BlTWFya2Vy
KEhUTUxTdGFja0l0ZW0mIGl0ZW0pCiB7Ci0gICAgcmV0dXJuIGlzPEhUTUxUYWJsZUVsZW1lbnQ+
KGl0ZW0ubm9kZSgpKQorICAgIHJldHVybiBpdGVtLmhhc1RhZ05hbWUodGFibGVUYWcpCiAgICAg
ICAgIHx8IGl0ZW0uaGFzVGFnTmFtZSh0ZW1wbGF0ZVRhZykKICAgICAgICAgfHwgaXNSb290Tm9k
ZShpdGVtKTsKIH0KQEAgLTExMCw3ICsxMTAsNyBAQCBpbmxpbmUgYm9vbCBpc0J1dHRvblNjb3Bl
TWFya2VyKEhUTUxTdGFja0l0ZW0mIGl0ZW0pCiAKIGlubGluZSBib29sIGlzU2VsZWN0U2NvcGVN
YXJrZXIoSFRNTFN0YWNrSXRlbSYgaXRlbSkKIHsKLSAgICByZXR1cm4gIWlzPEhUTUxPcHRHcm91
cEVsZW1lbnQ+KGl0ZW0ubm9kZSgpKSAmJiAhaXM8SFRNTE9wdGlvbkVsZW1lbnQ+KGl0ZW0ubm9k
ZSgpKTsKKyAgICByZXR1cm4gIWl0ZW0uaGFzVGFnTmFtZShvcHRncm91cFRhZykgJiYgIWl0ZW0u
aGFzVGFnTmFtZShvcHRpb25UYWcpOwogfQogCiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>