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

Issue 8422003: Disable SSLClientSocketTest.ConnectExpired on the "Windows Tests (tsan)" bot. (Closed)

Created:
9 years, 1 month ago by Hironori Bono
Modified:
9 years, 1 month ago
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Disable SSLClientSocketTest.ConnectExpired on the "Windows Tests (tsan)" bot. This change just disables the SSLClientSocketTest.ConnectExpired, which has been failing somehow since Build 102330. TBR=timurrrr,glider,thestig BUG=102330 TEST=make the "Windows Tests (tsan)" bot green. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107923

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M tools/valgrind/gtest_exclude/net_unittests.gtest-tsan_win32.txt View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Hironori Bono
9 years, 1 month ago (2011-10-31 04:39:04 UTC) #1
Timur Iskhodzhanov
9 years, 1 month ago (2011-10-31 07:17:09 UTC) #2
LGTM
31.10.2011 8:39 пользователь <hbono@chromium.org> написал:

> Reviewers: Timur Iskhodzhanov, Alexander Potapenko, Lei Zhang,
>
> Description:
> Disable SSLClientSocketTest.**ConnectExpired on the "Windows Tests
> (tsan)" bot.
> This change just disables the SSLClientSocketTest.**ConnectExpired, which
> has been
> failing somehow since Build 102330.
>
> TBR=timurrrr,glider,thestig
> BUG=102330
> TEST=make the "Windows Tests (tsan)" bot green.
>
> Please review this at
http://codereview.chromium.**org/8422003/<http://codereview.chromium.org/8422...
>
> SVN Base: svn://chrome-svn/chrome/trunk/**src/
>
> Affected files:
>  M     tools/valgrind/gtest_exclude/**net_unittests.gtest-tsan_**win32.txt
>
>
> Index: tools/valgrind/gtest_exclude/**net_unittests.gtest-tsan_**win32.txt
> ==============================**==============================**=======
> --- tools/valgrind/gtest_exclude/**net_unittests.gtest-tsan_**win32.txt
>   (revision 107915)
> +++ tools/valgrind/gtest_exclude/**net_unittests.gtest-tsan_**win32.txt
>   (working copy)
> @@ -32,3 +32,6 @@
>  # Time out, see http://crbug.com/68482
>  SSLServerSocketTest.*
>  X509CertificateTest.**TestKnownRoot
> +
> +# See http://crbug.com/102330
> +SSLClientSocketTest.**ConnectExpired
>
>
>

Powered by Google App Engine
This is Rietveld 408576698