Unify usage of logging/assert macros in base/

base/sequence_checker_unittest.cc

 // Copyright 2013 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.
 
 #include "base/sequence_checker.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/single_thread_task_runner.h"
 #include "base/test/gtest_util.h"
 #include "base/test/sequenced_worker_pool_owner.h"
 #include "base/threading/thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-// Duplicated from base/sequence_checker.h so that we can be good citizens
-// there and undef the macro.
-#if (!defined(NDEBUG) || defined(DCHECK_ALWAYS_ON))
-#define ENABLE_SEQUENCE_CHECKER 1
-#else
-#define ENABLE_SEQUENCE_CHECKER 0
-#endif
-
 namespace base {
 
 namespace {
 
 const size_t kNumWorkerThreads = 3;
 
 // Simple class to exercise the basics of SequenceChecker.
 // DoStuff should verify that it's called on a valid sequenced thread.
 // SequenceCheckedObject can be destroyed on any thread (like WeakPtr).
 class SequenceCheckedObject {
@@ skipping 144 matching lines @@
 void SequenceCheckerTest::MethodOnDifferentThreadDeathTest() {
   std::unique_ptr<SequenceCheckedObject> sequence_checked_object(
       new SequenceCheckedObject);
 
   // DoStuff should assert in debug builds only when called on a
   // different thread.
   PostDoStuffToOtherThread(sequence_checked_object.get());
   other_thread()->Stop();
 }
 
-#if ENABLE_SEQUENCE_CHECKER
+#if DCHECK_IS_ON()
 TEST_F(SequenceCheckerTest, MethodNotAllowedOnDifferentThreadDeathTestInDebug) {
   // The default style "fast" does not support multi-threaded tests.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   ASSERT_DCHECK_DEATH({ MethodOnDifferentThreadDeathTest(); }, "");
 }
 #else
 TEST_F(SequenceCheckerTest, MethodAllowedOnDifferentThreadDeathTestInRelease) {
   MethodOnDifferentThreadDeathTest();
 }
-#endif  // ENABLE_SEQUENCE_CHECKER
+#endif  // DCHECK_IS_ON()
 
 void SequenceCheckerTest::DetachThenCallFromDifferentThreadDeathTest() {
   std::unique_ptr<SequenceCheckedObject> sequence_checked_object(
       new SequenceCheckedObject);
 
   // DoStuff doesn't assert when called on a different thread
   // after a call to DetachFromSequence.
   sequence_checked_object->DetachFromSequence();
   PostDoStuffToOtherThread(sequence_checked_object.get());
   other_thread()->Stop();
 
   // DoStuff should assert in debug builds only after moving to
   // another thread.
   sequence_checked_object->DoStuff();
 }
 
-#if ENABLE_SEQUENCE_CHECKER
+#if DCHECK_IS_ON()
 TEST_F(SequenceCheckerTest, DetachFromSequenceDeathTestInDebug) {
   // The default style "fast" does not support multi-threaded tests.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   ASSERT_DCHECK_DEATH({ DetachThenCallFromDifferentThreadDeathTest(); }, "");
 }
 #else
 TEST_F(SequenceCheckerTest, DetachFromThreadDeathTestInRelease) {
   DetachThenCallFromDifferentThreadDeathTest();
 }
-#endif  // ENABLE_SEQUENCE_CHECKER
+#endif  // DCHECK_IS_ON()
 
 void SequenceCheckerTest::DifferentSequenceTokensDeathTest() {
   std::unique_ptr<SequenceCheckedObject> sequence_checked_object(
       new SequenceCheckedObject);
 
   sequence_checked_object->DetachFromSequence();
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "B");
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "B");
   pool()->FlushForTesting();
 
   PostDeleteToOtherThread(std::move(sequence_checked_object));
   other_thread()->Stop();
 }
 
-#if ENABLE_SEQUENCE_CHECKER
+#if DCHECK_IS_ON()
 TEST_F(SequenceCheckerTest, DifferentSequenceTokensDeathTestInDebug) {
   // The default style "fast" does not support multi-threaded tests.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   ASSERT_DCHECK_DEATH({ DifferentSequenceTokensDeathTest(); }, "");
 }
 #else
 TEST_F(SequenceCheckerTest, DifferentSequenceTokensDeathTestInRelease) {
   DifferentSequenceTokensDeathTest();
 }
-#endif  // ENABLE_SEQUENCE_CHECKER
+#endif  // DCHECK_IS_ON()
 
 void SequenceCheckerTest::WorkerPoolAndSimpleThreadDeathTest() {
   std::unique_ptr<SequenceCheckedObject> sequence_checked_object(
       new SequenceCheckedObject);
 
   sequence_checked_object->DetachFromSequence();
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   pool()->FlushForTesting();
 
   PostDoStuffToOtherThread(sequence_checked_object.get());
   other_thread()->Stop();
 }
 
-#if ENABLE_SEQUENCE_CHECKER
+#if DCHECK_IS_ON()
 TEST_F(SequenceCheckerTest, WorkerPoolAndSimpleThreadDeathTestInDebug) {
   // The default style "fast" does not support multi-threaded tests.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   ASSERT_DCHECK_DEATH({ WorkerPoolAndSimpleThreadDeathTest(); }, "");
 }
 #else
 TEST_F(SequenceCheckerTest, WorkerPoolAndSimpleThreadDeathTestInRelease) {
   WorkerPoolAndSimpleThreadDeathTest();
 }
-#endif  // ENABLE_SEQUENCE_CHECKER
+#endif  // DCHECK_IS_ON()
 
 void SequenceCheckerTest::TwoDifferentWorkerPoolsDeathTest() {
   std::unique_ptr<SequenceCheckedObject> sequence_checked_object(
       new SequenceCheckedObject);
 
   sequence_checked_object->DetachFromSequence();
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   PostDoStuffToWorkerPool(sequence_checked_object.get(), "A");
   pool()->FlushForTesting();
 
   SequencedWorkerPoolOwner second_pool_owner(kNumWorkerThreads, "test2");
   second_pool_owner.pool()->PostNamedSequencedWorkerTask(
       "A",
       FROM_HERE,
       base::Bind(&SequenceCheckedObject::DoStuff,
                  base::Unretained(sequence_checked_object.get())));
   second_pool_owner.pool()->FlushForTesting();
 }
 
-#if ENABLE_SEQUENCE_CHECKER
+#if DCHECK_IS_ON()
 TEST_F(SequenceCheckerTest, TwoDifferentWorkerPoolsDeathTestInDebug) {
   // The default style "fast" does not support multi-threaded tests.
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
   ASSERT_DCHECK_DEATH({ TwoDifferentWorkerPoolsDeathTest(); }, "");
 }
 #else
 TEST_F(SequenceCheckerTest, TwoDifferentWorkerPoolsDeathTestInRelease) {
   TwoDifferentWorkerPoolsDeathTest();
 }
-#endif  // ENABLE_SEQUENCE_CHECKER
+#endif  // DCHECK_IS_ON()
 
 }  // namespace
 
 }  // namespace base
-
-// Just in case we ever get lumped together with other compilation units.
-#undef ENABLE_SEQUENCE_CHECKER