<?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>231186</bug_id>
          
          <creation_ts>2021-10-04 11:53:36 -0700</creation_ts>
          <short_desc>Don&apos;t use namespace WebCore::XPath at global scope</short_desc>
          <delta_ts>2021-10-06 11:02:49 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</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>
          
          <blocked>231302</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Alex Christensen">achristensen</reporter>
          <assigned_to name="Alex Christensen">achristensen</assigned_to>
          <cc>basuke</cc>
    
    <cc>ross.kirsling</cc>
    
    <cc>sam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1800152</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2021-10-04 11:53:36 -0700</bug_when>
    <thetext>Don&apos;t use namespace WebCore::XPath at global scope</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1800155</commentid>
    <comment_count>1</comment_count>
      <attachid>440084</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2021-10-04 11:55:14 -0700</bug_when>
    <thetext>Created attachment 440084
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1800157</commentid>
    <comment_count>2</comment_count>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2021-10-04 11:57:30 -0700</bug_when>
    <thetext>I&apos;m checking right now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1800182</commentid>
    <comment_count>3</comment_count>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2021-10-04 12:47:35 -0700</bug_when>
    <thetext>In PlayStation port, it still has these compile errors:

C:/PSDev/webkit/Source/WebCore/platform/graphics\Path.h:140:58: error: expected &apos;)&apos;
    bool strokeContains(const FloatPoint&amp;, const Function&lt;void(GraphicsContext&amp;)&gt;&amp; strokeStyleApplier) const;
                                                         ^
C:/PSDev/webkit/Source/WebCore/platform/graphics\Path.h:140:24: note: to match this &apos;(&apos;
    bool strokeContains(const FloatPoint&amp;, const Function&lt;void(GraphicsContext&amp;)&gt;&amp; strokeStyleApplier) const;
                       ^
C:/PSDev/webkit/Source/WebCore/platform/graphics\Path.h:146:48: error: expected &apos;)&apos;
    FloatRect strokeBoundingRect(const Function&lt;void(GraphicsContext&amp;)&gt;&amp; strokeStyleApplier = { }) const;
                                               ^
