<?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>208733</bug_id>
          
          <creation_ts>2020-03-06 13:37:40 -0800</creation_ts>
          <short_desc>Evaluating JavaScript in main frame before loading should succeed</short_desc>
          <delta_ts>2020-03-07 11:28:26 -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>New Bugs</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=208767</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alex Christensen">achristensen</reporter>
          <assigned_to name="Alex Christensen">achristensen</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>ap</cc>
    
    <cc>beidson</cc>
    
    <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>thorton</cc>
    
    <cc>webkit-bot-watchers-bugzilla</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1626845</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2020-03-06 13:37:40 -0800</bug_when>
    <thetext>Evaluating JavaScript in main frame before loading should succeed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1626847</commentid>
    <comment_count>1</comment_count>
      <attachid>392765</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2020-03-06 13:39:27 -0800</bug_when>
    <thetext>Created attachment 392765
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1626859</commentid>
    <comment_count>2</comment_count>
      <attachid>392765</attachid>
    <who name="Tim Horton">thorton</who>
    <bug_when>2020-03-06 14:04:18 -0800</bug_when>
    <thetext>Comment on attachment 392765
Patch

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

&gt; Source/WebKit/ChangeLog:8
&gt; +        This fixes a bug in the SPI I introduced in r257882 with a test.

And what is the bug! Why not write like 10 more words? :D</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1626860</commentid>
    <comment_count>3</comment_count>
      <attachid>392765</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2020-03-06 14:05:03 -0800</bug_when>
    <thetext>Comment on attachment 392765
Patch

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

&gt;&gt; Source/WebKit/ChangeLog:8
&gt;&gt; +        This fixes a bug in the SPI I introduced in r257882 with a test.
&gt; 
&gt; And what is the bug! Why not write like 10 more words? :D

The title is that description.  It only needed 9 words.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1627026</commentid>
    <comment_count>4</comment_count>
      <attachid>392765</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-03-06 17:35:46 -0800</bug_when>
    <thetext>Comment on attachment 392765
Patch

Clearing flags on attachment: 392765

Committed r258042: &lt;https://trac.webkit.org/changeset/258042&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1627027</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-03-06 17:35:47 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1627028</commentid>
    <comment_count>6</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-03-06 17:36:18 -0800</bug_when>
    <thetext>&lt;rdar://problem/60177714&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1627258</commentid>
    <comment_count>7</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2020-03-07 11:27:04 -0800</bug_when>
    <thetext>&gt; Committed r258042: &lt;https://trac.webkit.org/changeset/258042&gt;
This seems to have broken API test TestWebKitAPI.URLSchemeHandler.Frames, tracked in Bug 208733.

This was also indicated by EWS in https://ews-build.webkit.org/#/builders/9/builds/18987</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1627259</commentid>
    <comment_count>8</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2020-03-07 11:28:26 -0800</bug_when>
    <thetext>Tracked in Bug 208767.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>392765</attachid>
            <date>2020-03-06 13:39:27 -0800</date>
            <delta_ts>2020-03-06 17:35:46 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-208733-20200306133926.patch</filename>
            <type>text/plain</type>
            <size>3709</size>
            <attacher name="Alex Christensen">achristensen</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL
aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNTc5NjMpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv
Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDIwLTAzLTA2ICBBbGV4IENocmlz
dGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgRXZhbHVhdGluZyBK
YXZhU2NyaXB0IGluIG1haW4gZnJhbWUgYmVmb3JlIGxvYWRpbmcgc2hvdWxkIHN1Y2NlZWQKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwODczMworCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgZml4ZXMg
YSBidWcgaW4gdGhlIFNQSSBJIGludHJvZHVjZWQgaW4gcjI1Nzg4MiB3aXRoIGEgdGVzdC4KKwor
ICAgICAgICAqIFVJUHJvY2Vzcy9BUEkvQ29jb2EvV0tXZWJWaWV3Lm1tOgorICAgICAgICAoLVtX
S1dlYlZpZXcgX2V2YWx1YXRlSmF2YVNjcmlwdDphc0FzeW5jRnVuY3Rpb246d2l0aEFyZ3VtZW50
czpmb3JjZVVzZXJHZXN0dXJlOmluRnJhbWU6aW5Xb3JsZDpjb21wbGV0aW9uSGFuZGxlcjpdKToK
KwogMjAyMC0wMy0wNSAgQWxleCBDaHJpc3RlbnNlbiAgPGFjaHJpc3RlbnNlbkB3ZWJraXQub3Jn
PgogCiAgICAgICAgIEFkZCBXS05hdmlnYXRpb25EZWxlZ2F0ZVByaXZhdGUgU1BJIHRvIGluZm9y
bSBhcHBsaWNhdGlvbiBvZiBzdWJmcmFtZSBuYXZpZ2F0aW9ucwpJbmRleDogU291cmNlL1dlYktp
dC9VSVByb2Nlc3MvQVBJL0NvY29hL1dLV2ViVmlldy5tbQo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv
V2ViS2l0L1VJUHJvY2Vzcy9BUEkvQ29jb2EvV0tXZWJWaWV3Lm1tCShyZXZpc2lvbiAyNTc5NjMp
CisrKyBTb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9BUEkvQ29jb2EvV0tXZWJWaWV3Lm1tCSh3b3Jr
aW5nIGNvcHkpCkBAIC05MjAsOCArOTIwLDEwIEBAIC0gKHZvaWQpX2V2YWx1YXRlSmF2YVNjcmlw
dDooTlNTdHJpbmcgKikKICAgICB9CiAgICAgCiAgICAgT3B0aW9uYWw8V2ViQ29yZTo6RnJhbWVJ
ZGVudGlmaWVyPiBmcmFtZUlEOwotICAgIGlmIChmcmFtZSkKLSAgICAgICAgZnJhbWVJRCA9IG1h
a2VPYmplY3RJZGVudGlmaWVyPFdlYkNvcmU6OkZyYW1lSWRlbnRpZmllclR5cGU+KGZyYW1lLl9o
YW5kbGUuZnJhbWVJRCk7CisgICAgaWYgKGZyYW1lKSB7CisgICAgICAgIGlmICh1aW50NjRfdCBp
ZGVudGlmaWVyID0gZnJhbWUuX2hhbmRsZS5mcmFtZUlEKQorICAgICAgICAgICAgZnJhbWVJRCA9
IG1ha2VPYmplY3RJZGVudGlmaWVyPFdlYkNvcmU6OkZyYW1lSWRlbnRpZmllclR5cGU+KGlkZW50
aWZpZXIpOworICAgIH0KIAogICAgIF9wYWdlLT5ydW5KYXZhU2NyaXB0SW5GcmFtZUluU2NyaXB0
V29ybGQoV2ViQ29yZTo6UnVuSmF2YVNjcmlwdFBhcmFtZXRlcnMgeyBqYXZhU2NyaXB0U3RyaW5n
LCAhIWFzQXN5bmNGdW5jdGlvbiwgV1RGTW92ZShhcmd1bWVudHNNYXApLCAhIWZvcmNlVXNlckdl
c3R1cmUgfSwgZnJhbWVJRCwgKndvcmxkLT5fY29udGVudFdvcmxkLmdldCgpLCBbaGFuZGxlcl0o
QVBJOjpTZXJpYWxpemVkU2NyaXB0VmFsdWUqIHNlcmlhbGl6ZWRTY3JpcHRWYWx1ZSwgT3B0aW9u
YWw8V2ViQ29yZTo6RXhjZXB0aW9uRGV0YWlscz4gZGV0YWlscywgV2ViS2l0OjpTY3JpcHRWYWx1
ZUNhbGxiYWNrOjpFcnJvciBlcnJvckNvZGUpIHsKICAgICAgICAgaWYgKCFoYW5kbGVyKQpJbmRl
eDogVG9vbHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFRvb2xzL0NoYW5nZUxvZwkocmV2aXNp
b24gMjU4MDI3KQorKysgVG9vbHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEs
MTIgQEAKKzIwMjAtMDMtMDYgIEFsZXggQ2hyaXN0ZW5zZW4gIDxhY2hyaXN0ZW5zZW5Ad2Via2l0
Lm9yZz4KKworICAgICAgICBFdmFsdWF0aW5nIEphdmFTY3JpcHQgaW4gbWFpbiBmcmFtZSBiZWZv
cmUgbG9hZGluZyBzaG91bGQgc3VjY2VlZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MjA4NzMzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL1dLVVJM
U2NoZW1lSGFuZGxlci0xLm1tOgorCiAyMDIwLTAzLTA2ICBEYXZpZCBRdWVzYWRhICA8ZGF2aWRf
cXVlc2FkYUBhcHBsZS5jb20+CiAKICAgICAgICAgQWRkIF9XS1Jlc291cmNlTG9hZEluZm8ucmVz
b3VyY2VUeXBlIFNQSQpJbmRleDogVG9vbHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2Nv
YS9XS1VSTFNjaGVtZUhhbmRsZXItMS5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9UZXN0V2ViS2l0
QVBJL1Rlc3RzL1dlYktpdENvY29hL1dLVVJMU2NoZW1lSGFuZGxlci0xLm1tCShyZXZpc2lvbiAy
NTc5NjMpCisrKyBUb29scy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL1dLVVJMU2No
ZW1lSGFuZGxlci0xLm1tCSh3b3JraW5nIGNvcHkpCkBAIC0zMSw2ICszMSw3IEBACiAjaW1wb3J0
ICJUZXN0TmF2aWdhdGlvbkRlbGVnYXRlLmgiCiAjaW1wb3J0ICJUZXN0VVJMU2NoZW1lSGFuZGxl
ci5oIgogI2ltcG9ydCAiVGVzdFdLV2ViVmlldy5oIgorI2ltcG9ydCA8V2ViS2l0L1dLRXJyb3JQ
cml2YXRlLmg+CiAjaW1wb3J0IDxXZWJLaXQvV0tGcmFtZUluZm9Qcml2YXRlLmg+CiAjaW1wb3J0
IDxXZWJLaXQvV0tQcm9jZXNzUG9vbFByaXZhdGUuaD4KICNpbXBvcnQgPFdlYktpdC9XS1VSTFNj
aGVtZUhhbmRsZXIuaD4KQEAgLTEwOTMsNCArMTA5NCwxNiBAQCBURVNUKFVSTFNjaGVtZUhhbmRs
ZXIsIEZyYW1lcykKICAgICAgICAgfV07CiAgICAgfV07CiAgICAgVGVzdFdlYktpdEFQSTo6VXRp
bDo6cnVuKCZkb25lKTsKKyAgICAKKyAgICBkb25lID0gZmFsc2U7CisgICAgYXV0byBlbXB0eVdl
YlZpZXcgPSBhZG9wdE5TKFtXS1dlYlZpZXcgbmV3XSk7CisgICAgW2VtcHR5V2ViVmlldyBfZnJh
bWVzOl4oX1dLRnJhbWVUcmVlTm9kZSAqbWFpbkZyYW1lKSB7CisgICAgICAgIEVYUEVDVF9OT1Rf
TlVMTChtYWluRnJhbWUuX2hhbmRsZSk7CisgICAgICAgIEVYUEVDVF9FUShtYWluRnJhbWUuX2hh
bmRsZS5mcmFtZUlELCAwdSk7CisgICAgICAgIFtlbXB0eVdlYlZpZXcgX2V2YWx1YXRlSmF2YVNj
cmlwdDpAIndpbmRvdy5sb2NhdGlvbi5ocmVmIiBpbkZyYW1lOm1haW5GcmFtZSBpbkNvbnRlbnRX
b3JsZDpbV0tDb250ZW50V29ybGQgZGVmYXVsdENsaWVudFdvcmxkXSBjb21wbGV0aW9uSGFuZGxl
cjpeKGlkIHJlc3VsdCwgTlNFcnJvciAqZXJyb3IpIHsKKyAgICAgICAgICAgIEVYUEVDVF9XS19T
VFJFUShyZXN1bHQsICJhYm91dDpibGFuayIpOworICAgICAgICAgICAgZG9uZSA9IHRydWU7Cisg
ICAgICAgIH1dOworICAgIH1dOworICAgIFRlc3RXZWJLaXRBUEk6OlV0aWw6OnJ1bigmZG9uZSk7
CiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>