Fix ToolsSanityTest.DataRace under TSan v1.

base/tools_sanity_unittest.cc

 // Copyright (c) 2012 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.
 //
 // This file contains intentional memory errors, some of which may lead to
 // crashes if the test is ran without special memory testing tools. We use these
 // errors to verify the sanity of the tools.
 
 #include "base/atomicops.h"
 #include "base/message_loop.h"
@@ skipping 229 matching lines @@
   PlatformThread::Create(0, d1, &a);
   PlatformThread::Create(0, d2, &b);
   PlatformThread::Join(a);
   PlatformThread::Join(b);
 }
 
 }  // namespace
 
 // A data race detector should report an error in this test.
 TEST(ToolsSanityTest, DataRace) {
-  bool shared = false;
-  TOOLS_SANITY_TEST_CONCURRENT_THREAD thread1(&shared), thread2(&shared);
+  bool *shared = new bool(false);

[Lei Zhang, 2013/03/23 15:33:14]

small nit: bool*

+  TOOLS_SANITY_TEST_CONCURRENT_THREAD thread1(shared), thread2(shared);
   RunInParallel(&thread1, &thread2);
-  EXPECT_TRUE(shared);
+  EXPECT_TRUE(*shared);
+  delete shared;
 }
 
 TEST(ToolsSanityTest, AnnotateBenignRace) {
   bool shared = false;
   ANNOTATE_BENIGN_RACE(&shared, "Intentional race - make sure doesn't show up");
   TOOLS_SANITY_TEST_CONCURRENT_THREAD thread1(&shared), thread2(&shared);
   RunInParallel(&thread1, &thread2);
   EXPECT_TRUE(shared);
 }
 
 TEST(ToolsSanityTest, AtomicsAreIgnored) {
   base::subtle::Atomic32 shared = 0;
   ReleaseStoreThread thread1(&shared);
   AcquireLoadThread thread2(&shared);
   RunInParallel(&thread1, &thread2);
   EXPECT_EQ(kMagicValue, shared);
 }
 
 }  // namespace base