Remove redundant methods for reloading DataReductionProxyConfig
Currently, there are two ways of updating the configurator in
DataReductionProxyConfig: UpdateConfigurator() and ReloadConfig().
This CL combines them into a single method. Methods that are only
exposed for testing are suffixed with ForTesting() to remove
confusion.
Also, remove some other unreachable code, and a test which is
duplicate of other test.
BUG=476386
Review-Url: https://codereview.chromium.org/2552623004
Cr-Commit-Position: refs/heads/master@{#436650}
9 files changed
