Bug 201348

Summary: Remove support for load/error on link=prefetch
Product: WebKit Reporter: Rob Buis <rbuis>
Component: Page LoadingAssignee: Rob Buis <rbuis>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, cdumez, commit-queue, esprehn+autocc, ews-watchlist, gyuyoung.kim, rniwa, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Rob Buis
Reported 2019-08-30 10:56:48 PDT
As discussed in https://bugs.webkit.org/show_bug.cgi?id=199162.
Attachments
Patch (9.84 KB, patch)
2019-08-30 11:09 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (11.74 KB, patch)
2019-08-30 12:15 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (13.77 KB, patch)
2019-08-31 08:38 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (15.99 KB, patch)
2019-08-31 10:07 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (16.29 KB, patch)
2019-09-01 07:59 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (16.29 KB, patch)
2019-09-01 10:10 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (16.39 KB, patch)
2019-09-03 03:04 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Show Obsolete (6) View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2019-08-30 11:09:26 PDT
Created attachment 377733 [details] Patch
Rob Buis
Comment 2 2019-08-30 12:15:35 PDT
Created attachment 377737 [details] Patch
Rob Buis
Comment 3 2019-08-31 08:38:07 PDT
Created attachment 377802 [details] Patch
Rob Buis
Comment 4 2019-08-31 10:07:55 PDT
Created attachment 377803 [details] Patch
Rob Buis
Comment 5 2019-09-01 07:59:48 PDT
Created attachment 377823 [details] Patch
Rob Buis
Comment 6 2019-09-01 10:10:51 PDT
Created attachment 377826 [details] Patch
youenn fablet
Comment 7 2019-09-03 02:22:36 PDT
Comment on attachment 377826 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377826&action=review > LayoutTests/http/tests/cache/link-prefetch-main-resource-iframe.html:39 > + internals.addPrefetchLoadEventListener(link, () => { loadAfterPrefetch(); }); add an if (window.internals) check. > LayoutTests/http/tests/cache/link-prefetch-main-resource.html:17 > + internals.addPrefetchLoadEventListener(link, () => { loadAfterPrefetch(); }); Ditto.
Rob Buis
Comment 8 2019-09-03 03:04:32 PDT
Created attachment 377880 [details] Patch
WebKit Commit Bot
Comment 9 2019-09-03 04:48:17 PDT
Comment on attachment 377880 [details] Patch Clearing flags on attachment: 377880 Committed r249426: <https://trac.webkit.org/changeset/249426>
WebKit Commit Bot
Comment 10 2019-09-03 04:48:19 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 11 2019-09-03 04:49:16 PDT
<rdar://problem/54971865>
Note You need to log in before you can comment on or make changes to this bug.