Add bounds checking to StaticSocketDataProvider, to make tests more reliable

net/websockets/websocket_unittest.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <string>
 #include <vector>
 
 #include "base/task.h"
 #include "net/base/completion_callback.h"
 #include "net/base/io_buffer.h"
@@ skipping 130 matching lines @@
   };
   MockWrite data_writes[] = {
     MockWrite("GET /demo HTTP/1.1\r\n"
               "Upgrade: WebSocket\r\n"
               "Connection: Upgrade\r\n"
               "Host: example.com\r\n"
               "Origin: http://example.com\r\n"
               "WebSocket-Protocol: sample\r\n"
               "\r\n"),
   };
-  StaticSocketDataProvider data(data_reads, data_writes);
+  StaticSocketDataProvider data(data_reads, arraysize(data_reads),
+                                data_writes, arraysize(data_writes));
   mock_socket_factory.AddSocketDataProvider(&data);
 
   WebSocket::Request* request(
       new WebSocket::Request(GURL("ws://example.com/demo"),
                              "sample",
                              "http://example.com",
                              "ws://example.com/demo",
                              new TestURLRequestContext()));
   request->SetHostResolver(new MockHostResolver());
   request->SetClientSocketFactory(&mock_socket_factory);
@@ skipping 39 matching lines @@
   };
   MockWrite data_writes[] = {
     MockWrite("GET /demo HTTP/1.1\r\n"
               "Upgrade: WebSocket\r\n"
               "Connection: Upgrade\r\n"
               "Host: example.com\r\n"
               "Origin: http://example.com\r\n"
               "WebSocket-Protocol: sample\r\n"
               "\r\n"),
   };
-  StaticSocketDataProvider data(data_reads, data_writes);
+  StaticSocketDataProvider data(data_reads, arraysize(data_reads),
+                                data_writes, arraysize(data_writes));
   mock_socket_factory.AddSocketDataProvider(&data);
 
   WebSocket::Request* request(
       new WebSocket::Request(GURL("ws://example.com/demo"),
                              "sample",
                              "http://example.com",
                              "ws://example.com/demo",
                              new TestURLRequestContext()));
   request->SetHostResolver(new MockHostResolver());
   request->SetClientSocketFactory(&mock_socket_factory);
@@ skipping 224 matching lines @@
                              "sample",
                              "http://example.com",
                              "wss://example.com/demo",
                              NULL);
   // :80 should be preserved as it's not the default port for wss://.
   EXPECT_THAT(request.CreateClientHandshakeMessage(),
               testing::HasSubstr("Host: example.com:80\r\n"));
 }
 
 }  // namespace net