Don't bother passing DNT information during redirects. It's only ever used as a fingerprinting component.
<rdar://problem/45555965>
Created attachment 358190 [details] Patch
Comment on attachment 358190 [details] Patch Attachment 358190 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/10606969 New failing tests: http/wpt/fetch/dnt-header-after-redirection.html
Created attachment 358198 [details] Archive of layout-test-results from ews125 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Comment on attachment 358190 [details] Patch Attachment 358190 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/10607487 New failing tests: http/wpt/fetch/dnt-header-after-redirection.html
Created attachment 358199 [details] Archive of layout-test-results from ews107 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews107 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Whoops! It looks like tests to make sure we include DNT in redirects started failing when I removed the DNT in redirects. :-)
Created attachment 358209 [details] Patch
Comment on attachment 358209 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=358209&action=review > Source/WebKit/ChangeLog:18 > +2019-01-02 Brent Fulgham <bfulgham@apple.com> double change log. > Source/WebKit/NetworkProcess/NetworkLoadChecker.cpp:90 > void NetworkLoadChecker::prepareRedirectedRequest(ResourceRequest& request) Can we drop the method that is now empty?
Comment on attachment 358209 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=358209&action=review >> Source/WebKit/ChangeLog:18 >> +2019-01-02 Brent Fulgham <bfulgham@apple.com> > > double change log. Doh! Fixing. >> Source/WebKit/NetworkProcess/NetworkLoadChecker.cpp:90 >> void NetworkLoadChecker::prepareRedirectedRequest(ResourceRequest& request) > > Can we drop the method that is now empty? Sure!
Created attachment 358256 [details] Patch
Comment on attachment 358256 [details] Patch r=me
Committed r239594: <https://trac.webkit.org/changeset/239594>