Remove ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code.

Remove ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code.

Remove ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code, because (1) right
now, remote TCP alternative services (where the origin and the alternative
service have different hostnames) are disabled, and (2) in the future, if an
existing connection to the destination of the alternative service with an origin
different from the origin of the request does not have a valid certificate for
the origin of the request, then a new connection to the destination of the
alternative service will be opened with the origin of the request, and if that
connection does not have a valid certificate for the origin of the request, then
that's simply ERR_CERT_COMMON_NAME_INVALID.  So there is no need for
ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN.

Also remove ValidSpdySessionPool class, because (1) right now there is no need
for the extra cert verification, and (2) in the future,
SpdySessionPool::FindAvailableSession() and
SpdySessionPool::CreateAvailableSessionFromSocket() will take both an origin and
a destination argument, and will make sure that the SpdySession they return has
a certificate that is valid for the origin.

BUG=615413
Committed: https://crrev.com/9989bd51828a155d65cd9131cee6700d2bb3164e
Cr-Commit-Position: refs/heads/master@{#405342}

[Bence, 2016-07-13 14:12:15]

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

[commit-bot: I haz the power, 2016-07-13 14:12:42]

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-07-13 15:36:43]

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

[commit-bot: I haz the power, 2016-07-13 15:36:43]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-07-13 15:37:28]

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

[Bence, 2016-07-13 15:37:29]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-07-13 23:02:30]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-07-13 23:02:30]

lgtm

Nice!

[commit-bot: I haz the power, 2016-07-13 23:03:11]

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-13 23:09:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 23:10:00]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 23:12:07]

Description was changed from

==========
Remove ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code.

Remove ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code, because (1) right
now, remote TCP alternative services (where the origin and the alternative
service have different hostnames) are disabled, and (2) in the future, if an
existing connection to the destination of the alternative service with an origin
different from the origin of the request does not have a valid certificate for
the origin of the request, then a new connection to the destination of the
alternative service will be opened with the origin of the request, and if that
connection does not have a valid certificate for the origin of the request, then
that's simply ERR_CERT_COMMON_NAME_INVALID.  So there is no need for
ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN.

Also remove ValidSpdySessionPool class, because (1) right now there is no need
for the extra cert verification, and (2) in the future,
SpdySessionPool::FindAvailableSession() and
SpdySessionPool::CreateAvailableSessionFromSocket() will take both an origin and
a destination argument, and will make sure that the SpdySession they return has
a certificate that is valid for the origin.

BUG=615413
==========

to

==========
Remove ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code.

Remove ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN error code, because (1) right
now, remote TCP alternative services (where the origin and the alternative
service have different hostnames) are disabled, and (2) in the future, if an
existing connection to the destination of the alternative service with an origin
different from the origin of the request does not have a valid certificate for
the origin of the request, then a new connection to the destination of the
alternative service will be opened with the origin of the request, and if that
connection does not have a valid certificate for the origin of the request, then
that's simply ERR_CERT_COMMON_NAME_INVALID.  So there is no need for
ERR_ALTERNATIVE_CERT_NOT_VALID_FOR_ORIGIN.

Also remove ValidSpdySessionPool class, because (1) right now there is no need
for the extra cert verification, and (2) in the future,
SpdySessionPool::FindAvailableSession() and
SpdySessionPool::CreateAvailableSessionFromSocket() will take both an origin and
a destination argument, and will make sure that the SpdySession they return has
a certificate that is valid for the origin.

BUG=615413

Committed: https://crrev.com/9989bd51828a155d65cd9131cee6700d2bb3164e
Cr-Commit-Position: refs/heads/master@{#405342}
==========

[commit-bot: I haz the power, 2016-07-13 23:12:08]

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