• John Abd-El-Malek's avatar
    Fix safe search policy not working. · d8b32e5e
    John Abd-El-Malek authored
    This broke for the non-network-service path in r579953 because the new URL from the
    GoogleURLLoaderThrottle wasn't being sent to ResourceDispatcherHost.
    
    While I had added tests to ensure that the correct network request is made, there was a bug in how
    NavigationURLLoaderImpl called the URLLoaderInterceptor in the non-network service path. It was
    giving interceptors the ResourceRequest structure that wasn't used, instead of recreating it again
    from the NavigationRequestInfo that would be used to make the network request.
    
    Bug: 899268
    Change-Id: I248c8b0614aab803a2a916a1e55d91acbed2e28e
    Reviewed-on: https://chromium-review.googlesource.com/c/1317865Reviewed-by: default avatarNasko Oskov <nasko@chromium.org>
    Commit-Queue: John Abd-El-Malek <jam@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#605518}
    d8b32e5e
navigation_url_loader_impl.cc 79.2 KB