Move EXPECT_DCHECK_DEATH from base/task_scheduler and use it in relevant base/ tests

base/synchronization/cancellation_flag_unittest.cc

 // Copyright (c) 2011 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.
 
 // Tests of CancellationFlag class.
 
 #include "base/synchronization/cancellation_flag.h"
 
 #include "base/bind.h"
 #include "base/location.h"
-#include "base/logging.h"
 #include "base/single_thread_task_runner.h"
 #include "base/synchronization/spin_wait.h"
+#include "base/test/gtest_util.h"
 #include "base/threading/thread.h"
 #include "base/time/time.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
 namespace base {
 
-namespace {
-
-//------------------------------------------------------------------------------
-// Define our test class.
-//------------------------------------------------------------------------------
-
-void CancelHelper(CancellationFlag* flag) {
-#if GTEST_HAS_DEATH_TEST
-  ASSERT_DEBUG_DEATH(flag->Set(), "");
-#endif
-}
-
 TEST(CancellationFlagTest, SimpleSingleThreadedTest) {
   CancellationFlag flag;
   ASSERT_FALSE(flag.IsSet());
   flag.Set();
   ASSERT_TRUE(flag.IsSet());
 }
 
 TEST(CancellationFlagTest, DoubleSetTest) {
   CancellationFlag flag;
   ASSERT_FALSE(flag.IsSet());
   flag.Set();
   ASSERT_TRUE(flag.IsSet());
   flag.Set();
   ASSERT_TRUE(flag.IsSet());
 }
 
+namespace {
+
+void CancelHelper(CancellationFlag* flag) {
+  EXPECT_DCHECK_DEATH(flag->Set(), "");
+}
+
+}  // namespace
+
 TEST(CancellationFlagTest, SetOnDifferentThreadDeathTest) {
   // Checks that Set() can't be called from any other thread.
   // CancellationFlag should die on a DCHECK if Set() is called from
   // other thread.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   Thread t("CancellationFlagTest.SetOnDifferentThreadDeathTest");
   ASSERT_TRUE(t.Start());
   ASSERT_TRUE(t.message_loop());
   ASSERT_TRUE(t.IsRunning());
 
   CancellationFlag flag;
   t.task_runner()->PostTask(FROM_HERE, base::Bind(&CancelHelper, &flag));
 }
 
-}  // namespace

[danakj, 2016/07/20 19:33:45]

I think it is usual to wrap a unit test file an a nested anonymous namespace,
and I'd leave it here. Other random example:
https://cs.chromium.org/chromium/src/cc/layers/layer_unittest.cc?rcl=0&l=2542

[gab, 2016/07/20 21:21:08]

Done.

-
 }  // namespace base