Bug 183299

Summary: didReceiveServerRedirectForProvisionalNavigation is not called in case of document redirection with service worker registration change
Product: WebKit Reporter: youenn fablet <youennf>
Component: Service WorkersAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, achristensen, ap, cdumez, commit-queue, dbates, ews-watchlist, japhet, jbedard, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch achristensen: review+, commit-queue: commit-queue-

Description youenn fablet 2018-03-02 12:17:30 PST
didReceiveServerRedirectForProvisionalNavigation is not called in case of document redirection with service worker registration change
Comment 1 youenn fablet 2018-03-02 12:17:42 PST
<rdar://problem/37547029>
Comment 2 youenn fablet 2018-03-02 12:28:10 PST
Created attachment 334916 [details]
Patch
Comment 3 WebKit Commit Bot 2018-03-06 17:15:41 PST
Comment on attachment 334916 [details]
Patch

Rejecting attachment 334916 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 334916, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
st, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.
Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
error: The last gc run reported the following. Please correct the root cause
and remove /Volumes/Data/EWS/WebKit/.git/gc.log.
Automatic cleanup will not be performed until the file is removed.

warning: There are too many unreachable loose objects; run 'git prune' to remove them.


Full output: http://webkit-queues.webkit.org/results/6833744
Comment 4 youenn fablet 2018-03-06 17:17:46 PST
Apparently, it did go through EWS, not sure what is happening there.
Comment 5 youenn fablet 2018-03-06 17:18:36 PST
EWS seems to have landed it as https://trac.webkit.org/r229349 despite the error message
Comment 6 Aakash Jain 2018-03-06 17:55:58 PST
> Apparently, it did go through EWS, not sure what is happening there.
I am also not sure what was the issue here. Tracking in https://bugs.webkit.org/show_bug.cgi?id=183392