|
[WIP] Upgrade-insecure-request: upgrade insecurely-redirected requests.
* Adds two Layout tests that check that redirected requests are upgraded. It
covers the bug: crbug.com/659684.
* Fixes the bug for both resource requests and navigational requests.
* Upgrade requests on the PlzNavigate path (browser-side-navigation) too.
For information, this CL **partially** fixes the tests:
* http/tests/navigation/post-frame-goback1.html
* virtual/stable/http/tests/navigation/post-frames-goback1.html
with --enable-browser-side-navigation flag.
BUG= 659684, 576261
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+294 lines, -33 lines) |
Patch |
 |
M |
content/browser/BUILD.gn
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/frame_tree_node.h
|
View
|
|
1 chunk |
+3 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/frame_tree_node.cc
|
View
|
|
1 chunk |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/navigation_request.h
|
View
|
|
3 chunks |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/navigation_request.cc
|
View
|
|
5 chunks |
+32 lines, -2 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/navigation_request_info.h
|
View
|
|
3 chunks |
+15 lines, -10 lines |
0 comments
|
Download
|
 |
M |
content/browser/frame_host/navigation_request_info.cc
|
View
|
|
2 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
content/browser/loader/async_resource_handler_unittest.cc
|
View
|
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/browser/loader/navigation_url_loader_unittest.cc
|
View
|
|
1 chunk |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
content/browser/loader/resource_dispatcher_host_impl.cc
|
View
|
|
3 chunks |
+7 lines, -3 lines |
0 comments
|
Download
|
 |
M |
content/browser/loader/resource_dispatcher_host_unittest.cc
|
View
|
|
2 chunks |
+5 lines, -4 lines |
0 comments
|
Download
|
 |
M |
content/browser/loader/resource_request_info_impl.h
|
View
|
|
3 chunks |
+8 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/browser/loader/resource_request_info_impl.cc
|
View
|
|
3 chunks |
+6 lines, -3 lines |
0 comments
|
Download
|
 |
A |
content/browser/net/url_request_insecure_interceptor.h
|
View
|
|
1 chunk |
+37 lines, -0 lines |
0 comments
|
Download
|
 |
A |
content/browser/net/url_request_insecure_interceptor.cc
|
View
|
1
|
1 chunk |
+50 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/storage_partition_impl_map.cc
|
View
|
|
2 chunks |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/child/web_url_loader_impl.cc
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
content/common/resource_messages.h
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
content/common/resource_request.h
|
View
|
|
2 chunks |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
A |
third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/basic-upgrade-redirect.html
|
View
|
|
1 chunk |
+35 lines, -0 lines |
0 comments
|
Download
|
 |
A |
third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-redirect.https.html
|
View
|
|
1 chunk |
+39 lines, -0 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/core/loader/FrameLoader.cpp
|
View
|
|
3 chunks |
+7 lines, -4 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/platform/exported/WebURLRequest.cpp
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/platform/network/ResourceRequest.h
|
View
|
|
2 chunks |
+10 lines, -0 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/public/platform/WebURLRequest.h
|
View
|
|
2 chunks |
+6 lines, -0 lines |
0 comments
|
Download
|
Total messages: 15 (15 generated)
|