<?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>198692</bug_id>
          
          <creation_ts>2019-06-08 14:01:49 -0700</creation_ts>
          <short_desc>[iOS] ResourceLoadStatistics state is not properly relayed to the NetworkProcess</short_desc>
          <delta_ts>2019-06-09 17:25:27 -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>WebKit2</component>
          <version>WebKit Nightly 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=198694</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>
          
          <blocked>198694</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Brent Fulgham">bfulgham</reporter>
          <assigned_to name="Brent Fulgham">bfulgham</assigned_to>
          <cc>achristensen</cc>
    
    <cc>beidson</cc>
    
    <cc>bfulgham</cc>
    
    <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>jiewen_tan</cc>
    
    <cc>mjs</cc>
    
    <cc>pvollan</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wilander</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1543110</commentid>
    <comment_count>0</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 14:01:49 -0700</bug_when>
    <thetext>When I moved the ResourceLoadStatistics logic to the NetworkProcess, I did not realize that the iOS process startup does not have a WebsiteDataStore attached until much later in the launch cycle. Because of this, we do not pass the ITP state to the NetworkProcess at launch (as part of its configuration parameters), and when we attempt to message the state to the NetworkProcess during startup we do so at a moment when we have not associated the WebProcessPool with the WebsiteDataStore object that knows about ITP status.

Furthermore, I failed to notice that the method to activate ITP only messages the WebContent processes, not the NetworkProcess. This had not been needed before I moved the logic to the NetworkProcess, and was an oversight.

This patch updates WebProcessPool::setResourceLoadStatisticsEnabled so that the process pool tells the NetworkProcess when the ITP state changes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543111</commentid>
    <comment_count>1</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 14:03:37 -0700</bug_when>
    <thetext>&lt;rdar://problem/51538088&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543112</commentid>
    <comment_count>2</comment_count>
      <attachid>371666</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 14:04:43 -0700</bug_when>
    <thetext>Created attachment 371666
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543113</commentid>
    <comment_count>3</comment_count>
      <attachid>371666</attachid>
    <who name="John Wilander">wilander</who>
    <bug_when>2019-06-08 14:15:42 -0700</bug_when>
    <thetext>Comment on attachment 371666
Patch

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

&gt; Source/WebKit/UIProcess/WebProcessPool.cpp:1491
&gt; +    sendToNetworkingProcess(Messages::WebProcess::SetResourceLoadStatisticsEnabled(enabled));

This message type should be for the NetworkProcess, not the WebProcess.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543116</commentid>
    <comment_count>4</comment_count>
      <attachid>371666</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2019-06-08 14:25:00 -0700</bug_when>
    <thetext>Comment on attachment 371666
Patch

This really needs a test, since it is a fix for a real regression and one that wen&apos;t long unnoticed. r- for lack of test case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543142</commentid>
    <comment_count>5</comment_count>
      <attachid>371681</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 16:50:00 -0700</bug_when>
    <thetext>Created attachment 371681
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543143</commentid>
    <comment_count>6</comment_count>
      <attachid>371681</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 16:51:33 -0700</bug_when>
    <thetext>Comment on attachment 371681
Patch

Updated to use proper message heading.

I&apos;ll tackle the test separately since it will be a larger patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543144</commentid>
    <comment_count>7</comment_count>
      <attachid>371682</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 17:01:23 -0700</bug_when>
    <thetext>Created attachment 371682
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543145</commentid>
    <comment_count>8</comment_count>
      <attachid>371682</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 17:01:46 -0700</bug_when>
    <thetext>Comment on attachment 371682
Patch

Updated to address WPE build failure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543147</commentid>
    <comment_count>9</comment_count>
      <attachid>371682</attachid>
    <who name="John Wilander">wilander</who>
    <bug_when>2019-06-08 17:19:13 -0700</bug_when>
    <thetext>Comment on attachment 371682
Patch

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

&gt; Source/WebKit/UIProcess/WebProcessPool.cpp:1492
&gt; +    sendToNetworkingProcess(Messages::NetworkProcess::SetResourceLoadStatisticsEnabled(enabled));

