Add SSLClientSocket::SerializeNextProtos unittest.

Add SSLClientSocket::SerializeNextProtos unittest.

BUG=

Committed: https://crrev.com/90c006969d775809dc32415f3b47f41b0dcbfe65
Cr-Commit-Position: refs/heads/master@{#308098}

[Bence, 2014-12-12 13:53:04]

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

[Bence, 2014-12-12 13:53:05]

Ryan: PTAL.  Thanks.

[Ryan Hamilton, 2014-12-12 15:12:12]

lgtm

https://codereview.chromium.org/801803002/diff/1/net/socket/ssl_client_socket...
File net/socket/ssl_client_socket_unittest.cc (right):

https://codereview.chromium.org/801803002/diff/1/net/socket/ssl_client_socket...
net/socket/ssl_client_socket_unittest.cc:2378: EXPECT_EQ('1', serialized[17]);
Could you do something like EXPECT_EQ("\x08http/1.1\x08spdy/3.1", serialized) ?

Not sure fi that would be better, but it might be easier to read? Either way.

[Bence, 2014-12-12 15:35:49]

Patchset #2 (id:20001) has been deleted

[Bence, 2014-12-12 15:40:07]

The CQ bit was checked by bnc@chromium.org

[Bence, 2014-12-12 15:40:16]

Thanks.

https://codereview.chromium.org/801803002/diff/1/net/socket/ssl_client_socket...
File net/socket/ssl_client_socket_unittest.cc (right):

https://codereview.chromium.org/801803002/diff/1/net/socket/ssl_client_socket...
net/socket/ssl_client_socket_unittest.cc:2378: EXPECT_EQ('1', serialized[17]);
On 2014/12/12 15:12:12, Ryan Hamilton wrote:
> Could you do something like EXPECT_EQ("\x08http/1.1\x08spdy/3.1", serialized)
?
> 
> Not sure fi that would be better, but it might be easier to read? Either way.

I couldn't quite do this, since one is a vector, the other is a string.  I would
love to do std::vector<uint8_t> expected {8, 'h', 't', 't', 'p', ...}, but
initializer lists are not yet allowed.  I guess we should leave it as that for
now, sorry.

[commit-bot: I haz the power, 2014-12-12 15:41:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/801803002/1

[commit-bot: I haz the power, 2014-12-12 16:26:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-12 16:27:14]

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