Revert 113419 - Revert 113409 - Add a new method to SSLClientSocket:

Revert 113419 - Revert 113409 - Add a new method to SSLClientSocket:

was_origin_cert_sent()

This will help SpdySession decide to send a CREDENTIAL frame.

BUG=106103
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113528

[Ryan Hamilton, 2011-12-07 22:55:09]

Attempting to reland 113409.

[wtc, 2011-12-07 22:59:58]

LGTM.

http://codereview.chromium.org/8869004/diff/1/net/socket/ssl_client_socket_ns...
File net/socket/ssl_client_socket_nss.cc (right):

http://codereview.chromium.org/8869004/diff/1/net/socket/ssl_client_socket_ns...
net/socket/ssl_client_socket_nss.cc:2279: rv = SECFailure;

Nit: let's take the opportunity to add curly braces to this
if-else statement.

Note: in the early days, code in src/net followed the
convention that only one-line if statements omitted the curly
braces.  If there is an "else", then it's not considered
one line.  This convention is followed in this file.

This seems to be the personal preference of some early
Chrome developers.  It is not documented in the Style Guide.
So I will mention it to new team members, but leave it up
to them.

[Ryan Hamilton, 2011-12-07 23:02:31]

THanks!

http://codereview.chromium.org/8869004/diff/1/net/socket/ssl_client_socket_ns...
File net/socket/ssl_client_socket_nss.cc (right):

http://codereview.chromium.org/8869004/diff/1/net/socket/ssl_client_socket_ns...
net/socket/ssl_client_socket_nss.cc:2279: rv = SECFailure;
On 2011/12/07 22:59:59, wtc wrote:
> 
> Nit: let's take the opportunity to add curly braces to this
> if-else statement.
> 
> Note: in the early days, code in src/net followed the
> convention that only one-line if statements omitted the curly
> braces.  If there is an "else", then it's not considered
> one line.  This convention is followed in this file.
> 
> This seems to be the personal preference of some early
> Chrome developers.  It is not documented in the Style Guide.
> So I will mention it to new team members, but leave it up
> to them.

Done.

[commit-bot: I haz the power, 2011-12-07 23:03:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/8869004/3004

[commit-bot: I haz the power, 2011-12-08 01:51:42]

Change committed as 113528