<?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>215156</bug_id>
          
          <creation_ts>2020-08-04 22:11:03 -0700</creation_ts>
          <short_desc>Make report-non-inclusive-language ignore files within .svn and .git</short_desc>
          <delta_ts>2020-10-20 10:54:50 -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>Tools / Tests</component>
          <version>WebKit Local Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=217972</see_also>
          <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="Beth Dakin">bdakin</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bdakin</cc>
    
    <cc>darin</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1677883</commentid>
    <comment_count>0</comment_count>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-04 22:11:03 -0700</bug_when>
    <thetext>Make report-non-inclusive-language ignore files within .svn and .git</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1677885</commentid>
    <comment_count>1</comment_count>
      <attachid>405985</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-04 22:14:15 -0700</bug_when>
    <thetext>Created attachment 405985
Patch

I&apos;m not sure if this is the best way to do this…</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1677997</commentid>
    <comment_count>2</comment_count>
      <attachid>405985</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-08-05 11:39:38 -0700</bug_when>
    <thetext>Comment on attachment 405985
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=405985&amp;action=review

&gt; Tools/Scripts/report-non-inclusive-language:69
&gt; +        if prefix.find(&quot;.svn&quot;) &gt;= 0:
&gt; +            continue
&gt; +        if prefix.find(&quot;.git&quot;) &gt;= 0:
&gt; +            continue

1. I suggest doing before the &quot;for file in files:&quot; loop, since it’s a check on the entire directory, not individual files.

2. I found documentation here &lt;https://docs.python.org/2/library/stdtypes.html#string-methods&gt; that says you can and should write this instead:

    if &quot;.svn&quot; in prefix:
        continue;

3. (This seems to supersede both 1 and 2 above.)

II did a little research and found a different solution, a way to make os.walk not even recurse into these directories. We can write this in the os.walk loop (outside the &quot;for file in files&quot; loop):

    directories = [directory for directory in directories if not directory.startswith(&quot;.svn&quot;)]

Then it will never even recurse into a that starts with .svn. Maybe there’s a nicer way to write it if you are a Python expert.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678011</commentid>
    <comment_count>3</comment_count>
      <attachid>405985</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-05 11:57:51 -0700</bug_when>
    <thetext>Comment on attachment 405985
Patch

Going to rewrite this for sure, so R- for now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678038</commentid>
    <comment_count>4</comment_count>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-05 12:46:15 -0700</bug_when>
    <thetext>I might be being too much of a Python n00b here, but this patch did not work for me. .svn files were still traversed:

Index: Scripts/report-non-inclusive-language
===================================================================
--- Scripts/report-non-inclusive-language	(revision 265267)
+++ Scripts/report-non-inclusive-language	(working copy)
@@ -54,6 +55,7 @@
 
 root = os.getcwd()
 for subroot, directories, files in os.walk(root):
+    directories = [directory for directory in directories if not directory.startswith(&quot;.svn&quot;)]
     prefix = subroot[len(root) + 1:]
     for file in files:
         if file.startswith(&quot;ChangeLog&quot;):</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678040</commentid>
    <comment_count>5</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-08-05 12:50:28 -0700</bug_when>
    <thetext>OK, then land something that works, and we will figure this out later.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678051</commentid>
    <comment_count>6</comment_count>
      <attachid>406029</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-05 13:07:45 -0700</bug_when>
    <thetext>Created attachment 406029
Patch

Here&apos;s a patch the moves the directory check outside of the files loop.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678052</commentid>
    <comment_count>7</comment_count>
      <attachid>406029</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-08-05 13:09:58 -0700</bug_when>
    <thetext>Comment on attachment 406029
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406029&amp;action=review

&gt; Tools/Scripts/report-non-inclusive-language:58
&gt; +    directories = [directory for directory in directories if not directory.startswith(&quot;.svn&quot;)]

Leave this out since it&apos;s not working yet

&gt; Tools/Scripts/report-non-inclusive-language:63
&gt; +    if prefix.find(&quot;.svn&quot;) &gt;= 0:
&gt; +        continue
&gt; +    if prefix.find(&quot;.git&quot;) &gt;= 0:
&gt; +        continue

