NetworkSessionConfigurator claims to implement command line argument handling, however, it does not. I want to move handling of (most) command line arguments that affect HttpNetworkSession::Params over to that class, so both the IOThread and NetworkService can easily share it. Note that basically everything it handles are test-only options and field trials, so I don't think there's a problem promoting it to be a content/ dependency.
It does include data_reduction_proxy_params.h, which it doesn't currently look to be using, variations_associated_data.h (Already used by content/), and version_info.h (Not currently used), so I think to use this at the content/ layer, we can just remove those headers and we're done.
I'm not sure about longer term, though - currently the network_service is in content/, but eventually we'll probably want to move it to its own service, above content/, and not sure if we can depend on components/variations there, but think that's an issue for another day.
Comment 1 by bugdroid1@chromium.org
, Jun 12 2017