• David Van Cleve's avatar
    Removes the --no-referrers command line switch · 361ea625
    David Van Cleve authored
    This is one of a series of three CLs adding more uniformity
    to referrer controls' implementation. The switch's functionality
    remains accessible via disabling the enable_referrers pref.
    
    The changes in this CL comprise:
    1. Removing the switch
    2. Removing the previous calls directly checking the switch, rather than
    the enable_referrers pref. Happily, these checks of the switch were both
    redundant with the later check of enable_referrers performed in
    NetworkDelegate::NotifyBeforeURLRequest.
    3. A nominal amount of refactoring of some dead/nearly-dead referrer-related
    code:
      - network::ComputeReferrer removed from loader_util.cc (after removing
    the --no-referrers switch, this method now duplicates GURL::GetAsReferrer).
      - content::Referrer::SetReferrerForRequest removed (it had no callers).
    
    R=gab, falken
    
    Bug: 1004486
    Change-Id: I00bc503d0954700565047a188bbb9842c4d4bc9e
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808144
    Commit-Queue: David Van Cleve <davidvc@chromium.org>
    Reviewed-by: default avatarGabriel Charette <gab@chromium.org>
    Reviewed-by: default avatarMatt Falkenhagen <falken@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#699403}
    361ea625
utility_process_host.cc 18.6 KB