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

Unified Diff: media/audio/cross_process_notification_unittest.cc

Issue 9874021: Disable TwoNotifiersTwoThreads1000Signals as it trips off the TSAN bot. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/cross_process_notification_unittest.cc
diff --git a/media/audio/cross_process_notification_unittest.cc b/media/audio/cross_process_notification_unittest.cc
index c045416cd6cf405b74e319c26fd5310590d69b98..814b527d09662ba6584be4dbcad32612b1bff727 100644
--- a/media/audio/cross_process_notification_unittest.cc
+++ b/media/audio/cross_process_notification_unittest.cc
@@ -282,7 +282,9 @@ TEST(CrossProcessNotification, DISABLED_ThousandNotifiersTwoThreads) {
// Functionally equivalent (as far as the shared data goes) to the
// ThousandNotifiersTwoThreads test but uses a single pair of notifiers +
// FlagArray for the 1000 signals. This approach is significantly faster.
-TEST(CrossProcessNotification, TwoNotifiersTwoThreads1000Signals) {
+// Disabled as it trips of the TSAN bot - "Possible data race during write of
+// size 4" (the flag array).
+TEST(CrossProcessNotification, DISABLED_TwoNotifiersTwoThreads1000Signals) {
CrossProcessNotification a, b;
ASSERT_TRUE(CrossProcessNotification::InitializePair(&a, &b));
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698