C:/PSDev/webkit/Source/WebCore/platform/graphics\Path.h:146:33: note: to match this &apos;(&apos;
    FloatRect strokeBoundingRect(const Function&lt;void(GraphicsContext&amp;)&gt;&amp; strokeStyleApplier = { }) const;
                                ^
2 errors generated.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1800261</commentid>
    <comment_count>4</comment_count>
    <who name="Ross Kirsling">ross.kirsling</who>
    <bug_when>2021-10-04 15:46:45 -0700</bug_when>
    <thetext>Perhaps the issue is the remaining `using namespace`s at XPathGrammar.y:51?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1801190</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-10-06 11:00:09 -0700</bug_when>
    <thetext>Committed r283635 (242586@main): &lt;https://commits.webkit.org/242586@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 440084.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1801192</commentid>
    <comment_count>6</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-10-06 11:01:22 -0700</bug_when>
    <thetext>&lt;rdar://problem/83942126&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>440084</attachid>
            <date>2021-10-04 11:55:14 -0700</date>
            <delta_ts>2021-10-06 11:00:11 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-231186-20211004115513.patch</filename>
            <type>text/plain</type>
            <size>4627</size>
            <attacher name="Alex Christensen">achristensen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjgzNTA0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDhjN2VhNmVhNDQ4Nzg1
NmMzYjkzMDEzOWE1Y2ZkMTdlYTcxZmRlOC4uODRhMWFiYWQ1Njk0NmU5NTNhOGQzMWY0Y2U0YmUw
ZDkwYmM1M2ZlOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDIxLTEwLTA0ICBBbGV4
IENocmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgRG9uJ3Qg
dXNlIG5hbWVzcGFjZSBXZWJDb3JlOjpYUGF0aCBhdCBnbG9iYWwgc2NvcGUKKyAgICAgICAgaHR0
cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzMTE4NgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEl0IGNhdXNlcyBwcm9ibGVtcyB3
aXRoIHVuaWZpZWQgYnVpbGRzLCB3aGljaCBnZXQgY29uZnVzZWQgYmV0d2VlbiBXZWJDb3JlOjpY
UGF0aDo6RnVuY3Rpb24gYW5kIFdURjo6RnVuY3Rpb24uCisKKyAgICAgICAgKiB4bWwvWFBhdGhH
cmFtbWFyLmNwcDoKKyAgICAgICAgKiB4bWwvWFBhdGhHcmFtbWFyLmg6CisgICAgICAgICogeG1s
L1hQYXRoR3JhbW1hci55OgorICAgICAgICAqIHhtbC9YUGF0aFBhcnNlci5jcHA6CisKIDIwMjEt
MTAtMDQgIEthdGUgQ2hlbmV5ICA8a2F0aGVyaW5lX2NoZW5leUBhcHBsZS5jb20+CiAKICAgICAg
ICAgVW5yZXZpZXdlZCBtYWNjYXRhbHlzdCBidWlsZCBmaXguCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS94bWwvWFBhdGhHcmFtbWFyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3htbC9YUGF0aEdy
YW1tYXIuY3BwCmluZGV4IDA4NzVhNWVjZmZhOWM2N2U2MGMxYWY2NmM5MGM3MDk1ZmU3YmQxMDQu
LjBkNDg4NjdhOThmNmU0MzEyODJmMzIwNGRlNTBkNmYyMDk0MDM3NDMgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJDb3JlL3htbC9YUGF0aEdyYW1tYXIuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3ht
bC9YUGF0aEdyYW1tYXIuY3BwCkBAIC0xNzgsMTUgKzE3OCwxNSBAQCB1c2luZyBuYW1lc3BhY2Ug
WFBhdGg7CiB0eXBlZGVmIHVuaW9uIFlZU1RZUEUKICNsaW5lIDYwICJXZWJDb3JlL3htbC9YUGF0
aEdyYW1tYXIueSIKIHsgCi0gICAgTnVtZXJpY09wOjpPcGNvZGUgbnVtZXJpY09wY29kZTsgCi0g
ICAgRXFUZXN0T3A6Ok9wY29kZSBlcXVhbGl0eVRlc3RPcGNvZGU7CisgICAgV2ViQ29yZTo6WFBh
dGg6Ok51bWVyaWNPcDo6T3Bjb2RlIG51bWVyaWNPcGNvZGU7CisgICAgV2ViQ29yZTo6WFBhdGg6
OkVxVGVzdE9wOjpPcGNvZGUgZXF1YWxpdHlUZXN0T3Bjb2RlOwogICAgIFN0cmluZ0ltcGwqIHN0
cmluZzsKLSAgICBTdGVwOjpBeGlzIGF4aXM7Ci0gICAgTG9jYXRpb25QYXRoKiBsb2NhdGlvblBh
dGg7Ci0gICAgU3RlcDo6Tm9kZVRlc3QqIG5vZGVUZXN0OwotICAgIFZlY3RvcjxzdGQ6OnVuaXF1
ZV9wdHI8RXhwcmVzc2lvbj4+KiBleHByZXNzaW9uVmVjdG9yOwotICAgIFN0ZXAqIHN0ZXA7Ci0g
ICAgRXhwcmVzc2lvbiogZXhwcmVzc2lvbjsgCisgICAgV2ViQ29yZTo6WFBhdGg6OlN0ZXA6OkF4
aXMgYXhpczsKKyAgICBXZWJDb3JlOjpYUGF0aDo6TG9jYXRpb25QYXRoKiBsb2NhdGlvblBhdGg7
CisgICAgV2ViQ29yZTo6WFBhdGg6OlN0ZXA6Ok5vZGVUZXN0KiBub2RlVGVzdDsKKyAgICBWZWN0
b3I8c3RkOjp1bmlxdWVfcHRyPFdlYkNvcmU6OlhQYXRoOjpFeHByZXNzaW9uPj4qIGV4cHJlc3Np
b25WZWN0b3I7CisgICAgV2ViQ29yZTo6WFBhdGg6OlN0ZXAqIHN0ZXA7CisgICAgV2ViQ29yZTo6
WFBhdGg6OkV4cHJlc3Npb24qIGV4cHJlc3Npb247CiB9CiAvKiBMaW5lIDE5MyBvZiB5YWNjLmMu
ICAqLwogI2xpbmUgMTkwICIuL1hQYXRoR3JhbW1hci5jcHAiCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS94bWwvWFBhdGhHcmFtbWFyLmggYi9Tb3VyY2UvV2ViQ29yZS94bWwvWFBhdGhHcmFt
bWFyLmgKaW5kZXggMzliOTZjYTI0ODAyNDU1NTA2N2QyZDMxYWIwODY0YzBmYmQ0YzFhNy4uYzFk
YmI2ZjIxYTE5MjE4NjZmMDBjN2Q5ZWNlZWJiM2FkMGRkZTM3MiAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYkNvcmUveG1sL1hQYXRoR3JhbW1hci5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3htbC9YUGF0
aEdyYW1tYXIuaApAQCAtOTQsMTUgKzk0LDE1IEBACiB0eXBlZGVmIHVuaW9uIFlZU1RZUEUKICNs
aW5lIDYwICJXZWJDb3JlL3htbC9YUGF0aEdyYW1tYXIueSIKIHsgCi0gICAgTnVtZXJpY09wOjpP
cGNvZGUgbnVtZXJpY09wY29kZTsgCi0gICAgRXFUZXN0T3A6Ok9wY29kZSBlcXVhbGl0eVRlc3RP
cGNvZGU7CisgICAgV2ViQ29yZTo6WFBhdGg6Ok51bWVyaWNPcDo6T3Bjb2RlIG51bWVyaWNPcGNv
ZGU7CisgICAgV2ViQ29yZTo6WFBhdGg6OkVxVGVzdE9wOjpPcGNvZGUgZXF1YWxpdHlUZXN0T3Bj
b2RlOwogICAgIFN0cmluZ0ltcGwqIHN0cmluZzsKLSAgICBTdGVwOjpBeGlzIGF4aXM7Ci0gICAg
TG9jYXRpb25QYXRoKiBsb2NhdGlvblBhdGg7Ci0gICAgU3RlcDo6Tm9kZVRlc3QqIG5vZGVUZXN0
OwotICAgIFZlY3RvcjxzdGQ6OnVuaXF1ZV9wdHI8RXhwcmVzc2lvbj4+KiBleHByZXNzaW9uVmVj
dG9yOwotICAgIFN0ZXAqIHN0ZXA7Ci0gICAgRXhwcmVzc2lvbiogZXhwcmVzc2lvbjsgCisgICAg
V2ViQ29yZTo6WFBhdGg6OlN0ZXA6OkF4aXMgYXhpczsKKyAgICBXZWJDb3JlOjpYUGF0aDo6TG9j
YXRpb25QYXRoKiBsb2NhdGlvblBhdGg7CisgICAgV2ViQ29yZTo6WFBhdGg6OlN0ZXA6Ok5vZGVU
ZXN0KiBub2RlVGVzdDsKKyAgICBWZWN0b3I8c3RkOjp1bmlxdWVfcHRyPFdlYkNvcmU6OlhQYXRo
OjpFeHByZXNzaW9uPj4qIGV4cHJlc3Npb25WZWN0b3I7CisgICAgV2ViQ29yZTo6WFBhdGg6OlN0
ZXAqIHN0ZXA7CisgICAgV2ViQ29yZTo6WFBhdGg6OkV4cHJlc3Npb24qIGV4cHJlc3Npb247CiB9
CiAvKiBMaW5lIDE1Mjkgb2YgeWFjYy5jLiAgKi8KICNsaW5lIDEwNyAiLi9YUGF0aEdyYW1tYXIu
aHBwIgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUveG1sL1hQYXRoR3JhbW1hci55IGIvU291
cmNlL1dlYkNvcmUveG1sL1hQYXRoR3JhbW1hci55CmluZGV4IGI3NTFhNjFiYTNmNDIyMjQwZTdj
MGJlMGU0ZjVkNjEzZTJmN2E1NWIuLmU0MDVkM2E0MzA1NTcyMjdlOTRiZGFiYTVjMjA1ZGIzY2Uy
MDFlMDMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3htbC9YUGF0aEdyYW1tYXIueQorKysg
Yi9Tb3VyY2UvV2ViQ29yZS94bWwvWFBhdGhHcmFtbWFyLnkKQEAgLTU4LDE1ICs1OCwxNSBAQCB1
c2luZyBuYW1lc3BhY2UgWFBhdGg7CiAlcGFyc2UtcGFyYW0geyBQYXJzZXImIHBhcnNlciB9CiAK
ICV1bmlvbiB7IAotICAgIE51bWVyaWNPcDo6T3Bjb2RlIG51bWVyaWNPcGNvZGU7IAotICAgIEVx
VGVzdE9wOjpPcGNvZGUgZXF1YWxpdHlUZXN0T3Bjb2RlOworICAgIFdlYkNvcmU6OlhQYXRoOjpO
dW1lcmljT3A6Ok9wY29kZSBudW1lcmljT3Bjb2RlOworICAgIFdlYkNvcmU6OlhQYXRoOjpFcVRl
c3RPcDo6T3Bjb2RlIGVxdWFsaXR5VGVzdE9wY29kZTsKICAgICBTdHJpbmdJbXBsKiBzdHJpbmc7
Ci0gICAgU3RlcDo6QXhpcyBheGlzOwotICAgIExvY2F0aW9uUGF0aCogbG9jYXRpb25QYXRoOwot
ICAgIFN0ZXA6Ok5vZGVUZXN0KiBub2RlVGVzdDsKLSAgICBWZWN0b3I8c3RkOjp1bmlxdWVfcHRy
PEV4cHJlc3Npb24+PiogZXhwcmVzc2lvblZlY3RvcjsKLSAgICBTdGVwKiBzdGVwOwotICAgIEV4
cHJlc3Npb24qIGV4cHJlc3Npb247IAorICAgIFdlYkNvcmU6OlhQYXRoOjpTdGVwOjpBeGlzIGF4
aXM7CisgICAgV2ViQ29yZTo6WFBhdGg6OkxvY2F0aW9uUGF0aCogbG9jYXRpb25QYXRoOworICAg
IFdlYkNvcmU6OlhQYXRoOjpTdGVwOjpOb2RlVGVzdCogbm9kZVRlc3Q7CisgICAgVmVjdG9yPHN0
ZDo6dW5pcXVlX3B0cjxXZWJDb3JlOjpYUGF0aDo6RXhwcmVzc2lvbj4+KiBleHByZXNzaW9uVmVj
dG9yOworICAgIFdlYkNvcmU6OlhQYXRoOjpTdGVwKiBzdGVwOworICAgIFdlYkNvcmU6OlhQYXRo
OjpFeHByZXNzaW9uKiBleHByZXNzaW9uOwogfQogJWxlZnQgPG51bWVyaWNPcGNvZGU+IE1VTE9Q
CiAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3htbC9YUGF0aFBhcnNlci5jcHAgYi9Tb3Vy
Y2UvV2ViQ29yZS94bWwvWFBhdGhQYXJzZXIuY3BwCmluZGV4IDg5ZjdlZDlkNzIyZjJhOWZjMTU5
YzlkZDViZWZmZDAzYzEzN2ZhZmIuLmNlMmQwODE1MmM2YmQyNTBjY2U4OTE4NmI5MDgwYjg3Y2Zk
MzYxNWIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3htbC9YUGF0aFBhcnNlci5jcHAKKysr
IGIvU291cmNlL1dlYkNvcmUveG1sL1hQYXRoUGFyc2VyLmNwcApAQCAtMzcsOCArMzcsNiBAQAog
I2luY2x1ZGUgPHd0Zi9TdGRMaWJFeHRyYXMuaD4KICNpbmNsdWRlIDx3dGYvdGV4dC9TdHJpbmdI
YXNoLmg+CiAKLXVzaW5nIG5hbWVzcGFjZSBXZWJDb3JlOjpYUGF0aDsKLQogZXh0ZXJuIGludCB4
cGF0aHl5cGFyc2UoV2ViQ29yZTo6WFBhdGg6OlBhcnNlciYpOwogCiAjaW5jbHVkZSAiWFBhdGhH
cmFtbWFyLmgiCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>