<?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>126129</bug_id>
          
          <creation_ts>2013-12-22 03:05:16 -0800</creation_ts>
          <short_desc>Implement NetworkResourceLoader::receivedCancellation</short_desc>
          <delta_ts>2014-01-10 01:07:08 -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>WebKit2</component>
          <version>528+ (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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>108832</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>andersca</cc>
    
    <cc>beidson</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>962308</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-12-22 03:05:16 -0800</bug_when>
    <thetext>$ Programs/WebKit2APITests/TestWebKitWebView -p /webkit2/WebKitWebView/authentication-cancel
/webkit2/WebKitWebView/authentication-cancel: OK

$ WEBKIT_USE_NETWORK_PROCESS=1 Programs/WebKit2APITests/TestWebKitWebView -p /webkit2/WebKitWebView/authentication-cancel
/webkit2/WebKitWebView/authentication-cancel: **
ERROR:../../Source/WebKit2/UIProcess/API/gtk/tests/TestWebKitWebView.cpp:1331:void testWebViewAuthenticationCancel(AuthenticationTest*, gconstpointer): assertion failed (test-&gt;m_loadEvents[1] == LoadTrackingTest::ProvisionalLoadFailed): (3 == 2)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>963302</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-12-30 02:23:53 -0800</bug_when>
    <thetext>The problem is that NetworkResourceLoader doesn&apos;t implement ResourceHandleClient::receivedCancellation() and the load finished normally instead of failing with cancel error like the web process does.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>963303</commentid>
    <comment_count>2</comment_count>
      <attachid>220104</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-12-30 02:27:07 -0800</bug_when>
    <thetext>Created attachment 220104
Patch

$ WEBKIT_USE_NETWORK_PROCESS=1 Programs/WebKit2APITests/TestAuthentication 
/webkit2/WebKitWebView/authentication-request: OK
/webkit2/WebKitWebView/authentication-cancel: OK
/webkit2/WebKitWebView/authentication-load-cancelled: OK
/webkit2/WebKitWebView/authentication-failure: OK
/webkit2/WebKitWebView/authentication-no-credential: OK
/webkit2/WebKitWebView/authentication-storage: OK
/webkit2/WebKitWebView/authentication-success: OK</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>963493</commentid>
    <comment_count>3</comment_count>
      <attachid>220153</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-12-31 00:39:11 -0800</bug_when>
    <thetext>Created attachment 220153
Updated patch

Just rebased to apply on current git master</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965454</commentid>
    <comment_count>4</comment_count>
      <attachid>220153</attachid>
    <who name="Brian Holt">brian.holt</who>
    <bug_when>2014-01-08 07:01:53 -0800</bug_when>
    <thetext>Comment on attachment 220153
Updated patch

Informal review: looks good to me, hope to see it landed soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>965961</commentid>
    <comment_count>5</comment_count>
      <attachid>220153</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2014-01-09 10:15:21 -0800</bug_when>
    <thetext>Comment on attachment 220153
Updated patch

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

&gt; Source/WebKit2/ChangeLog:3
&gt; +        [GTK] Test /webkit2/WebKitWebView/authentication-cancel fails with network process enabled

This patch is cross-platform, please remove &quot;[GTK]&quot; here and from the bug title.

&gt; Source/WebKit2/NetworkProcess/NetworkResourceLoader.cpp:174
&gt; +void NetworkResourceLoader::cancel()

I&apos;m not a fan of this function. It looks like someone tells NetworkResourceLoader to cancel, but this is not what it is - it implements what needs to be done in two places where NetworkResourceLoader decides to cancel the load on its own, based on indirect hints. In other future places where NetworkResourceLoader decides to cancel, it may need to do something else.

I would have just put the code inline, not factoring it out into a cancel() function. But at the very least, the function should be private, not public.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>966278</commentid>
    <comment_count>6</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-10 01:07:08 -0800</bug_when>
    <thetext>Committed r161625: &lt;http://trac.webkit.org/changeset/161625&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220104</attachid>
            <date>2013-12-30 02:27:07 -0800</date>
            <delta_ts>2013-12-31 00:39:11 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wk2-network-process-cancel.diff</filename>
            <type>text/plain</type>
            <size>3804</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCBkMTVlNTQ0Li43Y2QxZWM3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjEg
QEAKKzIwMTMtMTItMzAgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFtHVEtdIFRlc3QgL3dlYmtpdDIvV2ViS2l0V2ViVmlldy9hdXRoZW50aWNh
dGlvbi1jYW5jZWwgZmFpbHMgd2l0aCBuZXR3b3JrIHByb2Nlc3MgZW5hYmxlZAorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI2MTI5CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSW1wbGVtZW50IFJlc291cmNl
SGFuZGxlQ2xpZW50OjpyZWNlaXZlZENhbmNlbGxhdGlvbigpIGluIHRoZQorICAgICAgICBuZXR3
b3JrIHByb2Nlc3MgdG8gY2FuY2VsIHRoZSBsb2FkIHdoZW4gY2FuY2VsbGF0aW9uIGlzIHJlY2Vp
dmVkCisgICAgICAgIGZyb20gdGhlIGF1dGhlbnRpY2F0aW9uIG1hbmFnZXIgbGlrZSB0aGUgd2Vi
IHByb2Nlc3MgZG9lcy4KKworICAgICAgICAqIE5ldHdvcmtQcm9jZXNzL05ldHdvcmtSZXNvdXJj
ZUxvYWRlci5jcHA6CisgICAgICAgIChXZWJLaXQ6Ok5ldHdvcmtSZXNvdXJjZUxvYWRlcjo6Y2Fu
Y2VsKTogSGVscGVyIGZ1bmN0aW9uIHRvIGNhbmNlbAorICAgICAgICB0aGUgbG9hZC4KKyAgICAg
ICAgKFdlYktpdDo6TmV0d29ya1Jlc291cmNlTG9hZGVyOjpjb250aW51ZVdpbGxTZW5kUmVxdWVz
dCk6IENhbGwgY2FuY2VsKCkuCisgICAgICAgIChXZWJLaXQ6Ok5ldHdvcmtSZXNvdXJjZUxvYWRl
cjo6cmVjZWl2ZWRDYW5jZWxsYXRpb24pOiBEaXR0by4KKyAgICAgICAgKiBOZXR3b3JrUHJvY2Vz
cy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuaDoKKwogMjAxMy0xMi0yNyAgQ2FybG9zIEdhcmNpYSBD
YW1wb3MgIDxjZ2FyY2lhQGlnYWxpYS5jb20+CiAKICAgICAgICAgQ3Jhc2ggd2hlbiBzdGFydGlu
ZyBhIGRvd25sb2FkIGJlZm9yZSB0aGUgbmV0d29yayBwcm9jZXNzIGhhcyBiZWVuIGxhdW5jaGVk
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3Vy
Y2VMb2FkZXIuY3BwIGIvU291cmNlL1dlYktpdDIvTmV0d29ya1Byb2Nlc3MvTmV0d29ya1Jlc291
cmNlTG9hZGVyLmNwcAppbmRleCA5NjM3OWI5Li5lNDFhNzA4IDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuY3BwCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL05ldHdvcmtSZXNvdXJjZUxvYWRlci5jcHAKQEAg
LTE3MSw2ICsxNzEsMTMgQEAgdm9pZCBOZXR3b3JrUmVzb3VyY2VMb2FkZXI6OmFib3J0KCkKICAg
ICBjbGVhbnVwKCk7CiB9CiAKK3ZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpjYW5jZWwoKQor
eworICAgIEFTU0VSVChtX2hhbmRsZSk7CisgICAgbV9oYW5kbGUtPmNhbmNlbCgpOworICAgIGRp
ZEZhaWwobV9oYW5kbGUuZ2V0KCksIGNhbmNlbGxlZEVycm9yKG1fcmVxdWVzdCkpOworfQorCiB2
b2lkIE5ldHdvcmtSZXNvdXJjZUxvYWRlcjo6ZGlkUmVjZWl2ZVJlc3BvbnNlQXN5bmMoUmVzb3Vy
Y2VIYW5kbGUqIGhhbmRsZSwgY29uc3QgUmVzb3VyY2VSZXNwb25zZSYgcmVzcG9uc2UpCiB7CiAg
ICAgQVNTRVJUX1VOVVNFRChoYW5kbGUsIGhhbmRsZSA9PSBtX2hhbmRsZSk7CkBAIC0yNTcsMTAg
KzI2NCw4IEBAIHZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpjb250aW51ZVdpbGxTZW5kUmVx
dWVzdChjb25zdCBSZXNvdXJjZVJlcXVlc3QmIG5ld1JlCiAKICAgICBtX2hhbmRsZS0+Y29udGlu
dWVXaWxsU2VuZFJlcXVlc3QobV9yZXF1ZXN0KTsKIAotICAgIGlmIChtX3JlcXVlc3QuaXNOdWxs
KCkpIHsKLSAgICAgICAgbV9oYW5kbGUtPmNhbmNlbCgpOwotICAgICAgICBkaWRGYWlsKG1faGFu
ZGxlLmdldCgpLCBjYW5jZWxsZWRFcnJvcihtX3JlcXVlc3QpKTsKLSAgICB9CisgICAgaWYgKG1f
cmVxdWVzdC5pc051bGwoKSkKKyAgICAgICAgY2FuY2VsKCk7CiB9CiAKIHZvaWQgTmV0d29ya1Jl
c291cmNlTG9hZGVyOjpjb250aW51ZURpZFJlY2VpdmVSZXNwb25zZSgpCkBAIC0zMzYsNiArMzQx
LDEzIEBAIHZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpkaWRDYW5jZWxBdXRoZW50aWNhdGlv
bkNoYWxsZW5nZShSZXNvdXJjZUhhbmRsZSogaGFuCiAgICAgbm90SW1wbGVtZW50ZWQoKTsKIH0K
IAordm9pZCBOZXR3b3JrUmVzb3VyY2VMb2FkZXI6OnJlY2VpdmVkQ2FuY2VsbGF0aW9uKFJlc291
cmNlSGFuZGxlKiBoYW5kbGUsIGNvbnN0IEF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdlJiBjaGFsbGVu
Z2UpCit7CisgICAgQVNTRVJUX1VOVVNFRChoYW5kbGUsIGhhbmRsZSA9PSBtX2hhbmRsZSk7CisK
KyAgICBjYW5jZWwoKTsKK30KKwogQ29yZUlQQzo6Q29ubmVjdGlvbiogTmV0d29ya1Jlc291cmNl
TG9hZGVyOjptZXNzYWdlU2VuZGVyQ29ubmVjdGlvbigpCiB7CiAgICAgcmV0dXJuIGNvbm5lY3Rp
b25Ub1dlYlByb2Nlc3MoKS0+Y29ubmVjdGlvbigpOwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktp
dDIvTmV0d29ya1Byb2Nlc3MvTmV0d29ya1Jlc291cmNlTG9hZGVyLmggYi9Tb3VyY2UvV2ViS2l0
Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuaAppbmRleCAxODk2OThkLi41
YTYyNzZlIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3Jr
UmVzb3VyY2VMb2FkZXIuaAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3
b3JrUmVzb3VyY2VMb2FkZXIuaApAQCAtNzcsNiArNzcsNyBAQCBwdWJsaWM6CiAKICAgICB2b2lk
IHN0YXJ0KCk7CiAgICAgdm9pZCBhYm9ydCgpOworICAgIHZvaWQgY2FuY2VsKCk7CiAKICAgICAv
LyBSZXNvdXJjZUhhbmRsZUNsaWVudCBtZXRob2RzCiAgICAgdmlydHVhbCB2b2lkIHdpbGxTZW5k
UmVxdWVzdEFzeW5jKFdlYkNvcmU6OlJlc291cmNlSGFuZGxlKiwgY29uc3QgV2ViQ29yZTo6UmVz
b3VyY2VSZXF1ZXN0JiwgY29uc3QgV2ViQ29yZTo6UmVzb3VyY2VSZXNwb25zZSYgcmVkaXJlY3RS
ZXNwb25zZSkgT1ZFUlJJREU7CkBAIC05Miw2ICs5Myw3IEBAIHB1YmxpYzoKICAgICB2aXJ0dWFs
IHZvaWQgc2hvdWxkVXNlQ3JlZGVudGlhbFN0b3JhZ2VBc3luYyhXZWJDb3JlOjpSZXNvdXJjZUhh
bmRsZSopIE9WRVJSSURFOwogICAgIHZpcnR1YWwgdm9pZCBkaWRSZWNlaXZlQXV0aGVudGljYXRp
b25DaGFsbGVuZ2UoV2ViQ29yZTo6UmVzb3VyY2VIYW5kbGUqLCBjb25zdCBXZWJDb3JlOjpBdXRo
ZW50aWNhdGlvbkNoYWxsZW5nZSYpIE9WRVJSSURFOwogICAgIHZpcnR1YWwgdm9pZCBkaWRDYW5j
ZWxBdXRoZW50aWNhdGlvbkNoYWxsZW5nZShXZWJDb3JlOjpSZXNvdXJjZUhhbmRsZSosIGNvbnN0
IFdlYkNvcmU6OkF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdlJikgT1ZFUlJJREU7CisgICAgdmlydHVh
bCB2b2lkIHJlY2VpdmVkQ2FuY2VsbGF0aW9uKFdlYkNvcmU6OlJlc291cmNlSGFuZGxlKiwgY29u
c3QgV2ViQ29yZTo6QXV0aGVudGljYXRpb25DaGFsbGVuZ2UmKSBPVkVSUklERTsKICAgICB2aXJ0
dWFsIGJvb2wgdXNlc0FzeW5jQ2FsbGJhY2tzKCkgT1ZFUlJJREUgeyByZXR1cm4gdHJ1ZTsgfQog
CiAjaWYgVVNFKFBST1RFQ1RJT05fU1BBQ0VfQVVUSF9DQUxMQkFDSykK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220153</attachid>
            <date>2013-12-31 00:39:11 -0800</date>
            <delta_ts>2014-01-09 10:15:21 -0800</delta_ts>
            <desc>Updated patch</desc>
            <filename>wk-auth-cancel-network-process.diff</filename>
            <type>text/plain</type>
            <size>3798</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCAxMzY3MGIyLi5jYTdlMTI5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjEg
QEAKKzIwMTMtMTItMzAgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFtHVEtdIFRlc3QgL3dlYmtpdDIvV2ViS2l0V2ViVmlldy9hdXRoZW50aWNh
dGlvbi1jYW5jZWwgZmFpbHMgd2l0aCBuZXR3b3JrIHByb2Nlc3MgZW5hYmxlZAorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI2MTI5CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSW1wbGVtZW50IFJlc291cmNl
SGFuZGxlQ2xpZW50OjpyZWNlaXZlZENhbmNlbGxhdGlvbigpIGluIHRoZQorICAgICAgICBuZXR3
b3JrIHByb2Nlc3MgdG8gY2FuY2VsIHRoZSBsb2FkIHdoZW4gY2FuY2VsbGF0aW9uIGlzIHJlY2Vp
dmVkCisgICAgICAgIGZyb20gdGhlIGF1dGhlbnRpY2F0aW9uIG1hbmFnZXIgbGlrZSB0aGUgd2Vi
IHByb2Nlc3MgZG9lcy4KKworICAgICAgICAqIE5ldHdvcmtQcm9jZXNzL05ldHdvcmtSZXNvdXJj
ZUxvYWRlci5jcHA6CisgICAgICAgIChXZWJLaXQ6Ok5ldHdvcmtSZXNvdXJjZUxvYWRlcjo6Y2Fu
Y2VsKTogSGVscGVyIGZ1bmN0aW9uIHRvIGNhbmNlbAorICAgICAgICB0aGUgbG9hZC4KKyAgICAg
ICAgKFdlYktpdDo6TmV0d29ya1Jlc291cmNlTG9hZGVyOjpjb250aW51ZVdpbGxTZW5kUmVxdWVz
dCk6IENhbGwgY2FuY2VsKCkuCisgICAgICAgIChXZWJLaXQ6Ok5ldHdvcmtSZXNvdXJjZUxvYWRl
cjo6cmVjZWl2ZWRDYW5jZWxsYXRpb24pOiBEaXR0by4KKyAgICAgICAgKiBOZXR3b3JrUHJvY2Vz
cy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuaDoKKwogMjAxMy0xMi0yNyAgQ2FybG9zIEdhcmNpYSBD
YW1wb3MgIDxjZ2FyY2lhQGlnYWxpYS5jb20+CiAKICAgICAgICAgQ3Jhc2ggd2hlbiBzdGFydGlu
ZyBhIGRvd25sb2FkIGJlZm9yZSB0aGUgbmV0d29yayBwcm9jZXNzIGhhcyBiZWVuIGxhdW5jaGVk
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3Vy
Y2VMb2FkZXIuY3BwIGIvU291cmNlL1dlYktpdDIvTmV0d29ya1Byb2Nlc3MvTmV0d29ya1Jlc291
cmNlTG9hZGVyLmNwcAppbmRleCAwNzM4OTI5Li44ODlmMWMzIDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuY3BwCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL05ldHdvcmtSZXNvdXJjZUxvYWRlci5jcHAKQEAg
LTE3MSw2ICsxNzEsMTMgQEAgdm9pZCBOZXR3b3JrUmVzb3VyY2VMb2FkZXI6OmFib3J0KCkKICAg
ICBjbGVhbnVwKCk7CiB9CiAKK3ZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpjYW5jZWwoKQor
eworICAgIEFTU0VSVChtX2hhbmRsZSk7CisgICAgbV9oYW5kbGUtPmNhbmNlbCgpOworICAgIGRp
ZEZhaWwobV9oYW5kbGUuZ2V0KCksIGNhbmNlbGxlZEVycm9yKG1fcmVxdWVzdCkpOworfQorCiB2
b2lkIE5ldHdvcmtSZXNvdXJjZUxvYWRlcjo6ZGlkUmVjZWl2ZVJlc3BvbnNlQXN5bmMoUmVzb3Vy
Y2VIYW5kbGUqIGhhbmRsZSwgY29uc3QgUmVzb3VyY2VSZXNwb25zZSYgcmVzcG9uc2UpCiB7CiAg
ICAgQVNTRVJUX1VOVVNFRChoYW5kbGUsIGhhbmRsZSA9PSBtX2hhbmRsZSk7CkBAIC0yNTcsMTAg
KzI2NCw4IEBAIHZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpjb250aW51ZVdpbGxTZW5kUmVx
dWVzdChjb25zdCBSZXNvdXJjZVJlcXVlc3QmIG5ld1JlCiAKICAgICBtX2hhbmRsZS0+Y29udGlu
dWVXaWxsU2VuZFJlcXVlc3QobV9yZXF1ZXN0KTsKIAotICAgIGlmIChtX3JlcXVlc3QuaXNOdWxs
KCkpIHsKLSAgICAgICAgbV9oYW5kbGUtPmNhbmNlbCgpOwotICAgICAgICBkaWRGYWlsKG1faGFu
ZGxlLmdldCgpLCBjYW5jZWxsZWRFcnJvcihtX3JlcXVlc3QpKTsKLSAgICB9CisgICAgaWYgKG1f
cmVxdWVzdC5pc051bGwoKSkKKyAgICAgICAgY2FuY2VsKCk7CiB9CiAKIHZvaWQgTmV0d29ya1Jl
c291cmNlTG9hZGVyOjpjb250aW51ZURpZFJlY2VpdmVSZXNwb25zZSgpCkBAIC0zMzYsNiArMzQx
LDEyIEBAIHZvaWQgTmV0d29ya1Jlc291cmNlTG9hZGVyOjpkaWRDYW5jZWxBdXRoZW50aWNhdGlv
bkNoYWxsZW5nZShSZXNvdXJjZUhhbmRsZSogaGFuCiAgICAgbm90SW1wbGVtZW50ZWQoKTsKIH0K
IAordm9pZCBOZXR3b3JrUmVzb3VyY2VMb2FkZXI6OnJlY2VpdmVkQ2FuY2VsbGF0aW9uKFJlc291
cmNlSGFuZGxlKiBoYW5kbGUsIGNvbnN0IEF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdlJiBjaGFsbGVu
Z2UpCit7CisgICAgQVNTRVJUX1VOVVNFRChoYW5kbGUsIGhhbmRsZSA9PSBtX2hhbmRsZSk7Cisg
ICAgY2FuY2VsKCk7Cit9CisKIElQQzo6Q29ubmVjdGlvbiogTmV0d29ya1Jlc291cmNlTG9hZGVy
OjptZXNzYWdlU2VuZGVyQ29ubmVjdGlvbigpCiB7CiAgICAgcmV0dXJuIGNvbm5lY3Rpb25Ub1dl
YlByb2Nlc3MoKS0+Y29ubmVjdGlvbigpOwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvTmV0
d29ya1Byb2Nlc3MvTmV0d29ya1Jlc291cmNlTG9hZGVyLmggYi9Tb3VyY2UvV2ViS2l0Mi9OZXR3
b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3VyY2VMb2FkZXIuaAppbmRleCA5ZDIzMDMyLi5lYmQwZDBj
IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVzb3Vy
Y2VMb2FkZXIuaAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrUmVz
b3VyY2VMb2FkZXIuaApAQCAtNzcsNiArNzcsNyBAQCBwdWJsaWM6CiAKICAgICB2b2lkIHN0YXJ0
KCk7CiAgICAgdm9pZCBhYm9ydCgpOworICAgIHZvaWQgY2FuY2VsKCk7CiAKICAgICAvLyBSZXNv
dXJjZUhhbmRsZUNsaWVudCBtZXRob2RzCiAgICAgdmlydHVhbCB2b2lkIHdpbGxTZW5kUmVxdWVz
dEFzeW5jKFdlYkNvcmU6OlJlc291cmNlSGFuZGxlKiwgY29uc3QgV2ViQ29yZTo6UmVzb3VyY2VS
ZXF1ZXN0JiwgY29uc3QgV2ViQ29yZTo6UmVzb3VyY2VSZXNwb25zZSYgcmVkaXJlY3RSZXNwb25z
ZSkgT1ZFUlJJREU7CkBAIC05Miw2ICs5Myw3IEBAIHB1YmxpYzoKICAgICB2aXJ0dWFsIHZvaWQg
c2hvdWxkVXNlQ3JlZGVudGlhbFN0b3JhZ2VBc3luYyhXZWJDb3JlOjpSZXNvdXJjZUhhbmRsZSop
IE9WRVJSSURFOwogICAgIHZpcnR1YWwgdm9pZCBkaWRSZWNlaXZlQXV0aGVudGljYXRpb25DaGFs
bGVuZ2UoV2ViQ29yZTo6UmVzb3VyY2VIYW5kbGUqLCBjb25zdCBXZWJDb3JlOjpBdXRoZW50aWNh
dGlvbkNoYWxsZW5nZSYpIE9WRVJSSURFOwogICAgIHZpcnR1YWwgdm9pZCBkaWRDYW5jZWxBdXRo
ZW50aWNhdGlvbkNoYWxsZW5nZShXZWJDb3JlOjpSZXNvdXJjZUhhbmRsZSosIGNvbnN0IFdlYkNv
cmU6OkF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdlJikgT1ZFUlJJREU7CisgICAgdmlydHVhbCB2b2lk
IHJlY2VpdmVkQ2FuY2VsbGF0aW9uKFdlYkNvcmU6OlJlc291cmNlSGFuZGxlKiwgY29uc3QgV2Vi
Q29yZTo6QXV0aGVudGljYXRpb25DaGFsbGVuZ2UmKSBPVkVSUklERTsKICAgICB2aXJ0dWFsIGJv
b2wgdXNlc0FzeW5jQ2FsbGJhY2tzKCkgT1ZFUlJJREUgeyByZXR1cm4gdHJ1ZTsgfQogCiAjaWYg
VVNFKFBST1RFQ1RJT05fU1BBQ0VfQVVUSF9DQUxMQkFDSykK
</data>
<flag name="review"
          id="243830"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>