net: remove test flakyness caused by uncorking.

net: remove test flakyness caused by uncorking.

r61546 (reverted r61546, landed r61638, reverted r61672; it was
blameless for the first revert.) caused flakyness in the
SSLClientSocket tests because the SSL_CONNECT event may be hidden by a
SOCKET_SENT event in the case that the test takes over 200ms to run.

This patch changes the test to match against either event.

BUG=none
TEST=net_unittests

[wtc, 2010-10-07 21:25:25]

LGTM.

http://codereview.chromium.org/3531019/diff/1/2
File net/socket/ssl_client_socket_unittest.cc (right):

http://codereview.chromium.org/3531019/diff/1/2#newcode46
net/socket/ssl_client_socket_unittest.cc:46: const
net::CapturingNetLog::EntryList& log, int i) {
BUG: the parameter 'i' is not used.  It seems that you
should pass 'i' instead of -1 to net::LogContainsEndEvent.

[wtc, 2010-10-07 21:29:54]

http://codereview.chromium.org/3531019/diff/1/2
File net/socket/ssl_client_socket_unittest.cc (right):

http://codereview.chromium.org/3531019/diff/1/2#newcode43
net/socket/ssl_client_socket_unittest.cc:43: // timeout. This means that an SSL
connect end event may appear as a socket
This sentence is confusing.  Can you explain exactly what
happens?  I think when the test fails, we have an
SSL connect end event, followed by a SOCKET_BYTES_SENT
event, right?  So the SSL connect end event is not missing;
it's just shadowed by the SOCKET_BYTES_SENT event caused by
the uncorking timer.