<?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>200770</bug_id>
          
          <creation_ts>2019-08-15 09:25:31 -0700</creation_ts>
          <short_desc>Web Inspector: CodeMirror still inserts a tab even when &quot;Prefer indent using&quot; is set to &quot;Spaces&quot;</short_desc>
          <delta_ts>2019-08-15 12:33:17 -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>Web Inspector</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Devin Rousso">hi</reporter>
          <assigned_to name="Devin Rousso">hi</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>hi</cc>
    
    <cc>inspector-bugzilla-changes</cc>
    
    <cc>ross.kirsling</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1561571</commentid>
    <comment_count>0</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 09:25:31 -0700</bug_when>
    <thetext># STEPS TO REPRODUCE:
1. inspect any page
2. go to any resource that is editable
3. press tab ↹
 =&gt; \t inserted instead of &quot;    &quot; (4)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561572</commentid>
    <comment_count>1</comment_count>
      <attachid>376384</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 09:26:45 -0700</bug_when>
    <thetext>Created attachment 376384
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561580</commentid>
    <comment_count>2</comment_count>
    <who name="Ross Kirsling">ross.kirsling</who>
    <bug_when>2019-08-15 09:50:28 -0700</bug_when>
    <thetext>Is this better than having WI make Tab execute `insertSoftTab` and specify `options.tabSize` appropriately?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561582</commentid>
    <comment_count>3</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 09:55:54 -0700</bug_when>
    <thetext>(In reply to Ross Kirsling from comment #2)
&gt; Is this better than having WI make Tab execute `insertSoftTab` and specify `options.tabSize` appropriately?
I didn&apos;t even know that was a thing!  I&apos;ll give it a shot =D</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561584</commentid>
    <comment_count>4</comment_count>
      <attachid>376387</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 09:59:01 -0700</bug_when>
    <thetext>Created attachment 376387
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561602</commentid>
    <comment_count>5</comment_count>
      <attachid>376387</attachid>
    <who name="Ross Kirsling">ross.kirsling</who>
    <bug_when>2019-08-15 10:23:35 -0700</bug_when>
    <thetext>Comment on attachment 376387
Patch

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

It&apos;s still a bit sad to me, but I guess it&apos;s how we&apos;ve got to do it:
https://github.com/codemirror/CodeMirror/pull/4317

&gt; Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js:624
&gt; +        CodeMirror.commands.insertTab = function(cm) {

Maybe it&apos;d still be better to overwrite `defaultTab`, so that you don&apos;t have to store `original`?

&gt; Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js:625
&gt; +            if (WI.settings.indentWithTabs.value)

I think this can be `cm.options.indentWithTabs`, given all this:
https://github.com/WebKit/webkit/blob/master/Source/WebInspectorUI/UserInterface/Views/CodeMirrorEditor.js#L36-L58</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561606</commentid>
    <comment_count>6</comment_count>
      <attachid>376387</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 10:29:58 -0700</bug_when>
    <thetext>Comment on attachment 376387
Patch

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

&gt;&gt; Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js:624
&gt;&gt; +        CodeMirror.commands.insertTab = function(cm) {
&gt; 
&gt; Maybe it&apos;d still be better to overwrite `defaultTab`, so that you don&apos;t have to store `original`?

I&apos;d rather not, as that would require that I duplicate the `if (cm.somethingSelected()) { cm.indentSelection(&quot;add&quot;) }` logic, which adds complexity and potential confusion each time we update CodeMirror.

&gt;&gt; Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js:625
&gt;&gt; +            if (WI.settings.indentWithTabs.value)
&gt; 
&gt; I think this can be `cm.options.indentWithTabs`, given all this:
&gt; https://github.com/WebKit/webkit/blob/master/Source/WebInspectorUI/UserInterface/Views/CodeMirrorEditor.js#L36-L58

Good point.  I&apos;ll switch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561607</commentid>
    <comment_count>7</comment_count>
      <attachid>376390</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 10:31:44 -0700</bug_when>
    <thetext>Created attachment 376390
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561613</commentid>
    <comment_count>8</comment_count>
    <who name="Ross Kirsling">ross.kirsling</who>
    <bug_when>2019-08-15 10:36:17 -0700</bug_when>
    <thetext>(In reply to Devin Rousso from comment #6)
&gt; I&apos;d rather not, as that would require that I duplicate the `if
&gt; (cm.somethingSelected()) { cm.indentSelection(&quot;add&quot;) }` logic, which adds
&gt; complexity and potential confusion each time we update CodeMirror.

Even with this solution though, I wonder if a comment would be in order to encourage us to keep an eye on it across CodeMirror updates? (Maybe we could link to the PR as a justification for why we had to go this route.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561614</commentid>
    <comment_count>9</comment_count>
      <attachid>376390</attachid>
    <who name="Ross Kirsling">ross.kirsling</who>
    <bug_when>2019-08-15 10:36:31 -0700</bug_when>
    <thetext>Comment on attachment 376390
Patch

r=me otherwise</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561639</commentid>
    <comment_count>10</comment_count>
      <attachid>376396</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-08-15 11:13:48 -0700</bug_when>
    <thetext>Created attachment 376396
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561691</commentid>
    <comment_count>11</comment_count>
      <attachid>376396</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-08-15 12:32:21 -0700</bug_when>
    <thetext>Comment on attachment 376396
Patch

Clearing flags on attachment: 376396

Committed r248739: &lt;https://trac.webkit.org/changeset/248739&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561692</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-08-15 12:32:22 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1561693</commentid>
    <comment_count>13</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-08-15 12:33:17 -0700</bug_when>
    <thetext>&lt;rdar://problem/54358187&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376384</attachid>
            <date>2019-08-15 09:26:45 -0700</date>
            <delta_ts>2019-08-15 09:58:57 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200770-20190815092644.patch</filename>
            <type>text/plain</type>
            <size>1668</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ2MTZiMDMxMDk0NjgwOGQ2YzRhYTYxMzgyNjhj
OWVjOTUzMjkyNWUuLjhkYjM1NzM0MTA1NjM1MGI4MWE2NjFkOThlNTY1MGUyYzdhYmRmZDYgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDE5LTA4LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBD
b2RlTWlycm9yIHN0aWxsIGluc2VydHMgYSB0YWIgZXZlbiB3aGVuICJQcmVmZXIgaW5kZW50IHVz
aW5nIiBpcyBzZXQgdG8gIlNwYWNlcyIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTIwMDc3MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgICogVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRpb25z
LmpzOgorICAgICAgICBSZW1hcCB0aGUgYGluc2VydFRhYmAgY29tbWFuZCB0byB1c2UgYFdJLmlu
ZGVudFN0cmluZ2AgKHdoaWNoIGRldGVybWluZXMgd2hldGhlciB0byB1c2UgdGFicworICAgICAg
ICBvciBzcGFjZXMpIGluc3RlYWQgb2YgYWx3YXlzIGluc2VydGluZyBhIHRhYiAoIlx0IikuCisK
IDIwMTktMDgtMTQgIERldmluIFJvdXNzbyAgPGRyb3Vzc29AYXBwbGUuY29tPgogCiAgICAgICAg
IFdlYiBJbnNwZWN0b3I6IEVsZW1lbnRzOiBDb21wdXRlZDogbW92ZSB0aGUgQm94IE1vZGVsIHNl
Y3Rpb24gdG8gdGhlIHRvcApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJ
bnRlcmZhY2UvVmlld3MvQ29kZU1pcnJvckFkZGl0aW9ucy5qcyBiL1NvdXJjZS9XZWJJbnNwZWN0
b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMuanMKaW5kZXggNDYz
MTRhMWU3OGQ3ODIzMjIwNjZhZGUzMTE3MTJkNzE1Mjg2MGU5Zi4uNjBlMjhiNGU1ZDdmZDAwMDdk
NWY0OTE5Yzg0NzczMDc5ZTEyMjBiNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkluc3BlY3RvclVJ
L1VzZXJJbnRlcmZhY2UvVmlld3MvQ29kZU1pcnJvckFkZGl0aW9ucy5qcworKysgYi9Tb3VyY2Uv
V2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRpb25zLmpz
CkBAIC02MTksNiArNjE5LDEwIEBACiAgICAgICAgIGZhbGx0aHJvdWdoOiBtYWMgPyAibWFjRGVm
YXVsdCIgOiAicGNEZWZhdWx0IgogICAgIH07CiAKKyAgICBDb2RlTWlycm9yLmNvbW1hbmRzLmlu
c2VydFRhYiA9IGZ1bmN0aW9uKGNtKSB7CisgICAgICAgIGNtLnJlcGxhY2VTZWxlY3Rpb24oV0ku
aW5kZW50U3RyaW5nKCkpOworICAgIH07CisKICAgICAvLyBSZWdpc3RlciBzb21lIGV4dHJhIE1J
TUUtdHlwZXMgZm9yIENvZGVNaXJyb3IuIFRoZXNlIGFyZSBpbiBhZGRpdGlvbiB0byB0aGUKICAg
ICAvLyBvbmVzIENvZGVNaXJyb3IgYWxyZWFkeSByZWdpc3RlcnMsIGxpa2UgdGV4dC9odG1sLCB0
ZXh0L2phdmFzY3JpcHQsIGV0Yy4KICAgICB2YXIgZXh0cmFYTUxUeXBlcyA9IFsidGV4dC94bWwi
LCAidGV4dC94c2wiXTsK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376387</attachid>
            <date>2019-08-15 09:59:01 -0700</date>
            <delta_ts>2019-08-15 10:31:43 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200770-20190815095859.patch</filename>
            <type>text/plain</type>
            <size>1925</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ2MTZiMDMxMDk0NjgwOGQ2YzRhYTYxMzgyNjhj
OWVjOTUzMjkyNWUuLjgzNDZlZDczYWFiMTFiZWEzMjcwMmFmMjdjMzUzNTc5YzA0NjA0MzcgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDE5LTA4LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBD
b2RlTWlycm9yIHN0aWxsIGluc2VydHMgYSB0YWIgZXZlbiB3aGVuICJQcmVmZXIgaW5kZW50IHVz
aW5nIiBpcyBzZXQgdG8gIlNwYWNlcyIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTIwMDc3MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgICogVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRpb25z
LmpzOgorICAgICAgICBSZW1hcCB0aGUgYGluc2VydFRhYmAgY29tbWFuZCB0byB1c2UgYGluc2Vy
dFNvZnRUYWJgIHdoZW4gIlByZWZlciBpbmRlbnQgdXNpbmciIGlzIHNldCB0bworICAgICAgICAi
U3BhY2VzIiBzbyB0aGF0IENvZGVNaXJyb3IgaW5zZXJ0cyB0aGUgbnVtYmVyIG9mIHNwYWNlcyB0
aGF0IHdvdWxkIG1hdGNoIGEgdGFiICgiXHQiKSBiZWluZworICAgICAgICBpbnNlcnRlZCBhdCB0
aGUgc2FtZSBzcG90LgorCiAyMDE5LTA4LTE0ICBEZXZpbiBSb3Vzc28gIDxkcm91c3NvQGFwcGxl
LmNvbT4KIAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBFbGVtZW50czogQ29tcHV0ZWQ6IG1vdmUg
dGhlIEJveCBNb2RlbCBzZWN0aW9uIHRvIHRoZSB0b3AKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJ
bnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMuanMgYi9T
b3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRp
b25zLmpzCmluZGV4IDQ2MzE0YTFlNzhkNzgyMzIyMDY2YWRlMzExNzEyZDcxNTI4NjBlOWYuLjE4
NjZkZjE5ZDIyNGRkZGY3ZmM2OTFhNWNmMDEwOTQ3NzAxZDM4YmMgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMu
anMKKysrIGIvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvQ29kZU1p
cnJvckFkZGl0aW9ucy5qcwpAQCAtNjE5LDYgKzYxOSwxNiBAQAogICAgICAgICBmYWxsdGhyb3Vn
aDogbWFjID8gIm1hY0RlZmF1bHQiIDogInBjRGVmYXVsdCIKICAgICB9OwogCisgICAgeworICAg
ICAgICBsZXQgb3JpZ2luYWwgPSBDb2RlTWlycm9yLmNvbW1hbmRzLmluc2VydFRhYjsKKyAgICAg
ICAgQ29kZU1pcnJvci5jb21tYW5kcy5pbnNlcnRUYWIgPSBmdW5jdGlvbihjbSkgeworICAgICAg
ICAgICAgaWYgKFdJLnNldHRpbmdzLmluZGVudFdpdGhUYWJzLnZhbHVlKQorICAgICAgICAgICAg
ICAgIG9yaWdpbmFsKGNtKTsKKyAgICAgICAgICAgIGVsc2UKKyAgICAgICAgICAgICAgICBDb2Rl
TWlycm9yLmNvbW1hbmRzLmluc2VydFNvZnRUYWIoY20pOworICAgICAgICB9OworICAgIH0KKwog
ICAgIC8vIFJlZ2lzdGVyIHNvbWUgZXh0cmEgTUlNRS10eXBlcyBmb3IgQ29kZU1pcnJvci4gVGhl
c2UgYXJlIGluIGFkZGl0aW9uIHRvIHRoZQogICAgIC8vIG9uZXMgQ29kZU1pcnJvciBhbHJlYWR5
IHJlZ2lzdGVycywgbGlrZSB0ZXh0L2h0bWwsIHRleHQvamF2YXNjcmlwdCwgZXRjLgogICAgIHZh
ciBleHRyYVhNTFR5cGVzID0gWyJ0ZXh0L3htbCIsICJ0ZXh0L3hzbCJdOwo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376390</attachid>
            <date>2019-08-15 10:31:44 -0700</date>
            <delta_ts>2019-08-15 11:13:46 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200770-20190815103143.patch</filename>
            <type>text/plain</type>
            <size>1918</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ2MTZiMDMxMDk0NjgwOGQ2YzRhYTYxMzgyNjhj
OWVjOTUzMjkyNWUuLjgzNDZlZDczYWFiMTFiZWEzMjcwMmFmMjdjMzUzNTc5YzA0NjA0MzcgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDE5LTA4LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBD
b2RlTWlycm9yIHN0aWxsIGluc2VydHMgYSB0YWIgZXZlbiB3aGVuICJQcmVmZXIgaW5kZW50IHVz
aW5nIiBpcyBzZXQgdG8gIlNwYWNlcyIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTIwMDc3MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgICogVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRpb25z
LmpzOgorICAgICAgICBSZW1hcCB0aGUgYGluc2VydFRhYmAgY29tbWFuZCB0byB1c2UgYGluc2Vy
dFNvZnRUYWJgIHdoZW4gIlByZWZlciBpbmRlbnQgdXNpbmciIGlzIHNldCB0bworICAgICAgICAi
U3BhY2VzIiBzbyB0aGF0IENvZGVNaXJyb3IgaW5zZXJ0cyB0aGUgbnVtYmVyIG9mIHNwYWNlcyB0
aGF0IHdvdWxkIG1hdGNoIGEgdGFiICgiXHQiKSBiZWluZworICAgICAgICBpbnNlcnRlZCBhdCB0
aGUgc2FtZSBzcG90LgorCiAyMDE5LTA4LTE0ICBEZXZpbiBSb3Vzc28gIDxkcm91c3NvQGFwcGxl
LmNvbT4KIAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBFbGVtZW50czogQ29tcHV0ZWQ6IG1vdmUg
dGhlIEJveCBNb2RlbCBzZWN0aW9uIHRvIHRoZSB0b3AKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJ
bnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMuanMgYi9T
b3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWlycm9yQWRkaXRp
b25zLmpzCmluZGV4IDQ2MzE0YTFlNzhkNzgyMzIyMDY2YWRlMzExNzEyZDcxNTI4NjBlOWYuLmFj
MzZmYzg2OTA0ZmViN2ZjZWNkODM1MWFhOWRjZjJkNTc4MTE2NWYgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMu
anMKKysrIGIvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvQ29kZU1p
cnJvckFkZGl0aW9ucy5qcwpAQCAtNjE5LDYgKzYxOSwxNiBAQAogICAgICAgICBmYWxsdGhyb3Vn
aDogbWFjID8gIm1hY0RlZmF1bHQiIDogInBjRGVmYXVsdCIKICAgICB9OwogCisgICAgeworICAg
ICAgICBsZXQgb3JpZ2luYWwgPSBDb2RlTWlycm9yLmNvbW1hbmRzLmluc2VydFRhYjsKKyAgICAg
ICAgQ29kZU1pcnJvci5jb21tYW5kcy5pbnNlcnRUYWIgPSBmdW5jdGlvbihjbSkgeworICAgICAg
ICAgICAgaWYgKGNtLm9wdGlvbnMuaW5kZW50V2l0aFRhYnMpCisgICAgICAgICAgICAgICAgb3Jp
Z2luYWwoY20pOworICAgICAgICAgICAgZWxzZQorICAgICAgICAgICAgICAgIENvZGVNaXJyb3Iu
Y29tbWFuZHMuaW5zZXJ0U29mdFRhYihjbSk7CisgICAgICAgIH07CisgICAgfQorCiAgICAgLy8g
UmVnaXN0ZXIgc29tZSBleHRyYSBNSU1FLXR5cGVzIGZvciBDb2RlTWlycm9yLiBUaGVzZSBhcmUg
aW4gYWRkaXRpb24gdG8gdGhlCiAgICAgLy8gb25lcyBDb2RlTWlycm9yIGFscmVhZHkgcmVnaXN0
ZXJzLCBsaWtlIHRleHQvaHRtbCwgdGV4dC9qYXZhc2NyaXB0LCBldGMuCiAgICAgdmFyIGV4dHJh
WE1MVHlwZXMgPSBbInRleHQveG1sIiwgInRleHQveHNsIl07Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>376396</attachid>
            <date>2019-08-15 11:13:48 -0700</date>
            <delta_ts>2019-08-15 12:32:21 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200770-20190815111347.patch</filename>
            <type>text/plain</type>
            <size>2024</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ2MTZiMDMxMDk0NjgwOGQ2YzRhYTYxMzgyNjhj
OWVjOTUzMjkyNWUuLjlhZmIxNDg5MDNiOGNmY2FkYzEyZTUxNTMxZTAyYzI0ODUxODIxZWQgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDE5LTA4LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBD
b2RlTWlycm9yIHN0aWxsIGluc2VydHMgYSB0YWIgZXZlbiB3aGVuICJQcmVmZXIgaW5kZW50IHVz
aW5nIiBpcyBzZXQgdG8gIlNwYWNlcyIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTIwMDc3MAorCisgICAgICAgIFJldmlld2VkIGJ5IFJvc3MgS2lyc2xp
bmcuCisKKyAgICAgICAgKiBVc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlvbnMu
anM6CisgICAgICAgIFJlbWFwIHRoZSBgaW5zZXJ0VGFiYCBjb21tYW5kIHRvIHVzZSBgaW5zZXJ0
U29mdFRhYmAgd2hlbiAiUHJlZmVyIGluZGVudCB1c2luZyIgaXMgc2V0IHRvCisgICAgICAgICJT
cGFjZXMiIHNvIHRoYXQgQ29kZU1pcnJvciBpbnNlcnRzIHRoZSBudW1iZXIgb2Ygc3BhY2VzIHRo
YXQgd291bGQgbWF0Y2ggYSB0YWIgKCJcdCIpIGJlaW5nCisgICAgICAgIGluc2VydGVkIGF0IHRo
ZSBzYW1lIHNwb3QuCisKIDIwMTktMDgtMTQgIERldmluIFJvdXNzbyAgPGRyb3Vzc29AYXBwbGUu
Y29tPgogCiAgICAgICAgIFdlYiBJbnNwZWN0b3I6IEVsZW1lbnRzOiBDb21wdXRlZDogbW92ZSB0
aGUgQm94IE1vZGVsIHNlY3Rpb24gdG8gdGhlIHRvcApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYklu
c3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvQ29kZU1pcnJvckFkZGl0aW9ucy5qcyBiL1Nv
dXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0NvZGVNaXJyb3JBZGRpdGlv
bnMuanMKaW5kZXggNDYzMTRhMWU3OGQ3ODIzMjIwNjZhZGUzMTE3MTJkNzE1Mjg2MGU5Zi4uODM2
M2I0MDViNDZjZTM3OTBkYWVlYzI3NjAyZjdlMzI1NzUxYzUwZiAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvQ29kZU1pcnJvckFkZGl0aW9ucy5q
cworKysgYi9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9Db2RlTWly
cm9yQWRkaXRpb25zLmpzCkBAIC02MTksNiArNjE5LDE3IEBACiAgICAgICAgIGZhbGx0aHJvdWdo
OiBtYWMgPyAibWFjRGVmYXVsdCIgOiAicGNEZWZhdWx0IgogICAgIH07CiAKKyAgICB7CisgICAg
ICAgIC8vIENvZGVNaXJyb3IncyBkZWZhdWx0IGJlaGF2aW9yIGlzIHRvIGFsd2F5cyBpbnNlcnQg
YSB0YWIgKCJcdCIpIHJlZ2FyZGxlc3Mgb2YgYGluZGVudFdpdGhUYWJzYC4KKyAgICAgICAgbGV0
IG9yaWdpbmFsID0gQ29kZU1pcnJvci5jb21tYW5kcy5pbnNlcnRUYWI7CisgICAgICAgIENvZGVN
aXJyb3IuY29tbWFuZHMuaW5zZXJ0VGFiID0gZnVuY3Rpb24oY20pIHsKKyAgICAgICAgICAgIGlm
IChjbS5vcHRpb25zLmluZGVudFdpdGhUYWJzKQorICAgICAgICAgICAgICAgIG9yaWdpbmFsKGNt
KTsKKyAgICAgICAgICAgIGVsc2UKKyAgICAgICAgICAgICAgICBDb2RlTWlycm9yLmNvbW1hbmRz
Lmluc2VydFNvZnRUYWIoY20pOworICAgICAgICB9OworICAgIH0KKwogICAgIC8vIFJlZ2lzdGVy
IHNvbWUgZXh0cmEgTUlNRS10eXBlcyBmb3IgQ29kZU1pcnJvci4gVGhlc2UgYXJlIGluIGFkZGl0
aW9uIHRvIHRoZQogICAgIC8vIG9uZXMgQ29kZU1pcnJvciBhbHJlYWR5IHJlZ2lzdGVycywgbGlr
ZSB0ZXh0L2h0bWwsIHRleHQvamF2YXNjcmlwdCwgZXRjLgogICAgIHZhciBleHRyYVhNTFR5cGVz
ID0gWyJ0ZXh0L3htbCIsICJ0ZXh0L3hzbCJdOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>