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

net/socket/ssl_client_socket_unittest.cc

Index: net/socket/ssl_client_socket_unittest.cc
diff --git a/net/socket/ssl_client_socket_unittest.cc b/net/socket/ssl_client_socket_unittest.cc
index 4feabafcf707114dc64852130d0fa942d00216da..02dd8f042e8066a6a94d5aa99b8aad4d54062ac9 100644
--- a/net/socket/ssl_client_socket_unittest.cc
+++ b/net/socket/ssl_client_socket_unittest.cc
@@ -402,7 +402,7 @@ TEST_F(SSLClientSocketTest, Read_FullDuplex) {
   // memio circular buffer (4k bytes) in SSLClientSocketNSS to wrap around.
   // This tests the fix for http://crbug.com/29815.
   std::string request_text = "GET / HTTP/1.1\r\nUser-Agent: long browser name ";
-  for (int i = 0; i < 3768; ++i)
+  for (int i = 0; i < 3800; ++i)

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

IMPORTANT: is this an unrelated change?  I don't understand
this change.

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

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.

     request_text.push_back('*');
   request_text.append("\r\n\r\n");
   scoped_refptr<net::IOBuffer> request_buffer(