Use this style:

    if &quot;.svn&quot; in prefix:</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678053</commentid>
    <comment_count>8</comment_count>
      <attachid>406030</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-05 13:18:36 -0700</bug_when>
    <thetext>Created attachment 406030
Patch

Oh yeah, I forgot that part of the feedback. Thank you!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678054</commentid>
    <comment_count>9</comment_count>
      <attachid>406030</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-08-05 13:23:52 -0700</bug_when>
    <thetext>Comment on attachment 406030
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406030&amp;action=review

&gt; Tools/Scripts/report-non-inclusive-language:58
&gt; +    directories = [directory for directory in directories if not directory.startswith(&quot;.svn&quot;)]

Take this out since it’s not working?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678055</commentid>
    <comment_count>10</comment_count>
      <attachid>406031</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2020-08-05 13:27:37 -0700</bug_when>
    <thetext>Created attachment 406031
Patch

Fixing the sloppy!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678072</commentid>
    <comment_count>11</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-08-05 14:08:10 -0700</bug_when>
    <thetext>Committed r265304: &lt;https://trac.webkit.org/changeset/265304&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 406031.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1678073</commentid>
    <comment_count>12</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-08-05 14:09:18 -0700</bug_when>
    <thetext>&lt;rdar://problem/66593010&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>405985</attachid>
            <date>2020-08-04 22:14:15 -0700</date>
            <delta_ts>2020-08-05 13:07:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>for-review.txt</filename>
            <type>text/plain</type>
            <size>1888</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI2NTI3NikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEyIEBACisyMDIwLTA4LTA0ICBCZXRoIERha2luICA8YmRha2luQGFwcGxlLmNvbT4KKwor
ICAgICAgICBNYWtlIHJlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlIGlnbm9yZSBmaWxlcyB3
aXRoaW4gLnN2biBhbmQgLmdpdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MjE1MTU2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBTY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlOgorCiAy
MDIwLTA4LTA0ICBGdWppaSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgogCiAg
ICAgICAgIFt3ZWJraXRjb3JlcHldIFJFR1JFU1NJT04ocjI2NTI1NCk6IGF1dG9pbnN0YWxsZXIg
aXMgZmFpbGluZyB0byBleGVjdXRlIHNldHVwLnB5IG9uIHdpbjMyIFB5dGhvbgpJbmRleDogVG9v
bHMvU2NyaXB0cy9yZXBvcnQtbm9uLWluY2x1c2l2ZS1sYW5ndWFnZQo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBU
b29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCShyZXZpc2lvbiAyNjUy
NjcpCisrKyBUb29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zMyw3ICszMyw4IEBAIGltcG9ydCBhcmdwYXJzZSwgb3MsIHJlCiAjICAg
Q2xldmVyIGRlZmF1bHQgZm9yIHVzZSBvbiBXZWJLaXQgc291cmNlIHRyZWUgaW5zdGVhZCBvZiBj
dXJyZW50IGRpcmVjdG9yeSBieSBkZWZhdWx0LgogIyAgIFJlcG9ydCB0aGlyZCBwYXJ0eSBzb3Vy
Y2VzIHNlcGFyYXRlbHkgZnJvbSBXZWJLaXQgcHJvamVjdCBmaWxlcy4KICMgICBTa2lwIGtub3du
IGV4YW1wbGVzIG9uIGFuIGV4Y2VwdGlvbiBsaXN0IHRoYXQgd2UgZG9uJ3QgaW50ZW5kIHRvIGZp
eC4KLSMgICBSZXBsYWNlIHNwZWNpYWwgY2FzZXMgZm9yIENoYW5nZUxvZywgIi5vcmRlciIsIGFu
ZCAiLnhjdXNlcnN0YXRlIiBmaWxlcyB3aXRoIHNvbWV0aGluZyBzbWFydGVyLgorIyAgIFJlcGxh
Y2Ugc3BlY2lhbCBjYXNlcyBmb3IgQ2hhbmdlTG9nLCAiLm9yZGVyIiwgIi54Y3VzZXJzdGF0ZSIs
ICIuZ2l0IiwgIi5zdm4iCisjICAgZmlsZXMgd2l0aCBzb21ldGhpbmcgc21hcnRlci4KIAogIyBI
YXJkLWNvZGVkIGZvciBub3csIGNvdWxkIHJlYWQgdGhlc2Ugb3V0IG9mIGEgZmlsZSBpbnN0ZWFk
LgogIyBUZXJtLCB0aGVuIHRoZSByZWd1bGFyIGV4cHJlc3Npb24gZm9yIHRoYXQgdGVybS4KQEAg
LTYyLDYgKzYzLDEwIEBAIGZvciBzdWJyb290LCBkaXJlY3RvcmllcywgZmlsZXMgaW4gb3Mud2EK
ICAgICAgICAgICAgIGNvbnRpbnVlCiAgICAgICAgIGlmIGZpbGUuZW5kc3dpdGgoIi54Y3VzZXJz
dGF0ZSIpOgogICAgICAgICAgICAgY29udGludWUKKyAgICAgICAgaWYgcHJlZml4LmZpbmQoIi5z
dm4iKSA+PSAwOgorICAgICAgICAgICAgY29udGludWUKKyAgICAgICAgaWYgcHJlZml4LmZpbmQo
Ii5naXQiKSA+PSAwOgorICAgICAgICAgICAgY29udGludWUKICAgICAgICAgaGFuZGxlID0gb3Bl
bihvcy5wYXRoLmpvaW4oc3Vicm9vdCwgZmlsZSksICJyIikKICAgICAgICAgZm9yIGxpbmUgaW4g
aGFuZGxlLnJlYWRsaW5lcygpOgogICAgICAgICAgICAgZm9yIHRlcm0gaW4gbm9uSW5jbHVzaXZl
VGVybXM6Cg==
</data>
<flag name="review"
          id="421397"
          type_id="1"
          status="-"
          setter="bdakin"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>406029</attachid>
            <date>2020-08-05 13:07:45 -0700</date>
            <delta_ts>2020-08-05 13:18:36 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>for-review-2.txt</filename>
            <type>text/plain</type>
            <size>1928</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI2NTI3NikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEyIEBACisyMDIwLTA4LTA0ICBCZXRoIERha2luICA8YmRha2luQGFwcGxlLmNvbT4KKwor
ICAgICAgICBNYWtlIHJlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlIGlnbm9yZSBmaWxlcyB3
aXRoaW4gLnN2biBhbmQgLmdpdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MjE1MTU2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBTY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlOgorCiAy
MDIwLTA4LTA0ICBGdWppaSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgogCiAg
ICAgICAgIFt3ZWJraXRjb3JlcHldIFJFR1JFU1NJT04ocjI2NTI1NCk6IGF1dG9pbnN0YWxsZXIg
aXMgZmFpbGluZyB0byBleGVjdXRlIHNldHVwLnB5IG9uIHdpbjMyIFB5dGhvbgpJbmRleDogVG9v
bHMvU2NyaXB0cy9yZXBvcnQtbm9uLWluY2x1c2l2ZS1sYW5ndWFnZQo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBU
b29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCShyZXZpc2lvbiAyNjUy
NjcpCisrKyBUb29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zMyw3ICszMyw4IEBAIGltcG9ydCBhcmdwYXJzZSwgb3MsIHJlCiAjICAg
Q2xldmVyIGRlZmF1bHQgZm9yIHVzZSBvbiBXZWJLaXQgc291cmNlIHRyZWUgaW5zdGVhZCBvZiBj
dXJyZW50IGRpcmVjdG9yeSBieSBkZWZhdWx0LgogIyAgIFJlcG9ydCB0aGlyZCBwYXJ0eSBzb3Vy
Y2VzIHNlcGFyYXRlbHkgZnJvbSBXZWJLaXQgcHJvamVjdCBmaWxlcy4KICMgICBTa2lwIGtub3du
IGV4YW1wbGVzIG9uIGFuIGV4Y2VwdGlvbiBsaXN0IHRoYXQgd2UgZG9uJ3QgaW50ZW5kIHRvIGZp
eC4KLSMgICBSZXBsYWNlIHNwZWNpYWwgY2FzZXMgZm9yIENoYW5nZUxvZywgIi5vcmRlciIsIGFu
ZCAiLnhjdXNlcnN0YXRlIiBmaWxlcyB3aXRoIHNvbWV0aGluZyBzbWFydGVyLgorIyAgIFJlcGxh
Y2Ugc3BlY2lhbCBjYXNlcyBmb3IgQ2hhbmdlTG9nLCAiLm9yZGVyIiwgIi54Y3VzZXJzdGF0ZSIs
ICIuZ2l0IiwgIi5zdm4iCisjICAgZmlsZXMgd2l0aCBzb21ldGhpbmcgc21hcnRlci4KIAogIyBI
YXJkLWNvZGVkIGZvciBub3csIGNvdWxkIHJlYWQgdGhlc2Ugb3V0IG9mIGEgZmlsZSBpbnN0ZWFk
LgogIyBUZXJtLCB0aGVuIHRoZSByZWd1bGFyIGV4cHJlc3Npb24gZm9yIHRoYXQgdGVybS4KQEAg
LTU0LDcgKzU1LDEyIEBAIGZvciB0ZXJtIGluIG5vbkluY2x1c2l2ZVRlcm1zOgogCiByb290ID0g
b3MuZ2V0Y3dkKCkKIGZvciBzdWJyb290LCBkaXJlY3RvcmllcywgZmlsZXMgaW4gb3Mud2Fsayhy
b290KToKKyAgICBkaXJlY3RvcmllcyA9IFtkaXJlY3RvcnkgZm9yIGRpcmVjdG9yeSBpbiBkaXJl
Y3RvcmllcyBpZiBub3QgZGlyZWN0b3J5LnN0YXJ0c3dpdGgoIi5zdm4iKV0KICAgICBwcmVmaXgg
PSBzdWJyb290W2xlbihyb290KSArIDE6XQorICAgIGlmIHByZWZpeC5maW5kKCIuc3ZuIikgPj0g
MDoKKyAgICAgICAgY29udGludWUKKyAgICBpZiBwcmVmaXguZmluZCgiLmdpdCIpID49IDA6Cisg
ICAgICAgIGNvbnRpbnVlCiAgICAgZm9yIGZpbGUgaW4gZmlsZXM6CiAgICAgICAgIGlmIGZpbGUu
c3RhcnRzd2l0aCgiQ2hhbmdlTG9nIik6CiAgICAgICAgICAgICBjb250aW51ZQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>406030</attachid>
            <date>2020-08-05 13:18:36 -0700</date>
            <delta_ts>2020-08-05 13:27:37 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>for-review-3.txt</filename>
            <type>text/plain</type>
            <size>1912</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI2NTI3NikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEyIEBACisyMDIwLTA4LTA0ICBCZXRoIERha2luICA8YmRha2luQGFwcGxlLmNvbT4KKwor
ICAgICAgICBNYWtlIHJlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlIGlnbm9yZSBmaWxlcyB3
aXRoaW4gLnN2biBhbmQgLmdpdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MjE1MTU2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBTY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlOgorCiAy
MDIwLTA4LTA0ICBGdWppaSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgogCiAg
ICAgICAgIFt3ZWJraXRjb3JlcHldIFJFR1JFU1NJT04ocjI2NTI1NCk6IGF1dG9pbnN0YWxsZXIg
aXMgZmFpbGluZyB0byBleGVjdXRlIHNldHVwLnB5IG9uIHdpbjMyIFB5dGhvbgpJbmRleDogVG9v
bHMvU2NyaXB0cy9yZXBvcnQtbm9uLWluY2x1c2l2ZS1sYW5ndWFnZQo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBU
b29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCShyZXZpc2lvbiAyNjUy
NjcpCisrKyBUb29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zMyw3ICszMyw4IEBAIGltcG9ydCBhcmdwYXJzZSwgb3MsIHJlCiAjICAg
Q2xldmVyIGRlZmF1bHQgZm9yIHVzZSBvbiBXZWJLaXQgc291cmNlIHRyZWUgaW5zdGVhZCBvZiBj
dXJyZW50IGRpcmVjdG9yeSBieSBkZWZhdWx0LgogIyAgIFJlcG9ydCB0aGlyZCBwYXJ0eSBzb3Vy
Y2VzIHNlcGFyYXRlbHkgZnJvbSBXZWJLaXQgcHJvamVjdCBmaWxlcy4KICMgICBTa2lwIGtub3du
IGV4YW1wbGVzIG9uIGFuIGV4Y2VwdGlvbiBsaXN0IHRoYXQgd2UgZG9uJ3QgaW50ZW5kIHRvIGZp
eC4KLSMgICBSZXBsYWNlIHNwZWNpYWwgY2FzZXMgZm9yIENoYW5nZUxvZywgIi5vcmRlciIsIGFu
ZCAiLnhjdXNlcnN0YXRlIiBmaWxlcyB3aXRoIHNvbWV0aGluZyBzbWFydGVyLgorIyAgIFJlcGxh
Y2Ugc3BlY2lhbCBjYXNlcyBmb3IgQ2hhbmdlTG9nLCAiLm9yZGVyIiwgIi54Y3VzZXJzdGF0ZSIs
ICIuZ2l0IiwgIi5zdm4iCisjICAgZmlsZXMgd2l0aCBzb21ldGhpbmcgc21hcnRlci4KIAogIyBI
YXJkLWNvZGVkIGZvciBub3csIGNvdWxkIHJlYWQgdGhlc2Ugb3V0IG9mIGEgZmlsZSBpbnN0ZWFk
LgogIyBUZXJtLCB0aGVuIHRoZSByZWd1bGFyIGV4cHJlc3Npb24gZm9yIHRoYXQgdGVybS4KQEAg
LTU0LDcgKzU1LDEyIEBAIGZvciB0ZXJtIGluIG5vbkluY2x1c2l2ZVRlcm1zOgogCiByb290ID0g
b3MuZ2V0Y3dkKCkKIGZvciBzdWJyb290LCBkaXJlY3RvcmllcywgZmlsZXMgaW4gb3Mud2Fsayhy
b290KToKKyAgICBkaXJlY3RvcmllcyA9IFtkaXJlY3RvcnkgZm9yIGRpcmVjdG9yeSBpbiBkaXJl
Y3RvcmllcyBpZiBub3QgZGlyZWN0b3J5LnN0YXJ0c3dpdGgoIi5zdm4iKV0KICAgICBwcmVmaXgg
PSBzdWJyb290W2xlbihyb290KSArIDE6XQorICAgIGlmICIuc3ZuIiBpbiBwcmVmaXg6CisgICAg
ICAgIGNvbnRpbnVlCisgICAgaWYgIi5naXQiIGluIHByZWZpeDoKKyAgICAgICAgY29udGludWUK
ICAgICBmb3IgZmlsZSBpbiBmaWxlczoKICAgICAgICAgaWYgZmlsZS5zdGFydHN3aXRoKCJDaGFu
Z2VMb2ciKToKICAgICAgICAgICAgIGNvbnRpbnVlCg==
</data>
<flag name="review"
          id="421437"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>406031</attachid>
            <date>2020-08-05 13:27:37 -0700</date>
            <delta_ts>2020-08-05 14:08:10 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>for-review-4.txt</filename>
            <type>text/plain</type>
            <size>1814</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI2NTI3NikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEyIEBACisyMDIwLTA4LTA0ICBCZXRoIERha2luICA8YmRha2luQGFwcGxlLmNvbT4KKwor
ICAgICAgICBNYWtlIHJlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlIGlnbm9yZSBmaWxlcyB3
aXRoaW4gLnN2biBhbmQgLmdpdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MjE1MTU2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBTY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlOgorCiAy
MDIwLTA4LTA0ICBGdWppaSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgogCiAg
ICAgICAgIFt3ZWJraXRjb3JlcHldIFJFR1JFU1NJT04ocjI2NTI1NCk6IGF1dG9pbnN0YWxsZXIg
aXMgZmFpbGluZyB0byBleGVjdXRlIHNldHVwLnB5IG9uIHdpbjMyIFB5dGhvbgpJbmRleDogVG9v
bHMvU2NyaXB0cy9yZXBvcnQtbm9uLWluY2x1c2l2ZS1sYW5ndWFnZQo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBU
b29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCShyZXZpc2lvbiAyNjUy
NjcpCisrKyBUb29scy9TY3JpcHRzL3JlcG9ydC1ub24taW5jbHVzaXZlLWxhbmd1YWdlCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zMyw3ICszMyw4IEBAIGltcG9ydCBhcmdwYXJzZSwgb3MsIHJlCiAjICAg
Q2xldmVyIGRlZmF1bHQgZm9yIHVzZSBvbiBXZWJLaXQgc291cmNlIHRyZWUgaW5zdGVhZCBvZiBj
dXJyZW50IGRpcmVjdG9yeSBieSBkZWZhdWx0LgogIyAgIFJlcG9ydCB0aGlyZCBwYXJ0eSBzb3Vy
Y2VzIHNlcGFyYXRlbHkgZnJvbSBXZWJLaXQgcHJvamVjdCBmaWxlcy4KICMgICBTa2lwIGtub3du
IGV4YW1wbGVzIG9uIGFuIGV4Y2VwdGlvbiBsaXN0IHRoYXQgd2UgZG9uJ3QgaW50ZW5kIHRvIGZp
eC4KLSMgICBSZXBsYWNlIHNwZWNpYWwgY2FzZXMgZm9yIENoYW5nZUxvZywgIi5vcmRlciIsIGFu
ZCAiLnhjdXNlcnN0YXRlIiBmaWxlcyB3aXRoIHNvbWV0aGluZyBzbWFydGVyLgorIyAgIFJlcGxh
Y2Ugc3BlY2lhbCBjYXNlcyBmb3IgQ2hhbmdlTG9nLCAiLm9yZGVyIiwgIi54Y3VzZXJzdGF0ZSIs
ICIuZ2l0IiwgIi5zdm4iCisjICAgZmlsZXMgd2l0aCBzb21ldGhpbmcgc21hcnRlci4KIAogIyBI
YXJkLWNvZGVkIGZvciBub3csIGNvdWxkIHJlYWQgdGhlc2Ugb3V0IG9mIGEgZmlsZSBpbnN0ZWFk
LgogIyBUZXJtLCB0aGVuIHRoZSByZWd1bGFyIGV4cHJlc3Npb24gZm9yIHRoYXQgdGVybS4KQEAg
LTU1LDYgKzU2LDEwIEBAIGZvciB0ZXJtIGluIG5vbkluY2x1c2l2ZVRlcm1zOgogcm9vdCA9IG9z
LmdldGN3ZCgpCiBmb3Igc3Vicm9vdCwgZGlyZWN0b3JpZXMsIGZpbGVzIGluIG9zLndhbGsocm9v
dCk6CiAgICAgcHJlZml4ID0gc3Vicm9vdFtsZW4ocm9vdCkgKyAxOl0KKyAgICBpZiAiLnN2biIg
aW4gcHJlZml4OgorICAgICAgICBjb250aW51ZQorICAgIGlmICIuZ2l0IiBpbiBwcmVmaXg6Cisg
ICAgICAgIGNvbnRpbnVlCiAgICAgZm9yIGZpbGUgaW4gZmlsZXM6CiAgICAgICAgIGlmIGZpbGUu
c3RhcnRzd2l0aCgiQ2hhbmdlTG9nIik6CiAgICAgICAgICAgICBjb250aW51ZQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>