Revert "base: Add ENABLE_LOG_ERROR_NOT_REACHED buildflag"

This reverts commit 3553b590.

Reason for revert: Causing win-rel compile error.

Original change's description:
> base: Add ENABLE_LOG_ERROR_NOT_REACHED buildflag
> 
> Currently NOTREACHED() uses LogErrorNotReached() on ChromeOS for release
> builds, and DCHECK(false) otherwise. However, there are other projects
> depending on Chromium that would prefer using DCHECK(false) in all cases,
> and currently there's no easy way to do so.
> 
> This CL adds an |enable_log_error_not_reached| gn arg to control whether
> to use LogErrorNotReached() for NOTREACHED() to solve this problem. It
> is enabled by default only for ChromeOS release builds to keep the
> current behavior.
> 
> Bug: 760393
> Change-Id: I512227563d8ad4a08a8d7b5d1ec966a8aacbcc08
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1586593
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#655701}

TBR=thakis@chromium.org,xhwang@chromium.org

Change-Id: Iadcee95984e8783f26aeb687dc1cd31f9186e03d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 760393
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592478Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#655714}

base/BUILD.gn

@@ -25,7 +25,6 @@ import("//build/config/chromecast_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/dcheck_always_on.gni")
 import("//build/config/jumbo.gni")
-import("//build/config/logging.gni")
 import("//build/config/nacl/config.gni")
 import("//build/config/sysroot.gni")
 import("//build/config/ui.gni")
@@ -1233,7 +1232,6 @@ jumbo_component("base") {
     ":build_date",
     ":cfi_buildflags",
     ":debugging_buildflags",
-    ":logging_buildflags",
     ":orderfile_buildflags",
     ":partition_alloc_buildflags",
     ":protected_memory_buildflags",
@@ -2007,12 +2005,6 @@ buildflag_header("debugging_buildflags") {
   ]
 }
 
-buildflag_header("logging_buildflags") {
-  header = "logging_buildflags.h"
-
-  flags = [ "ENABLE_LOG_ERROR_NOT_REACHED=$enable_log_error_not_reached" ]
-}
-
 buildflag_header("orderfile_buildflags") {
   header = "orderfile_buildflags.h"
   header_dir = "base/android/orderfile"

base/logging.h

@@ -18,7 +18,6 @@
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/debug/debugger.h"
-#include "base/logging_buildflags.h"
 #include "base/macros.h"
 #include "base/scoped_clear_last_error.h"
 #include "base/strings/string_piece_forward.h"
@@ -944,7 +943,7 @@ const LogSeverity LOG_DCHECK = LOG_FATAL;
 #define DCHECK_GE(val1, val2) DCHECK_OP(GE, >=, val1, val2)
 #define DCHECK_GT(val1, val2) DCHECK_OP(GT, > , val1, val2)
 
-#if BUILDFLAG(ENABLE_LOG_ERROR_NOT_REACHED)
+#if !DCHECK_IS_ON() && defined(OS_CHROMEOS)
 // Implement logging of NOTREACHED() as a dedicated function to get function
 // call overhead down to a minimum.
 void LogErrorNotReached(const char* file, int line);

build/config/logging.gni

-# Copyright 2019 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.
-
-import("//build/config/dcheck_always_on.gni")
-
-declare_args() {
-  # Use LogErrorNotReached() for NOTREACHED().
-  enable_log_error_not_reached = is_chromeos && !(is_debug || dcheck_always_on)
-}

chrome/browser/resources/chromeos/zip_archiver/BUILD.gn

@@ -96,8 +96,4 @@ static_library("char_coding") {
     "cpp/char_coding.cc",
     "cpp/char_coding.h",
   ]
-
-  deps = [
-    "//base",
-  ]
 }

components/version_info/android/BUILD.gn

@@ -32,7 +32,6 @@ static_library("channel_getter") {
   deps = [
     ":version_constants_bridge_jni",
     "..:channel",
-    "//base",  # for jni_generator_helper.h
   ]
 }