Disable SPDY/3.1.

Disable SPDY/3.1.

* Remove SPDY field trial.
* Add HTTP2 field trial to provide a kill switch for HTTP/2 just in case.
* Remove enable_spdy31 flag.

BUG=624095, 521597
Committed: https://crrev.com/2df4b5245b9c4350001c238dffe46d46fcf0c245
Cr-Commit-Position: refs/heads/master@{#404443}

[Bence, 2016-07-02 15:38:16]

bnc@chromium.org changed reviewers:
+ bengr@chromium.org, mef@chromium.org, rch@chromium.org, zea@chromium.org

[Bence, 2016-07-02 15:38:17]

Nicolas: PTAL at jingle/glue/
mef: PTAL at ios/crnet/
bengr: PTAL at components/data_reduction_proxy/
Ryan: PTAL at everything else.

I'll remove HttpStreamFactory::set_spdy_enabled() and kDisableSpdy in a
follow-up CL.

I'll remove tests with kProtoSPDY31 in another follow-up CL.

Thank you all.

[Nicolas Zea, 2016-07-06 17:14:34]

jingle lgtm

[mef, 2016-07-06 17:15:43]

ios/crnet lgtm

[Ryan Hamilton, 2016-07-06 19:24:58]

lgtm

https://codereview.chromium.org/2115213002/diff/1/net/http/http_network_trans...
File net/http/http_network_transaction_unittest.cc (right):

https://codereview.chromium.org/2115213002/diff/1/net/http/http_network_trans...
net/http/http_network_transaction_unittest.cc:137: "Alt-Svc:
npn-h2=\"mail.example.org:443\"\r\n";
Is "npn-h2" the right header?

[Bence, 2016-07-07 16:20:41]

https://codereview.chromium.org/2115213002/diff/1/net/http/http_network_trans...
File net/http/http_network_transaction_unittest.cc (right):

https://codereview.chromium.org/2115213002/diff/1/net/http/http_network_trans...
net/http/http_network_transaction_unittest.cc:137: "Alt-Svc:
npn-h2=\"mail.example.org:443\"\r\n";
On 2016/07/06 19:24:58, Ryan Hamilton wrote:
> Is "npn-h2" the right header?

I was wondering about the same.  Filed https://crbug.com/626375, and working on
the fix.

[bengr, 2016-07-08 16:25:44]

components/d_r_p/* lgtm

[Bence, 2016-07-08 17:16:03]

The CQ bit was checked by bnc@chromium.org

[Bence, 2016-07-08 17:16:04]

The patchset sent to the CQ was uploaded after l-g-t-m from zea@chromium.org,
mef@chromium.org, rch@chromium.org
Link to the patchset: https://codereview.chromium.org/2115213002/#ps40001
(title: "Rebase on https://crrev.com/2129973002.")

[commit-bot: I haz the power, 2016-07-08 17:16:38]

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-07-08 18:17:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-08 18:20:49]

Description was changed from

==========
Disable SPDY/3.1.

* Remove SPDY field trial.
* Add HTTP2 field trial to provide a kill switch for HTTP/2 just in case.
* Remove enable_spdy31 flag.

BUG=624095, 521597
==========

to

==========
Disable SPDY/3.1.

* Remove SPDY field trial.
* Add HTTP2 field trial to provide a kill switch for HTTP/2 just in case.
* Remove enable_spdy31 flag.

BUG=624095, 521597

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

[commit-bot: I haz the power, 2016-07-08 18:20:50]

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