<?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>189056</bug_id>
          
          <creation_ts>2018-08-28 13:38:50 -0700</creation_ts>
          <short_desc>[Curl][WebKit] Bugfix on redirect.</short_desc>
          <delta_ts>2018-08-30 11:14:02 -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>Platform</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Basuke Suzuki">basuke</reporter>
          <assigned_to name="Basuke Suzuki">basuke</assigned_to>
          <cc>achristensen</cc>
    
    <cc>basuke</cc>
    
    <cc>commit-queue</cc>
    
    <cc>don.olmstead</cc>
    
    <cc>fujii</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1454525</commentid>
    <comment_count>0</comment_count>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2018-08-28 13:38:50 -0700</bug_when>
    <thetext>Forget to call request update method.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1454528</commentid>
    <comment_count>1</comment_count>
      <attachid>348335</attachid>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2018-08-28 13:43:07 -0700</bug_when>
    <thetext>Created attachment 348335
PATCH</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1454902</commentid>
    <comment_count>2</comment_count>
      <attachid>348335</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2018-08-29 13:15:08 -0700</bug_when>
    <thetext>Comment on attachment 348335
PATCH

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

LGTM from my informal review.

&gt; Source/WebKit/NetworkProcess/NetworkLoad.cpp:154
&gt; +#elif USE(SOUP) || USE(CURL)

Minor nit. Should this just be an #else?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1455105</commentid>
    <comment_count>3</comment_count>
      <attachid>348335</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2018-08-29 18:55:51 -0700</bug_when>
    <thetext>Comment on attachment 348335
PATCH

Yep</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1455269</commentid>
    <comment_count>4</comment_count>
      <attachid>348504</attachid>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2018-08-30 10:34:59 -0700</bug_when>
    <thetext>Created attachment 348504
fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1455271</commentid>
    <comment_count>5</comment_count>
    <who name="Basuke Suzuki">basuke</who>
    <bug_when>2018-08-30 10:36:34 -0700</bug_when>
    <thetext>Thanks. I forgot we can forget about AppleWin in this component.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1455300</commentid>
    <comment_count>6</comment_count>
      <attachid>348504</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-08-30 11:13:09 -0700</bug_when>
    <thetext>Comment on attachment 348504
fix

Clearing flags on attachment: 348504

Committed r235510: &lt;https://trac.webkit.org/changeset/235510&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1455301</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-08-30 11:13:10 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>348335</attachid>
            <date>2018-08-28 13:43:07 -0700</date>
            <delta_ts>2018-08-30 10:34:59 -0700</delta_ts>
            <desc>PATCH</desc>
            <filename>189056.dif</filename>
            <type>text/plain</type>
            <size>1499</size>
            <attacher name="Basuke Suzuki">basuke</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu
Z2VMb2cKaW5kZXggZjZiODcyMjIyOGEuLjlkZTFhYjJjMmU0IDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0L0NoYW5nZUxvZwpAQCAtMSwzICsx
LDE1IEBACisyMDE4LTA4LTI4ICBCYXN1a2UgU3V6dWtpICA8QmFzdWtlLlN1enVraUBzb255LmNv
bT4KKworICAgICAgICBbQ3VybF1bV2ViS2l0XSBCdWdmaXggb24gcmVkaXJlY3QuCisgICAgICAg
IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODkwNTYKKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb3JnZXQgdG8gY2FsbCB1
cGRhdGVGcm9tRGVsZWdhdGVQcmVzZXJ2aW5nT2xkUHJvcGVydGllcyBpbiBjb250aW51ZVdpbGxT
ZW5kUmVxdWVzdCBsaWtlIG90aGVyIHBvcnRzIGRvLgorCisgICAgICAgICogTmV0d29ya1Byb2Nl
c3MvTmV0d29ya0xvYWQuY3BwOgorICAgICAgICAoV2ViS2l0OjpOZXR3b3JrTG9hZDo6Y29udGlu
dWVXaWxsU2VuZFJlcXVlc3QpOgorCiAyMDE4LTA4LTI0ICBCYXN1a2UgU3V6dWtpICA8QmFzdWtl
LlN1enVraUBzb255LmNvbT4KIAogICAgICAgICBbQ3VybF0gTWF0Y2ggdGhlIGludGVyZmFjZSB1
c2VkIGluIE5ldHdvcmtEYXRhVGFzayBhbmQgUmVzb3VyY2VIYW5kbGUuCmRpZmYgLS1naXQgYS9T
b3VyY2UvV2ViS2l0L05ldHdvcmtQcm9jZXNzL05ldHdvcmtMb2FkLmNwcCBiL1NvdXJjZS9XZWJL
aXQvTmV0d29ya1Byb2Nlc3MvTmV0d29ya0xvYWQuY3BwCmluZGV4IDM3OGE1NDg0NDFkLi5hOGMz
NmNkYjE2NyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3Jr
TG9hZC5jcHAKKysrIGIvU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrTG9hZC5j
cHAKQEAgLTE1MSw3ICsxNTEsNyBAQCB2b2lkIE5ldHdvcmtMb2FkOjpjb250aW51ZVdpbGxTZW5k
UmVxdWVzdChXZWJDb3JlOjpSZXNvdXJjZVJlcXVlc3QmJiBuZXdSZXF1ZXN0KQogewogI2lmIFBM
QVRGT1JNKENPQ09BKQogICAgIG1fY3VycmVudFJlcXVlc3QudXBkYXRlRnJvbURlbGVnYXRlUHJl
c2VydmluZ09sZFByb3BlcnRpZXMobmV3UmVxdWVzdC5uc1VSTFJlcXVlc3QoSFRUUEJvZHlVcGRh
dGVQb2xpY3k6OkRvTm90VXBkYXRlSFRUUEJvZHkpKTsKLSNlbGlmIFVTRShTT1VQKQorI2VsaWYg
VVNFKFNPVVApIHx8IFVTRShDVVJMKQogICAgIC8vIEZJWE1FOiBJbXBsZW1lbnQgUmVzb3VyY2VS
ZXF1ZXN0Ojp1cGRhdGVGcm9tRGVsZWdhdGVQcmVzZXJ2aW5nT2xkUHJvcGVydGllcy4gU2VlIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjYxMjcuCiAgICAgbV9jdXJy
ZW50UmVxdWVzdC51cGRhdGVGcm9tRGVsZWdhdGVQcmVzZXJ2aW5nT2xkUHJvcGVydGllcyhuZXdS
ZXF1ZXN0KTsKICNlbmRpZgo=
</data>
<flag name="review"
          id="365989"
          type_id="1"
          status="+"
          setter="achristensen"
    />
    <flag name="commit-queue"
          id="365990"
          type_id="3"
          status="-"
          setter="achristensen"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>348504</attachid>
            <date>2018-08-30 10:34:59 -0700</date>
            <delta_ts>2018-08-30 11:13:09 -0700</delta_ts>
            <desc>fix</desc>
            <filename>189056.diff</filename>
            <type>text/plain</type>
            <size>1476</size>
            <attacher name="Basuke Suzuki">basuke</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu
