This is a reland of https://codereview.chromium.org/2973433003/ It was reverted for compilation error.
Original CL description: > Redirects toward renderer-debug urls (chrome://crash, javascript:[...], > ...) were not expected to happen in the NavigationHandleImpl. It caused > WebContentsObserver::DidFinishNavigation not being called and > consequently, bugs in some WebContentsObservers. > > This CL blocks any redirects to renderer-debug urls. One test is added. > > The blocking of the URL was moved (and partially removed) in: > https://crrev.com/2436253002 > This CL tries to restore the previous behavior.
BUG=736658,704892
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation;master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel
Change-Id: I1e82511b88acfabcb29e4fad102a5efa32ddca9a Reviewed-on: https://chromium-review.googlesource.com/569878
98e5a23 Reland of: Block redirects to renderer-debug urls.
.../captive_portal/captive_portal_tab_helper.cc | 9 +--
.../browser/child_process_security_policy_impl.cc | 24 +++++++
.../browser/child_process_security_policy_impl.h | 7 ++
.../child_process_security_policy_unittest.cc | 81 +++++++++++++++++++++-
.../browser/frame_host/navigation_handle_impl.cc | 19 ++---
.../browser/frame_host/navigation_handle_impl.h | 4 --
.../navigation_handle_impl_browsertest.cc | 56 +++++++++++++++
content/browser/frame_host/navigation_request.cc | 41 +++++++----
8 files changed, 202 insertions(+), 39 deletions(-)
Upstream: git.chromium.org