Fix Alt-Svc and proxy interaction.

Fix Alt-Svc and proxy interaction.

See also https://github.com/httpwg/http-extensions/issues/62.

BUG=392575
Committed: https://crrev.com/2e884788b0536bb37f2bd38712aa4fca98db6be6
Cr-Commit-Position: refs/heads/master@{#411405}

[Bence, 2016-08-09 19:05:13]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-09 19:06:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-09 20:21:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-09 20:21:26]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-08-09 23:30:37]

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

[Bence, 2016-08-09 23:30:38]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-08-10 00:50:22]

lgtm. any possibility that adding a test is easy? I'm guessing not.

[Bence, 2016-08-11 14:32:40]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 14:33:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bence, 2016-08-11 14:37:09]

On 2016/08/10 00:50:22, Ryan Hamilton wrote:
> lgtm. any possibility that adding a test is easy? I'm guessing not.

PTAL.  I added a test that I believe is testing the right thing.  The test
passes.  However, it passes without the change in Job as well.  I am not sure I
understand what the code I am removing was supposed to be doing.  The comment
says "[...] so the proxy resolution works with the actual destination". 
However, url_for_proxy still has origin's hostname, only the scheme and port are
changed.  Also, the comment refers to |alternative_service_url_| which does not
exist.

[Bence, 2016-08-11 14:56:44]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 14:56:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 16:57:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 16:57:36]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2016-08-11 18:37:24]

lgtm thanks for adding the test!

[Bence, 2016-08-11 19:40:40]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-08-11 19:41:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 19:45:42]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-11 19:48:35]

Description was changed from

==========
Fix Alt-Svc and proxy interaction.

See also https://github.com/httpwg/http-extensions/issues/62.

BUG=392575
==========

to

==========
Fix Alt-Svc and proxy interaction.

See also https://github.com/httpwg/http-extensions/issues/62.

BUG=392575

Committed: https://crrev.com/2e884788b0536bb37f2bd38712aa4fca98db6be6
Cr-Commit-Position: refs/heads/master@{#411405}
==========

[commit-bot: I haz the power, 2016-08-11 19:48:36]

Patchset 4 (id:??) landed as
https://crrev.com/2e884788b0536bb37f2bd38712aa4fca98db6be6
Cr-Commit-Position: refs/heads/master@{#411405}