From 1442b29a275b076203b5e0a78206fe3feda3b739 Mon Sep 17 00:00:00 2001 From: "ukai@chromium.org" Date: Tue, 20 Jul 2010 11:14:54 +0000 Subject: Fix tsan failure in SpdyStreamTest. Under tsan, tcp connect job tries to connect several times and fails to find data provider for mock socket. Make connect synchronous (as done in SpdySessionTest) fixes the issue. BUG=none TEST=./tools/valgrind/chrome_tests.sh --build_dir out/Debug --test net --tool tsan --gtest_filter='SpdyStreamTest.*' Review URL: http://codereview.chromium.org/3054002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53036 0039d316-1c4b-4281-b951-d872f2087c98 --- net/socket/socket_test_util.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'net/socket') diff --git a/net/socket/socket_test_util.h b/net/socket/socket_test_util.h index bb52c0c..2bd0d23 100644 --- a/net/socket/socket_test_util.h +++ b/net/socket/socket_test_util.h @@ -350,7 +350,7 @@ class SocketDataProviderArray { } T* GetNext() { - DCHECK(next_index_ < data_providers_.size()); + DCHECK_LT(next_index_, data_providers_.size()); return data_providers_[next_index_++]; } -- cgit v1.1