Remove SpdyNetworkTransactionTestSSLType.

Remove SpdyNetworkTransactionTestSSLType.

HTTP_SPDY_VIA_NPN and HTTPS_SPDY_VIA_ALT_SVC type tests use the same origin,
same port, same scheme.  The only different is the preloaded alternative service
record and the extra hanging socket.  There is really no added value, it is just
a burden on code complexity and test infrastructure.

Committed: https://crrev.com/f23de8c58bd85a39611c60960fde9a6bf6dd9473
Cr-Commit-Position: refs/heads/master@{#403226}

[Bence, 2016-06-30 17:53:32]

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

[Bence, 2016-06-30 17:53:33]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-06-30 18:09:51]

lgtm

Do we have a "HTTP/2 via remote Alt-Svc" test somewhere? (I think we do).

[Bence, 2016-06-30 18:12:55]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-06-30 18:13:22]

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

[Bence, 2016-06-30 18:18:05]

On 2016/06/30 18:09:51, Ryan Hamilton wrote:
> lgtm
> 
> Do we have a "HTTP/2 via remote Alt-Svc" test somewhere? (I think we do).

Thank you.

I believe HttpNetworkTransactionTest.UseAlternateProtocolForNpnSpdy sufficiently
covers this case.

[commit-bot: I haz the power, 2016-06-30 18:21:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 18:21:37]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 18:25:46]

Description was changed from

==========
Remove SpdyNetworkTransactionTestSSLType.

HTTP_SPDY_VIA_NPN and HTTPS_SPDY_VIA_ALT_SVC type tests use the same origin,
same port, same scheme.  The only different is the preloaded alternative service
record and the extra hanging socket.  There is really no added value, it is just
a burden on code complexity and test infrastructure.
==========

to

==========
Remove SpdyNetworkTransactionTestSSLType.

HTTP_SPDY_VIA_NPN and HTTPS_SPDY_VIA_ALT_SVC type tests use the same origin,
same port, same scheme.  The only different is the preloaded alternative service
record and the extra hanging socket.  There is really no added value, it is just
a burden on code complexity and test infrastructure.

Committed: https://crrev.com/f23de8c58bd85a39611c60960fde9a6bf6dd9473
Cr-Commit-Position: refs/heads/master@{#403226}
==========

[commit-bot: I haz the power, 2016-06-30 18:25:47]

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

[Bence, 2016-06-30 22:35:38]

FYI This CL reduces the number of net_unittests from 22496 to 22312, a modest
reduction of 0.8%.