Revert "net: Precede each CBC encrypted application data record with an empty one."

Revert "net: Precede each CBC encrypted application data record with an empty one."

This reverts commit r91768.

BUG=91905
TEST=none


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

[wtc, 2011-08-08 17:06:05]

LGTM.  I have a question about the change to
ssl_client_socket_unittest.cc below.

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

http://codereview.chromium.org/7583036/diff/1/net/socket/ssl_client_socket_un...
net/socket/ssl_client_socket_unittest.cc:405: for (int i = 0; i < 3800; ++i)
IMPORTANT: is this an unrelated change?  I don't understand
this change.

[agl, 2011-08-08 17:29:02]

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

http://codereview.chromium.org/7583036/diff/1/net/socket/ssl_client_socket_un...
net/socket/ssl_client_socket_unittest.cc:405: for (int i = 0; i < 3800; ++i)
On 2011/08/08 17:06:05, wtc wrote:
> IMPORTANT: is this an unrelated change?  I don't understand
> this change.

This is a correct revert:
http://src.chromium.org/viewvc/chrome?view=rev&revision=91768

The issue here is that the test is trying to exactly cause a memio wrap around
and the additional empty record alters the amount of data written to the socket
and causes an IO_PENDING that the test doesn't expect.

[commit-bot: I haz the power, 2011-08-09 01:55:05]

Change committed as 95932