I’m looking at this on my phone so I don’t have the larger picture. Do we need to ensure that there is a network process here? I know we had some cases where that was a requirement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543155</commentid>
    <comment_count>10</comment_count>
      <attachid>371682</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 17:44:15 -0700</bug_when>
    <thetext>Comment on attachment 371682
Patch

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

&gt;&gt; Source/WebKit/UIProcess/WebProcessPool.cpp:1492
&gt;&gt; +    sendToNetworkingProcess(Messages::NetworkProcess::SetResourceLoadStatisticsEnabled(enabled));
&gt; 
&gt; I’m looking at this on my phone so I don’t have the larger picture. Do we need to ensure that there is a network process here? I know we had some cases where that was a requirement.

That&apos;s probably a good idea. It looks like we do something like that when dealing with Ephemeral sessions, so this probably has the same requirement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543157</commentid>
    <comment_count>11</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 17:47:14 -0700</bug_when>
    <thetext>(In reply to Maciej Stachowiak from comment #4)
&gt; Comment on attachment 371666 [details]
&gt; Patch
&gt; 
&gt; This really needs a test, since it is a fix for a real regression and one
&gt; that wen&apos;t long unnoticed. r- for lack of test case.

I started doing this, but the patch grew much larger. I would rather handle the test case as a distinct patch so that it&apos;s clear what&apos;s going on.

See Bug 198694 for that work.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543163</commentid>
    <comment_count>12</comment_count>
      <attachid>371691</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 18:20:44 -0700</bug_when>
    <thetext>Created attachment 371691
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543178</commentid>
    <comment_count>13</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 19:07:27 -0700</bug_when>
    <thetext>(In reply to Brent Fulgham from comment #10)
&gt; Comment on attachment 371682 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=371682&amp;action=review
&gt; 
&gt; &gt;&gt; Source/WebKit/UIProcess/WebProcessPool.cpp:1492
&gt; &gt;&gt; +    sendToNetworkingProcess(Messages::NetworkProcess::SetResourceLoadStatisticsEnabled(enabled));
&gt; &gt; 
&gt; &gt; I’m looking at this on my phone so I don’t have the larger picture. Do we need to ensure that there is a network process here? I know we had some cases where that was a requirement.
&gt; 
&gt; That&apos;s probably a good idea. It looks like we do something like that when
&gt; dealing with Ephemeral sessions, so this probably has the same requirement.

Actually it&apos;s not a good idea because it breaks the API tests involving a lack of a process pool (TestWebKitAPI.WebKit.WKHTTPCookieStoreWithoutProcessPoolWithPrewarming and the without prewarming case).

So, we don&apos;t want to force a network process launch if it doesn&apos;t already exist -- we just want to make sure we do send it the message.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543180</commentid>
    <comment_count>14</comment_count>
      <attachid>371697</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-06-08 19:10:16 -0700</bug_when>
    <thetext>Created attachment 371697
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543228</commentid>
    <comment_count>15</comment_count>
      <attachid>371697</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2019-06-09 16:30:20 -0700</bug_when>
    <thetext>Comment on attachment 371697
Patch

Looks good.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543231</commentid>
    <comment_count>16</comment_count>
      <attachid>371697</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-06-09 17:25:25 -0700</bug_when>
    <thetext>Comment on attachment 371697
Patch

Clearing flags on attachment: 371697

Committed r246248: &lt;https://trac.webkit.org/changeset/246248&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1543232</commentid>
    <comment_count>17</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-06-09 17:25:27 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>371666</attachid>
            <date>2019-06-08 14:04:43 -0700</date>
            <delta_ts>2019-06-08 16:49:58 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-198692-20190608140442.patch</filename>
            <type>text/plain</type>
            <size>1523</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ2MjA2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDFmNDk1M2E0ZjI0OTg4MjA2
NWIzMDlhNTA5ZTY3NjVkNzRhMjZkMzQuLmVhNjJhMTcwZGMyZDY4ODMwNDBhM2VhNGQyY2VkOTNm
Zjc1ZDY1MzggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTktMDYtMDggIEJyZW50IEZ1
bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gUmVzb3VyY2VMb2Fk
U3RhdGlzdGljcyBzdGF0ZSBpcyBub3QgcHJvcGVybHkgcmVsYXllZCB0byB0aGUgTmV0d29ya1By
b2Nlc3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5
ODY5MgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTE1MzgwODg+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvV2ViUHJvY2Vzc1Bv
b2wuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6c2V0UmVzb3VyY2VMb2Fk
U3RhdGlzdGljc0VuYWJsZWQpOiBNZXNzYWdlIHRoZSBOZXR3b3JrUHJvY2VzcworICAgICAgICBv
biBzdGF0ZSBjaGFuZ2UuCisKIDIwMTktMDYtMDcgIEJyZW50IEZ1bGdoYW0gIDxiZnVsZ2hhbUBh
cHBsZS5jb20+CiAKICAgICAgICAgW2lPU10gVXBkYXRlIHNhbmRib3hlcyB0byBhZGRyZXNzIGlv
a2l0LWdldC1wcm9wZXJ0eSBlcnJvcnMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvVUlQcm9j
ZXNzL1dlYlByb2Nlc3NQb29sLmNwcCBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nl
c3NQb29sLmNwcAppbmRleCAzNGJmYzAzOGI5NTdjMTc1MWE1YmQwMThmNjU0YTQ3MjA0M2E5MDQ0
Li43MWE5ZmFlNDVjZDE0MjBiNTg4MTA2NTdmMmY3ODAwMTFiNWMzNGNlIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKKysrIGIvU291cmNlL1dl
YktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwCkBAIC0xNDg4LDYgKzE0ODgsNyBAQCB2
b2lkIFdlYlByb2Nlc3NQb29sOjpzZXRTaG91bGRVc2VGb250U21vb3RoaW5nKGJvb2wgdXNlRm9u
dFNtb290aGluZykKIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldFJlc291cmNlTG9hZFN0YXRpc3Rp
Y3NFbmFibGVkKGJvb2wgZW5hYmxlZCkKIHsKICAgICBzZW5kVG9BbGxQcm9jZXNzZXMoTWVzc2Fn
ZXM6OldlYlByb2Nlc3M6OlNldFJlc291cmNlTG9hZFN0YXRpc3RpY3NFbmFibGVkKGVuYWJsZWQp
KTsKKyAgICBzZW5kVG9OZXR3b3JraW5nUHJvY2VzcyhNZXNzYWdlczo6V2ViUHJvY2Vzczo6U2V0
UmVzb3VyY2VMb2FkU3RhdGlzdGljc0VuYWJsZWQoZW5hYmxlZCkpOwogfQogCiB2b2lkIFdlYlBy
b2Nlc3NQb29sOjpjbGVhclJlc291cmNlTG9hZFN0YXRpc3RpY3MoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>371681</attachid>
            <date>2019-06-08 16:50:00 -0700</date>
            <delta_ts>2019-06-08 17:01:21 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-198692-20190608164959.patch</filename>
            <type>text/plain</type>
            <size>1527</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ2MjA2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDFmNDk1M2E0ZjI0OTg4MjA2
NWIzMDlhNTA5ZTY3NjVkNzRhMjZkMzQuLmVhNjJhMTcwZGMyZDY4ODMwNDBhM2VhNGQyY2VkOTNm
Zjc1ZDY1MzggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTktMDYtMDggIEJyZW50IEZ1
bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gUmVzb3VyY2VMb2Fk
U3RhdGlzdGljcyBzdGF0ZSBpcyBub3QgcHJvcGVybHkgcmVsYXllZCB0byB0aGUgTmV0d29ya1By
b2Nlc3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5
ODY5MgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTE1MzgwODg+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvV2ViUHJvY2Vzc1Bv
b2wuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6c2V0UmVzb3VyY2VMb2Fk
U3RhdGlzdGljc0VuYWJsZWQpOiBNZXNzYWdlIHRoZSBOZXR3b3JrUHJvY2VzcworICAgICAgICBv
biBzdGF0ZSBjaGFuZ2UuCisKIDIwMTktMDYtMDcgIEJyZW50IEZ1bGdoYW0gIDxiZnVsZ2hhbUBh
cHBsZS5jb20+CiAKICAgICAgICAgW2lPU10gVXBkYXRlIHNhbmRib3hlcyB0byBhZGRyZXNzIGlv
a2l0LWdldC1wcm9wZXJ0eSBlcnJvcnMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvVUlQcm9j
ZXNzL1dlYlByb2Nlc3NQb29sLmNwcCBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nl
c3NQb29sLmNwcAppbmRleCAzNGJmYzAzOGI5NTdjMTc1MWE1YmQwMThmNjU0YTQ3MjA0M2E5MDQ0
Li4wOGEzM2FjOTJmZDRiZDU4NTVlMDVjMGE2ODYzNzgyZjg5Yzg5YzZmIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKKysrIGIvU291cmNlL1dl
YktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwCkBAIC0xNDg4LDYgKzE0ODgsNyBAQCB2
b2lkIFdlYlByb2Nlc3NQb29sOjpzZXRTaG91bGRVc2VGb250U21vb3RoaW5nKGJvb2wgdXNlRm9u
dFNtb290aGluZykKIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldFJlc291cmNlTG9hZFN0YXRpc3Rp
Y3NFbmFibGVkKGJvb2wgZW5hYmxlZCkKIHsKICAgICBzZW5kVG9BbGxQcm9jZXNzZXMoTWVzc2Fn
ZXM6OldlYlByb2Nlc3M6OlNldFJlc291cmNlTG9hZFN0YXRpc3RpY3NFbmFibGVkKGVuYWJsZWQp
KTsKKyAgICBzZW5kVG9OZXR3b3JraW5nUHJvY2VzcyhNZXNzYWdlczo6TmV0d29ya1Byb2Nlc3M6
OlNldFJlc291cmNlTG9hZFN0YXRpc3RpY3NFbmFibGVkKGVuYWJsZWQpKTsKIH0KIAogdm9pZCBX
ZWJQcm9jZXNzUG9vbDo6Y2xlYXJSZXNvdXJjZUxvYWRTdGF0aXN0aWNzKCkK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>371682</attachid>
            <date>2019-06-08 17:01:23 -0700</date>
            <delta_ts>2019-06-08 18:20:42 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-198692-20190608170122.patch</filename>
            <type>text/plain</type>
            <size>1573</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ2MjA2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDFmNDk1M2E0ZjI0OTg4MjA2
NWIzMDlhNTA5ZTY3NjVkNzRhMjZkMzQuLmVhNjJhMTcwZGMyZDY4ODMwNDBhM2VhNGQyY2VkOTNm
Zjc1ZDY1MzggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTktMDYtMDggIEJyZW50IEZ1
bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gUmVzb3VyY2VMb2Fk
U3RhdGlzdGljcyBzdGF0ZSBpcyBub3QgcHJvcGVybHkgcmVsYXllZCB0byB0aGUgTmV0d29ya1By
b2Nlc3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5
ODY5MgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTE1MzgwODg+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvV2ViUHJvY2Vzc1Bv
b2wuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6c2V0UmVzb3VyY2VMb2Fk
U3RhdGlzdGljc0VuYWJsZWQpOiBNZXNzYWdlIHRoZSBOZXR3b3JrUHJvY2VzcworICAgICAgICBv
biBzdGF0ZSBjaGFuZ2UuCisKIDIwMTktMDYtMDcgIEJyZW50IEZ1bGdoYW0gIDxiZnVsZ2hhbUBh
cHBsZS5jb20+CiAKICAgICAgICAgW2lPU10gVXBkYXRlIHNhbmRib3hlcyB0byBhZGRyZXNzIGlv
a2l0LWdldC1wcm9wZXJ0eSBlcnJvcnMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvVUlQcm9j
ZXNzL1dlYlByb2Nlc3NQb29sLmNwcCBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nl
c3NQb29sLmNwcAppbmRleCAzNGJmYzAzOGI5NTdjMTc1MWE1YmQwMThmNjU0YTQ3MjA0M2E5MDQ0
Li42MDRmZjRmYTMzZDQyMDk1NGQ5ZTQwYzJhMDUyNTE0ZTZlZmNlNTBiIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKKysrIGIvU291cmNlL1dl
YktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwCkBAIC0xNDg4LDYgKzE0ODgsOSBAQCB2
b2lkIFdlYlByb2Nlc3NQb29sOjpzZXRTaG91bGRVc2VGb250U21vb3RoaW5nKGJvb2wgdXNlRm9u
dFNtb290aGluZykKIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldFJlc291cmNlTG9hZFN0YXRpc3Rp
Y3NFbmFibGVkKGJvb2wgZW5hYmxlZCkKIHsKICAgICBzZW5kVG9BbGxQcm9jZXNzZXMoTWVzc2Fn
ZXM6OldlYlByb2Nlc3M6OlNldFJlc291cmNlTG9hZFN0YXRpc3RpY3NFbmFibGVkKGVuYWJsZWQp
KTsKKyNpZiBFTkFCTEUoUkVTT1VSQ0VfTE9BRF9TVEFUSVNUSUNTKQorICAgIHNlbmRUb05ldHdv
cmtpbmdQcm9jZXNzKE1lc3NhZ2VzOjpOZXR3b3JrUHJvY2Vzczo6U2V0UmVzb3VyY2VMb2FkU3Rh
dGlzdGljc0VuYWJsZWQoZW5hYmxlZCkpOworI2VuZGlmCiB9CiAKIHZvaWQgV2ViUHJvY2Vzc1Bv
b2w6OmNsZWFyUmVzb3VyY2VMb2FkU3RhdGlzdGljcygpCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>371691</attachid>
            <date>2019-06-08 18:20:44 -0700</date>
            <delta_ts>2019-06-08 19:10:14 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-198692-20190608182043.patch</filename>
            <type>text/plain</type>
            <size>1615</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ2MjM1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDI1NGNmYmU0OGMwNjQzODQz
ZGIxNjdjNGM1MDY5MWFiMGUwNmEwZDEuLmQzNTNlOTgwMzMyNzZkNTE3OTM5ZDNhODVlMWY5YmFh
YTM0MzdmZDAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTktMDYtMDggIEJyZW50IEZ1
bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gUmVzb3VyY2VMb2Fk
U3RhdGlzdGljcyBzdGF0ZSBpcyBub3QgcHJvcGVybHkgcmVsYXllZCB0byB0aGUgTmV0d29ya1By
b2Nlc3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5
ODY5MgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTE1MzgwODg+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvV2ViUHJvY2Vzc1Bv
b2wuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6c2V0UmVzb3VyY2VMb2Fk
U3RhdGlzdGljc0VuYWJsZWQpOiBNZXNzYWdlIHRoZSBOZXR3b3JrUHJvY2VzcworICAgICAgICBv
biBzdGF0ZSBjaGFuZ2UuCisKIDIwMTktMDYtMDggIEFuZHkgRXN0ZXMgIDxhZXN0ZXNAYXBwbGUu
Y29tPgogCiAgICAgICAgIFtBcHBsZSBQYXldIElmIHdlIGhhdmUgYSBib3VuZCBpbnRlcmZhY2Ug
aWRlbnRpZmllciwgc2V0IGl0IG9uIG5ldyBQS1BheW1lbnRSZXF1ZXN0cwpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwIGIvU291cmNlL1dlYktp
dC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwCmluZGV4IDM0YmZjMDM4Yjk1N2MxNzUxYTVi
ZDAxOGY2NTRhNDcyMDQzYTkwNDQuLmI3MzU0NGFhZjMwZDM2YTAwYjc5YjkxMTQ1MmUyZWIyMzQ2
MjNlOGYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nlc3NQb29s
LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKQEAg
LTE0ODgsNiArMTQ4OCw5IEBAIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldFNob3VsZFVzZUZvbnRT
bW9vdGhpbmcoYm9vbCB1c2VGb250U21vb3RoaW5nKQogdm9pZCBXZWJQcm9jZXNzUG9vbDo6c2V0
UmVzb3VyY2VMb2FkU3RhdGlzdGljc0VuYWJsZWQoYm9vbCBlbmFibGVkKQogewogICAgIHNlbmRU
b0FsbFByb2Nlc3NlcyhNZXNzYWdlczo6V2ViUHJvY2Vzczo6U2V0UmVzb3VyY2VMb2FkU3RhdGlz
dGljc0VuYWJsZWQoZW5hYmxlZCkpOworI2lmIEVOQUJMRShSRVNPVVJDRV9MT0FEX1NUQVRJU1RJ
Q1MpCisgICAgc2VuZFRvTmV0d29ya2luZ1Byb2Nlc3NSZWxhdW5jaGluZ0lmTmVjZXNzYXJ5KE1l
c3NhZ2VzOjpOZXR3b3JrUHJvY2Vzczo6U2V0UmVzb3VyY2VMb2FkU3RhdGlzdGljc0VuYWJsZWQo
ZW5hYmxlZCkpOworI2VuZGlmCiB9CiAKIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OmNsZWFyUmVzb3Vy
Y2VMb2FkU3RhdGlzdGljcygpCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>371697</attachid>
            <date>2019-06-08 19:10:16 -0700</date>
            <delta_ts>2019-06-09 17:25:25 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-198692-20190608191015.patch</filename>
            <type>text/plain</type>
            <size>1593</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ2MjM1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDI1NGNmYmU0OGMwNjQzODQz
ZGIxNjdjNGM1MDY5MWFiMGUwNmEwZDEuLmQzNTNlOTgwMzMyNzZkNTE3OTM5ZDNhODVlMWY5YmFh
YTM0MzdmZDAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTktMDYtMDggIEJyZW50IEZ1
bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gUmVzb3VyY2VMb2Fk
U3RhdGlzdGljcyBzdGF0ZSBpcyBub3QgcHJvcGVybHkgcmVsYXllZCB0byB0aGUgTmV0d29ya1By
b2Nlc3MKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE5
ODY5MgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTE1MzgwODg+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvV2ViUHJvY2Vzc1Bv
b2wuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6c2V0UmVzb3VyY2VMb2Fk
U3RhdGlzdGljc0VuYWJsZWQpOiBNZXNzYWdlIHRoZSBOZXR3b3JrUHJvY2VzcworICAgICAgICBv
biBzdGF0ZSBjaGFuZ2UuCisKIDIwMTktMDYtMDggIEFuZHkgRXN0ZXMgIDxhZXN0ZXNAYXBwbGUu
Y29tPgogCiAgICAgICAgIFtBcHBsZSBQYXldIElmIHdlIGhhdmUgYSBib3VuZCBpbnRlcmZhY2Ug
aWRlbnRpZmllciwgc2V0IGl0IG9uIG5ldyBQS1BheW1lbnRSZXF1ZXN0cwpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwIGIvU291cmNlL1dlYktp
dC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Bvb2wuY3BwCmluZGV4IDM0YmZjMDM4Yjk1N2MxNzUxYTVi
ZDAxOGY2NTRhNDcyMDQzYTkwNDQuLjYwNGZmNGZhMzNkNDIwOTU0ZDllNDBjMmEwNTI1MTRlNmVm
Y2U1MGIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nlc3NQb29s
LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKQEAg
LTE0ODgsNiArMTQ4OCw5IEBAIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldFNob3VsZFVzZUZvbnRT
bW9vdGhpbmcoYm9vbCB1c2VGb250U21vb3RoaW5nKQogdm9pZCBXZWJQcm9jZXNzUG9vbDo6c2V0
UmVzb3VyY2VMb2FkU3RhdGlzdGljc0VuYWJsZWQoYm9vbCBlbmFibGVkKQogewogICAgIHNlbmRU
b0FsbFByb2Nlc3NlcyhNZXNzYWdlczo6V2ViUHJvY2Vzczo6U2V0UmVzb3VyY2VMb2FkU3RhdGlz
dGljc0VuYWJsZWQoZW5hYmxlZCkpOworI2lmIEVOQUJMRShSRVNPVVJDRV9MT0FEX1NUQVRJU1RJ
Q1MpCisgICAgc2VuZFRvTmV0d29ya2luZ1Byb2Nlc3MoTWVzc2FnZXM6Ok5ldHdvcmtQcm9jZXNz
OjpTZXRSZXNvdXJjZUxvYWRTdGF0aXN0aWNzRW5hYmxlZChlbmFibGVkKSk7CisjZW5kaWYKIH0K
IAogdm9pZCBXZWJQcm9jZXNzUG9vbDo6Y2xlYXJSZXNvdXJjZUxvYWRTdGF0aXN0aWNzKCkK
</data>

          </attachment>
      

    </bug>

</bugzilla>