• Matt Menke's avatar
    Don't apply ForceHTTP11 for HTTP proxies. · bac502d7
    Matt Menke authored
    We should be establishing HTTP connections to HTTP/1.1 proxies, so it's
    weird to apply REQUIRES_HTTP_11 logic to them.
    
    I'm planning on moving the force HTTP/1.1 bool into
    HttpServerProperties::ServerInfo, to simplify the class. ServerInfo is,
    however, keyed on HttpServerProperties rather than HostPortPair, and
    using either an HTTPS or HTTP scheme in the HTTP proxy case when
    calling MaybeForceHTTP11 seems weird.
    
    Bug: 993801
    Change-Id: I4237a28c6a16ea03fe9ad4f60657bd4c32a45b2f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761635Reviewed-by: default avatarBence Béky <bnc@chromium.org>
    Commit-Queue: Matt Menke <mmenke@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#688658}
    bac502d7
http_stream_factory_job.cc 48.9 KB