Bug 179641

Summary: FetchResponse should keep unfiltered ResourceResponse so that it can be used in Service Worker
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebCore Misc.Assignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, beidson, cdumez, commit-queue, dbates, esprehn+autocc, ews-watchlist, japhet, kangil.han, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=207720
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Archive of layout-test-results from ews106 for mac-elcapitan-wk2
none
Patch
none
Patch
none
Archive of layout-test-results from ews104 for mac-elcapitan-wk2 none

Description youenn fablet 2017-11-13 15:35:08 PST
Currently a fetch response filters its ResourceResponse.
Comment 1 youenn fablet 2017-11-13 16:08:01 PST
Created attachment 326819 [details]
Patch
Comment 2 youenn fablet 2017-11-17 13:18:27 PST
Created attachment 327221 [details]
Patch
Comment 3 youenn fablet 2017-11-17 13:35:02 PST
Created attachment 327226 [details]
Patch
Comment 4 EWS Watchlist 2017-11-17 14:40:27 PST
Comment on attachment 327226 [details]
Patch

Attachment 327226 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5280639

New failing tests:
imported/w3c/web-platform-tests/service-workers/service-worker/fetch-frame-resource.https.html
Comment 5 EWS Watchlist 2017-11-17 14:40:29 PST
Created attachment 327242 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 6 Radar WebKit Bug Importer 2017-12-07 16:52:25 PST
<rdar://problem/35923570>
Comment 7 youenn fablet 2017-12-07 17:00:39 PST
Created attachment 328757 [details]
Patch
Comment 8 youenn fablet 2017-12-07 18:01:51 PST
Created attachment 328767 [details]
Patch
Comment 9 EWS Watchlist 2017-12-07 19:36:21 PST
Comment on attachment 328767 [details]
Patch

Attachment 328767 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5537882

New failing tests:
imported/w3c/web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https.html
Comment 10 EWS Watchlist 2017-12-07 19:36:22 PST
Created attachment 328781 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 11 youenn fablet 2017-12-08 09:35:07 PST
(In reply to Build Bot from comment #9)
> Comment on attachment 328767 [details]
> Patch
> 
> Attachment 328767 [details] did not pass mac-wk2-ews (mac-wk2):
> Output: http://webkit-queues.webkit.org/results/5537882
> 
> New failing tests:
> imported/w3c/web-platform-tests/html/webappapis/scripting/events/
> messageevent-constructor.https.html

Crash is unrelated to this patch (see https://bugs.webkit.org/show_bug.cgi?id=180518).
Comment 12 WebKit Commit Bot 2017-12-08 14:41:52 PST
Comment on attachment 328767 [details]
Patch

Clearing flags on attachment: 328767

Committed r225702: <https://trac.webkit.org/changeset/225702>
Comment 13 WebKit Commit Bot 2017-12-08 14:41:54 PST
All reviewed patches have been landed.  Closing bug.