Replace CHECK(false) in base/

In most cases NOTREACHED() is now a better option. Also performs
dead-code removal.

Bug: 40122554, 40580068
Change-Id: I5d0ef0b0c05a913a040905f7bebbbbf22985fef8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6011172
Reviewed-by: Lei Zhang <[email protected]>
Commit-Queue: Peter Boström <[email protected]>
Code-Coverage: [email protected] <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1382693}
NOKEYCHECK=True
GitOrigin-RevId: 541196559fa83bb1abe11cedbca23d410a8b00d9

Author: pbos

allocator/partition_allocator/src/partition_alloc/partition_alloc_base/debug/alias.h

@@ -35,7 +35,7 @@ namespace partition_alloc::internal::base::debug {
 //   strncpy(name_copy, p->name, sizeof(name_copy)-1);
 //   name_copy[sizeof(name_copy)-1] = '\0';;
 //   base::debug::alias(name_copy);
-//   CHECK(false);
+//   NOTREACHED();
 //
 // Case #2: Prevent a tail call into a function. This is useful to make sure the
 // function containing the call to base::debug::Alias() will be present in the

allocator/partition_allocator/src/partition_alloc/partition_alloc_base/numerics/safe_conversions_impl.h

@@ -89,7 +89,7 @@ constexpr bool kEnableAsmCode = true;
 constexpr bool kEnableAsmCode = false;
 #endif
 
-// Forces a crash, like a CHECK(false). Used for numeric boundary errors.
+// Forces a crash, like a NOTREACHED(). Used for numeric boundary errors.
 // Also used in a constexpr template to trigger a compilation failure on
 // an error condition.
 struct CheckOnFailure {

barrier_callback.h

@@ -16,6 +16,7 @@
 #include "base/functional/bind.h"
 #include "base/functional/callback.h"
 #include "base/functional/callback_helpers.h"
+#include "base/notreached.h"
 #include "base/synchronization/lock.h"
 #include "base/thread_annotations.h"
 
@@ -55,7 +56,7 @@ class BarrierCallbackInfo {
 
 template <typename T>
 void ShouldNeverRun(T t) {
-  CHECK(false);
+  NOTREACHED();
 }
 
 }  // namespace internal

barrier_callback_unittest.cc

@@ -29,7 +29,7 @@ TEST(BarrierCallbackTest, ErrorToCallCallbackWithZeroCallbacks) {
       base::BarrierCallback<int>(0, base::BindOnce([](std::vector<int>) {}));
   EXPECT_FALSE(barrier_callback.is_null());
 
-  EXPECT_CHECK_DEATH(barrier_callback.Run(3));
+  EXPECT_NOTREACHED_DEATH(barrier_callback.Run(3));
 }
 
 TEST(BarrierCallbackTest, RunAfterNumCallbacks) {

barrier_closure.cc

@@ -9,6 +9,7 @@
 #include "base/atomic_ref_count.h"
 #include "base/functional/bind.h"
 #include "base/memory/ptr_util.h"
+#include "base/notreached.h"
 #include "base/numerics/safe_conversions.h"
 
 namespace base {
@@ -38,7 +39,7 @@ void BarrierInfo::Run() {
 }
 
 void ShouldNeverRun() {
-  CHECK(false);
+  NOTREACHED();
 }
 
 }  // namespace

barrier_closure_unittest.cc

@@ -26,7 +26,7 @@ TEST(BarrierClosureTest, ChecksIfCalledForZeroClosures) {
       base::BarrierClosure(0, base::DoNothing());
   EXPECT_FALSE(barrier_closure.is_null());
 
-  EXPECT_CHECK_DEATH(barrier_closure.Run());
+  EXPECT_NOTREACHED_DEATH(barrier_closure.Run());
 }
 
 TEST(BarrierClosureTest, RunAfterNumClosures) {

debug/alias.h

@@ -33,7 +33,7 @@ namespace debug {
 //   int last_error = err_;
 //   base::debug::Alias(&last_error);
 //   DEBUG_ALIAS_FOR_CSTR(name_copy, p->name, 16);
-//   CHECK(false);
+//   NOTREACHED();
 //
 // Case #2: Prevent a tail call into a function. This is useful to make sure the
 // function containing the call to base::debug::Alias() will be present in the

debug/asan_invalid_access.cc

@@ -15,6 +15,7 @@
 
 #include "base/check.h"
 #include "base/debug/alias.h"
+#include "base/immediate_crash.h"
 #include "build/build_config.h"
 
 #if BUILDFLAG(IS_WIN)
@@ -47,8 +48,9 @@ NOINLINE void CorruptMemoryBlock(bool induce_crash) {
   LONG volatile dummy = InterlockedIncrementFn(array - 1);
   base::debug::Alias(const_cast<LONG*>(&dummy));
 
-  if (induce_crash)
-    CHECK(false);
+  if (induce_crash) {
+    base::ImmediateCrash();
+  }
   delete[] array;
 }
 #endif  // BUILDFLAG(IS_WIN) && defined(ADDRESS_SANITIZER)

debug/gdi_debug_util_win.cc

@@ -16,6 +16,7 @@
 
 #include "base/debug/alias.h"
 #include "base/logging.h"
+#include "base/notreached.h"
 #include "base/process/process.h"
 #include "base/win/scoped_handle.h"
 #include "base/win/win_util.h"
@@ -369,7 +370,7 @@ NOINLINE DWORD GetNumGdiHandles() {
   if (num_gdi_handles == 0) {
     DWORD get_gui_resources_error = GetLastError();
     base::debug::Alias(&get_gui_resources_error);
-    CHECK(false);
+    NOTREACHED();
   }
   return num_gdi_handles;
 }
@@ -426,7 +427,7 @@ void CollectChildGDIUsageAndDie(DWORD parent_pid) {
   } while (Process32Next(snapshot, &proc_entry));
 
   CloseHandle(snapshot);
-  CHECK(false);
+  NOTREACHED();
 }
 
 }  // namespace

debug/invalid_access_win.cc

@@ -15,6 +15,7 @@
 #include <stdlib.h>
 
 #include "base/check.h"
+#include "base/notreached.h"
 #include "build/build_config.h"
 
 namespace base {
@@ -65,7 +66,7 @@ void TerminateWithHeapCorruption() {
     HeapDestroy(heap);
   } __except (EXCEPTION_EXECUTE_HANDLER) {
     // Heap corruption exception should never be caught.
-    CHECK(false);
+    NOTREACHED();
   }
   // Should never reach here.
   abort();
@@ -80,7 +81,7 @@ void TerminateWithControlFlowViolation() {
     IndirectCall(&func);
   } __except (EXCEPTION_EXECUTE_HANDLER) {
     // CFG fast fail should never be caught.
-    CHECK(false);
+    NOTREACHED();
   }
   // Should only reach here if CFG is disabled.
   abort();