<?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>210696</bug_id>
          
          <creation_ts>2020-04-18 12:58:54 -0700</creation_ts>
          <short_desc>Remove unneeded code from FrameLoader::loadURL</short_desc>
          <delta_ts>2020-04-19 23:21:14 -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>New Bugs</component>
          <version>Safari Technology Preview</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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Rob Buis">rbuis</reporter>
          <assigned_to name="Rob Buis">rbuis</assigned_to>
          <cc>cdumez</cc>
    
    <cc>darin</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>japhet</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1643037</commentid>
    <comment_count>0</comment_count>
    <who name="Rob Buis">rbuis</who>
    <bug_when>2020-04-18 12:58:54 -0700</bug_when>
    <thetext>Remove unneeded code from FrameLoader::loadURL, since the only way the load type can be Reload
is if loadFrameRequest set it, and the only way loadFrameRequest can set it is if cachePolicy
is ReloadIgnoringCacheData, so no need to set it again in FrameLoader::loadURL.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1643038</commentid>
    <comment_count>1</comment_count>
      <attachid>396861</attachid>
    <who name="Rob Buis">rbuis</who>
    <bug_when>2020-04-18 13:00:59 -0700</bug_when>
    <thetext>Created attachment 396861
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1643344</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-04-19 23:20:11 -0700</bug_when>
    <thetext>Committed r260351: &lt;https://trac.webkit.org/changeset/260351&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 396861.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1643346</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-04-19 23:21:14 -0700</bug_when>
    <thetext>&lt;rdar://problem/62044972&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>396861</attachid>
            <date>2020-04-18 13:00:59 -0700</date>
            <delta_ts>2020-04-19 23:20:11 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-210696-20200418220058.patch</filename>
            <type>text/plain</type>
            <size>1601</size>
            <attacher name="Rob Buis">rbuis</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjYwMjc1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNDU3OTM3NjNhMjg4ZTRi
ZmVjZjM5ZDcxMzkwOTM4YTc5MGI4ZWQyOC4uNzAyYWFjMWEzOGFiMTAyNjRhY2FmOTVhNjkxZTc1
YmYyMzIxODAyMyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDIwLTA0LTE4ICBSb2Ig
QnVpcyAgPHJidWlzQGlnYWxpYS5jb20+CisKKyAgICAgICAgUmVtb3ZlIHVubmVlZGVkIGNvZGUg
ZnJvbSBGcmFtZUxvYWRlcjo6bG9hZFVSTAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MjEwNjk2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgUmVtb3ZlIHVubmVlZGVkIGNvZGUgZnJvbSBGcmFtZUxvYWRlcjo6
bG9hZFVSTCwgc2luY2UgdGhlIG9ubHkgd2F5IHRoZSBsb2FkIHR5cGUgY2FuIGJlIFJlbG9hZAor
ICAgICAgICBpcyBpZiBsb2FkRnJhbWVSZXF1ZXN0IHNldCBpdCwgYW5kIHRoZSBvbmx5IHdheSBs
b2FkRnJhbWVSZXF1ZXN0IGNhbiBzZXQgaXQgaXMgaWYgY2FjaGVQb2xpY3kKKyAgICAgICAgaXMg
UmVsb2FkSWdub3JpbmdDYWNoZURhdGEsIHNvIG5vIG5lZWQgdG8gc2V0IGl0IGFnYWluIGluIEZy
YW1lTG9hZGVyOjpsb2FkVVJMLgorCisgICAgICAgICogbG9hZGVyL0ZyYW1lTG9hZGVyLmNwcDoK
KyAgICAgICAgKFdlYkNvcmU6OkZyYW1lTG9hZGVyOjpsb2FkVVJMKToKKwogMjAyMC0wNC0xNyAg
UnlhbiBIYWRkYWQgIDxyeWFuaGFkZGFkQGFwcGxlLmNvbT4KIAogICAgICAgICBVbnJldmlld2Vk
LCByZXZlcnRpbmcgcjI2MDI0NS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9G
cmFtZUxvYWRlci5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9sb2FkZXIvRnJhbWVMb2FkZXIuY3BwCmlu
ZGV4IGM2YzNmZjkxYzJhZTk3NTYxNWI1ODRjZTUwYjZhYTk3NGQyOGFhMGIuLjhhYTI2OTI2MTI2
MDUyNDBjZGM3YTU5YWE3YmViNGI2MzhmNjcwZTcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3Jl
L2xvYWRlci9GcmFtZUxvYWRlci5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvbG9hZGVyL0ZyYW1l
TG9hZGVyLmNwcApAQCAtMTM1NCw4ICsxMzU0LDYgQEAgdm9pZCBGcmFtZUxvYWRlcjo6bG9hZFVS
TChGcmFtZUxvYWRSZXF1ZXN0JiYgZnJhbWVMb2FkUmVxdWVzdCwgY29uc3QgU3RyaW5nJiByZWYK
ICAgICAgICAgcmVxdWVzdC5zZXRIVFRQUmVmZXJyZXIocmVmZXJyZXIpOwogCiAgICAgYWRkRXh0
cmFGaWVsZHNUb1JlcXVlc3QocmVxdWVzdCwgSXNNYWluUmVzb3VyY2U6OlllcywgbmV3TG9hZFR5
cGUpOwotICAgIGlmIChpc1JlbG9hZChuZXdMb2FkVHlwZSkpCi0gICAgICAgIHJlcXVlc3Quc2V0
Q2FjaGVQb2xpY3koUmVzb3VyY2VSZXF1ZXN0Q2FjaGVQb2xpY3k6OlJlbG9hZElnbm9yaW5nQ2Fj
aGVEYXRhKTsKIAogICAgIEFTU0VSVChuZXdMb2FkVHlwZSAhPSBGcmFtZUxvYWRUeXBlOjpTYW1l
KTsKIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>