Comments

Remove HttpNetworkSession argument from CreateSocketParamsAndGetGroupId.
Instead, this CL adds pointers to several global objects are to
CommonConnectJobParams, and moves the logic for testing_fixed_http_port
over to the calling function - it affects the endpoint used, which
affects part of the group_id, and hence is fine to keep around.
In order to avoid pooling sockets created with different SocketParams,
we're working on making it so there's a 1-to-1 mapping between GroupIds
and SocketParams. To that end, we're removing objects used to create
SocketParams.
Bug: 921369
Change-Id: Ib5b234ab8f4cc0c45c97d5a4c4e46e6e885536b4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1531083
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#642470}