Bad QUIC proxies are now bypassed.

Bad QUIC proxies are now bypassed.

This change ensures that if a QUIC proxy is unreachable,
then it is marked as bad. Allows chrome to retry the
request with the next proxy instead of showing
ERR_QUIC_PROTOCOL to user.

BUG=459426
Committed: https://crrev.com/53eef7f45df63b239e8ed04fce2f9e8f4cc11bfb
Cr-Commit-Position: refs/heads/master@{#316964}

[tbansal1, 2015-02-18 22:22:28]

tbansal@chromium.org changed reviewers:
+ rch@chromium.org

[Ryan Hamilton, 2015-02-19 00:02:33]

lgtm

Nice catch!

[tbansal1, 2015-02-19 00:10:00]

On 2015/02/19 00:02:33, Ryan Hamilton wrote:
> lgtm
> 
> Nice catch!

Thank you for taking time to review this.

Submitting to bots now.

[tbansal1, 2015-02-19 00:10:15]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2015-02-19 00:11:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/936243002/1

[commit-bot: I haz the power, 2015-02-19 01:47:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-19 01:48:52]

Patchset 1 (id:??) landed as
https://crrev.com/53eef7f45df63b239e8ed04fce2f9e8f4cc11bfb
Cr-Commit-Position: refs/heads/master@{#316964}

[Ryan Hamilton, 2015-02-19 17:02:25]

https://codereview.chromium.org/936243002/diff/1/net/http/http_stream_factory...
File net/http/http_stream_factory_impl_job.cc (right):

https://codereview.chromium.org/936243002/diff/1/net/http/http_stream_factory...
net/http/http_stream_factory_impl_job.cc:907: result == ERR_QUIC_PROTOCOL_ERROR)
{
I think you might also need to add ERR_QUIC_HANDSHAKE_FAILED to this clause.

[tbansal1, 2015-02-19 18:03:28]

On 2015/02/19 17:02:25, Ryan Hamilton wrote:
>
https://codereview.chromium.org/936243002/diff/1/net/http/http_stream_factory...
> File net/http/http_stream_factory_impl_job.cc (right):
> 
>
https://codereview.chromium.org/936243002/diff/1/net/http/http_stream_factory...
> net/http/http_stream_factory_impl_job.cc:907: result ==
ERR_QUIC_PROTOCOL_ERROR)
> {
> I think you might also need to add ERR_QUIC_HANDSHAKE_FAILED to this clause.

Doing in a separate CL. Thanks for noticing.