• ananta's avatar
    PlzNavigate: Fix the FindInPageControllerTest.SearchWithinSpecialURL browser test. · 7bb43d67
    ananta authored
    This test was initiating a navigation to the chrome\test\data folder and performing a
    number of find requests on the page returned. With PlzNavigate the request to navigate
    to chrome\test\data folder would get blocked in ResourceLoader::OnReceivedRedirect because
    the process id (-1) did not have access to the url being navigated to. In the non PlzNavigate
    case this is not an issue because the child process is given access to the URL during
    navigation.
    
    Proposed fix for PlzNavigate is to not do the ChildProcessSecurityPolicy check in
    the ResourceLoader::OnReceivedRedirect function and instead do this check on the UI thread
    in the NavigationRequest::OnRedirectChecksComplete function. Additionally we also grant
    access to the URL in the NavigatorImpl::RequestNavigation function.
    
    The above test is now enabled for Windows.
    
    BUG=175711
    
    Disabled due to crbug.com/175711
    CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
    
    Review-Url: https://codereview.chromium.org/2436253002
    Cr-Commit-Position: refs/heads/master@{#427279}
    7bb43d67
DEPS 15.1 KB