Enable MSVC warning for unused locals.

net/socket/socks5_client_socket_unittest.cc

Index: net/socket/socks5_client_socket_unittest.cc
diff --git a/net/socket/socks5_client_socket_unittest.cc b/net/socket/socks5_client_socket_unittest.cc
index c474a0b419843245e4887faf476eaf834565e96a..5bcc146fc1b24704df45e7f1151a8bc43c5f8a03 100644
--- a/net/socket/socks5_client_socket_unittest.cc
+++ b/net/socket/socks5_client_socket_unittest.cc
@@ -258,7 +258,7 @@ TEST_F(SOCKS5ClientSocketTest, PartialReadWrites) {
     const char partial1[] = { 0x05, 0x01 };
     const char partial2[] = { 0x00 };
     MockWrite data_writes[] = {
-        MockWrite(ASYNC, arraysize(partial1)),
+        MockWrite(ASYNC, partial1, arraysize(partial1)),

[Peter Kasting, 2014/11/18 00:41:03]

I assume we meant to actually write |partial1|?

[rvargas (doing something else), 2014/11/18 02:11:48]

wow. I'm surprised it worked at all.

[Peter Kasting, 2014/11/18 02:15:18]

Could be a sign that the tests are not as exhaustive as they should be?  :)

         MockWrite(ASYNC, partial2, arraysize(partial2)),
         MockWrite(ASYNC, kOkRequest, arraysize(kOkRequest)) };
     MockRead data_reads[] = {