Z2VMb2cKaW5kZXggZjZiODcyMjIyOGEuLjlkZTFhYjJjMmU0IDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0L0NoYW5nZUxvZwpAQCAtMSwzICsx
LDE1IEBACisyMDE4LTA4LTI4ICBCYXN1a2UgU3V6dWtpICA8QmFzdWtlLlN1enVraUBzb255LmNv
bT4KKworICAgICAgICBbQ3VybF1bV2ViS2l0XSBCdWdmaXggb24gcmVkaXJlY3QuCisgICAgICAg
IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODkwNTYKKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb3JnZXQgdG8gY2FsbCB1
cGRhdGVGcm9tRGVsZWdhdGVQcmVzZXJ2aW5nT2xkUHJvcGVydGllcyBpbiBjb250aW51ZVdpbGxT
ZW5kUmVxdWVzdCBsaWtlIG90aGVyIHBvcnRzIGRvLgorCisgICAgICAgICogTmV0d29ya1Byb2Nl
c3MvTmV0d29ya0xvYWQuY3BwOgorICAgICAgICAoV2ViS2l0OjpOZXR3b3JrTG9hZDo6Y29udGlu
dWVXaWxsU2VuZFJlcXVlc3QpOgorCiAyMDE4LTA4LTI0ICBCYXN1a2UgU3V6dWtpICA8QmFzdWtl
LlN1enVraUBzb255LmNvbT4KIAogICAgICAgICBbQ3VybF0gTWF0Y2ggdGhlIGludGVyZmFjZSB1
c2VkIGluIE5ldHdvcmtEYXRhVGFzayBhbmQgUmVzb3VyY2VIYW5kbGUuCmRpZmYgLS1naXQgYS9T
b3VyY2UvV2ViS2l0L05ldHdvcmtQcm9jZXNzL05ldHdvcmtMb2FkLmNwcCBiL1NvdXJjZS9XZWJL
aXQvTmV0d29ya1Byb2Nlc3MvTmV0d29ya0xvYWQuY3BwCmluZGV4IDM3OGE1NDg0NDFkLi4xZjAw
YWQ3N2MzNiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3Jr
TG9hZC5jcHAKKysrIGIvU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrTG9hZC5j
cHAKQEAgLTE1MSw3ICsxNTEsNyBAQCB2b2lkIE5ldHdvcmtMb2FkOjpjb250aW51ZVdpbGxTZW5k
UmVxdWVzdChXZWJDb3JlOjpSZXNvdXJjZVJlcXVlc3QmJiBuZXdSZXF1ZXN0KQogewogI2lmIFBM
QVRGT1JNKENPQ09BKQogICAgIG1fY3VycmVudFJlcXVlc3QudXBkYXRlRnJvbURlbGVnYXRlUHJl
c2VydmluZ09sZFByb3BlcnRpZXMobmV3UmVxdWVzdC5uc1VSTFJlcXVlc3QoSFRUUEJvZHlVcGRh
dGVQb2xpY3k6OkRvTm90VXBkYXRlSFRUUEJvZHkpKTsKLSNlbGlmIFVTRShTT1VQKQorI2Vsc2UK
ICAgICAvLyBGSVhNRTogSW1wbGVtZW50IFJlc291cmNlUmVxdWVzdDo6dXBkYXRlRnJvbURlbGVn
YXRlUHJlc2VydmluZ09sZFByb3BlcnRpZXMuIFNlZSBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTI2MTI3LgogICAgIG1fY3VycmVudFJlcXVlc3QudXBkYXRlRnJvbURl
bGVnYXRlUHJlc2VydmluZ09sZFByb3BlcnRpZXMobmV3UmVxdWVzdCk7CiAjZW5kaWYK
</data>

          </attachment>
      

    </bug>

</bugzilla>