Removed bogus threading test to make TSAN happy.
authorsvenpanne <svenpanne@chromium.org>
Thu, 22 Jan 2015 10:23:49 +0000 (02:23 -0800)
committerCommit bot <commit-bot@chromium.org>
Thu, 22 Jan 2015 10:23:58 +0000 (10:23 +0000)
TSAN hits an internal assertion on a self-join, and the test is not
really that useful, so let's just remove it.

BUG=https://code.google.com/p/thread-sanitizer/issues/detail?id=88
LOG=n

Review URL: https://codereview.chromium.org/824243007

Cr-Commit-Position: refs/heads/master@{#26217}

test/unittests/base/platform/platform-unittest.cc

index b17a9b9..8392b55 100644 (file)
@@ -36,24 +36,6 @@ TEST(OS, GetCurrentProcessId) {
 
 namespace {
 
-class SelfJoinThread FINAL : public Thread {
- public:
-  SelfJoinThread() : Thread(Options("SelfJoinThread")) {}
-  void Run() FINAL { Join(); }
-};
-
-}  // namespace
-
-
-TEST(Thread, DISABLE_ON_ANDROID(SelfJoin)) {
-  SelfJoinThread thread;
-  thread.Start();
-  thread.Join();
-}
-
-
-namespace {
-
 class ThreadLocalStorageTest : public Thread, public ::testing::Test {
  public:
   ThreadLocalStorageTest() : Thread(Options("ThreadLocalStorageTest")) {