Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(112)

Issue 582020: Add bounds checking to StaticSocketDataProvider, to make tests more reliable (Closed)

Created:
10 years, 10 months ago by Paweł Hajdan Jr.
Modified:
9 years, 7 months ago
Reviewers:
Mike Belshe, eroman, wtc
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Add bounds checking to StaticSocketDataProvider, to make tests more reliable when they fail. TEST=net_unittests BUG=27567 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=38453

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+288 lines, -167 lines) Patch
M net/ftp/ftp_network_transaction_unittest.cc View 5 chunks +10 lines, -6 lines 0 comments Download
M net/http/http_network_layer_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M net/http/http_network_transaction_unittest.cc View 69 chunks +154 lines, -85 lines 0 comments Download
M net/socket/socket_test_util.h View 1 chunk +17 lines, -12 lines 0 comments Download
M net/socket/socket_test_util.cc View 2 chunks +4 lines, -5 lines 0 comments Download
M net/socket/socks5_client_socket_unittest.cc View 9 chunks +26 lines, -8 lines 0 comments Download
M net/socket/socks_client_socket_unittest.cc View 10 chunks +30 lines, -18 lines 0 comments Download
M net/socket_stream/socket_stream_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M net/spdy/spdy_network_transaction_unittest.cc View 24 chunks +37 lines, -28 lines 0 comments Download
M net/websockets/websocket_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paweł Hajdan Jr.
10 years, 10 months ago (2010-02-08 19:44:51 UTC) #1
Mike Belshe
lgtm
10 years, 10 months ago (2010-02-08 20:31:16 UTC) #2
eroman
10 years, 10 months ago (2010-02-08 23:06:11 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698