Remove NPN from SSLConfig and SSLClientSocket*.

Remove NPN from SSLConfig and SSLClientSocket*.

Remove:
* SSLConfig::npn_protos,
* SSLClientSocket::NextProtoStatusToString(),
* enum SSLClientSocket::NextProtoStatus,
* enum SSLClientSocket::SSLNegotiationExtension,
* kDefaultSupportedNPNProtocol,
* SSLClientSocketImpl::npn_status_,
* SSLClientSocketImpl::negotiation_extension_,
* SSLClientSocketImpl::SelectNextProtoCallback,
* SSLClientSocketImpl::SSLContext::SelectNextProtoCallback,
* HttpNetworkSession::GetNpnProtos(),
* NPN unittests.

BUG=526713
Committed: https://crrev.com/ce6ea242a36b664f1f56d8174cebef6f341c817e
Cr-Commit-Position: refs/heads/master@{#418952}

[Bence, 2016-09-15 14:55:04]

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

[commit-bot: I haz the power, 2016-09-15 14:55:16]

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-09-15 16:05:40]

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

[commit-bot: I haz the power, 2016-09-15 16:05:41]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-09-15 18:16:20]

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

[Bence, 2016-09-15 18:16:21]

David: PTAL.  Thank you.

[davidben, 2016-09-15 18:57:00]

lgtm!

https://codereview.chromium.org/2348453002/diff/1/net/socket/ssl_client_socke...
File net/socket/ssl_client_socket_impl.cc (right):

https://codereview.chromium.org/2348453002/diff/1/net/socket/ssl_client_socke...
net/socket/ssl_client_socket_impl.cc:1047: SSL_set_options(ssl_,
SSL_OP_DISABLE_NPN);
This line can be removed. (If the callback isn't configured, NPN isn't enabled.)

[Bence, 2016-09-15 19:01:22]

The CQ bit was checked by bnc@chromium.org

[Bence, 2016-09-15 19:01:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
davidben@chromium.org
Link to the patchset: https://codereview.chromium.org/2348453002/#ps20001
(title: "Re: #7.")

[Bence, 2016-09-15 19:01:31]

Thank you.

https://codereview.chromium.org/2348453002/diff/1/net/socket/ssl_client_socke...
File net/socket/ssl_client_socket_impl.cc (right):

https://codereview.chromium.org/2348453002/diff/1/net/socket/ssl_client_socke...
net/socket/ssl_client_socket_impl.cc:1047: SSL_set_options(ssl_,
SSL_OP_DISABLE_NPN);
On 2016/09/15 18:57:00, davidben wrote:
> This line can be removed. (If the callback isn't configured, NPN isn't
enabled.)

Done.

[commit-bot: I haz the power, 2016-09-15 19:02:22]

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-09-15 20:22:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-15 20:25:13]

Description was changed from

==========
Remove NPN from SSLConfig and SSLClientSocket*.

Remove:
* SSLConfig::npn_protos,
* SSLClientSocket::NextProtoStatusToString(),
* enum SSLClientSocket::NextProtoStatus,
* enum SSLClientSocket::SSLNegotiationExtension,
* kDefaultSupportedNPNProtocol,
* SSLClientSocketImpl::npn_status_,
* SSLClientSocketImpl::negotiation_extension_,
* SSLClientSocketImpl::SelectNextProtoCallback,
* SSLClientSocketImpl::SSLContext::SelectNextProtoCallback,
* HttpNetworkSession::GetNpnProtos(),
* NPN unittests.

BUG=526713
==========

to

==========
Remove NPN from SSLConfig and SSLClientSocket*.

Remove:
* SSLConfig::npn_protos,
* SSLClientSocket::NextProtoStatusToString(),
* enum SSLClientSocket::NextProtoStatus,
* enum SSLClientSocket::SSLNegotiationExtension,
* kDefaultSupportedNPNProtocol,
* SSLClientSocketImpl::npn_status_,
* SSLClientSocketImpl::negotiation_extension_,
* SSLClientSocketImpl::SelectNextProtoCallback,
* SSLClientSocketImpl::SSLContext::SelectNextProtoCallback,
* HttpNetworkSession::GetNpnProtos(),
* NPN unittests.

BUG=526713

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

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

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