RESOLVED FIXED 208733
Evaluating JavaScript in main frame before loading should succeed 
https://bugs.webkit.org/show_bug.cgi?id=208733
Summary Evaluating JavaScript in main frame before loading should succeed
Alex Christensen
Reported 2020-03-06 13:37:40 PST
Evaluating JavaScript in main frame before loading should succeed
Attachments
Patch (3.62 KB, patch)
2020-03-06 13:39 PST, Alex Christensen 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2020-03-06 13:39:27 PST
Created attachment 392765 [details] Patch
Tim Horton
Comment 2 2020-03-06 14:04:18 PST
Comment on attachment 392765 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392765&action=review > Source/WebKit/ChangeLog:8 > + 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
Alex Christensen
Comment 3 2020-03-06 14:05:03 PST
Comment on attachment 392765 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392765&action=review >> Source/WebKit/ChangeLog:8 >> + 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 The title is that description. It only needed 9 words.
WebKit Commit Bot
Comment 4 2020-03-06 17:35:46 PST
Comment on attachment 392765 [details] Patch Clearing flags on attachment: 392765 Committed r258042: <https://trac.webkit.org/changeset/258042>
WebKit Commit Bot
Comment 5 2020-03-06 17:35:47 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2020-03-06 17:36:18 PST
<rdar://problem/60177714>
Aakash Jain
Comment 7 2020-03-07 11:27:04 PST
> Committed r258042: <https://trac.webkit.org/changeset/258042> 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
Aakash Jain
Comment 8 2020-03-07 11:28:26 PST
Tracked in Bug 208767.
Note You need to log in before you can comment on or make changes to this bug.