Remove/replace unnecessary logging.h includes in .cc files (ios)

CHECK, CHECK_EQ etc., and NOTREACHED/NOTIMPLEMENTED have moved
to the much smaller headers check.h, check_op.h, and notreached.h,
respectively.

This CL updates .cc files to use those headers instead when
possible, with the purpose of saving compile time.

(Split out from https://crrev.com/c/2164525 which also has
notes on how the change was generated.)

Bug: 1031540
Change-Id: I7ddc52fb3cc2ecdf730bdac44603760ef885f9c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2164530
Commit-Queue: Hans Wennborg <hans@chromium.org>
Commit-Queue: Mark Cogan <marq@chromium.org>
Auto-Submit: Hans Wennborg <hans@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#762786}

ios/chrome/browser/application_context.cc

@@ -4,7 +4,6 @@
 
 #include "ios/chrome/browser/application_context.h"
 
-#include "base/logging.h"
 
 namespace {
 // Global ApplicationContext instance.

ios/chrome/browser/application_context_impl.cc

@@ -8,10 +8,10 @@
 #include <vector>
 
 #include "base/bind.h"
+#include "base/check_op.h"
 #include "base/command_line.h"
 #include "base/feature_list.h"
 #include "base/files/file_path.h"
-#include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/path_service.h"

ios/chrome/browser/bookmarks/bookmark_client_impl.cc

@@ -4,7 +4,6 @@
 
 #include "ios/chrome/browser/bookmarks/bookmark_client_impl.h"
 
-#include "base/logging.h"
 #include "base/metrics/user_metrics.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "components/bookmarks/browser/bookmark_node.h"

ios/chrome/browser/bookmarks/bookmark_remover_helper.cc

@@ -5,7 +5,8 @@
 #include "ios/chrome/browser/bookmarks/bookmark_remover_helper.h"
 
 #include "base/bind.h"
-#include "base/logging.h"
+#include "base/check_op.h"
+#include "base/notreached.h"
 #include "base/threading/sequenced_task_runner_handle.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "ios/chrome/browser/bookmarks/bookmark_model_factory.h"

ios/chrome/browser/bookmarks/bookmarks_utils.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/bookmarks/bookmarks_utils.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/stl_util.h"
 #include "components/bookmarks/browser/bookmark_model.h"

ios/chrome/browser/browser_about_rewriter.cc

@@ -6,8 +6,8 @@
 
 #include <string>
 
+#include "base/check.h"
 #include "base/feature_list.h"
-#include "base/logging.h"
 #include "base/stl_util.h"
 #include "components/url_formatter/url_fixer.h"
 #include "ios/chrome/browser/chrome_url_constants.h"

ios/chrome/browser/browser_state/browser_state_info_cache.cc

@@ -10,9 +10,10 @@
 #include <memory>
 #include <utility>
 
+#include "base/check_op.h"
 #include "base/i18n/case_conversion.h"
-#include "base/logging.h"
 #include "base/memory/ptr_util.h"
+#include "base/notreached.h"
 #include "base/values.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/scoped_user_pref_update.h"

ios/chrome/browser/browser_state/chrome_browser_state_impl.cc

@@ -6,9 +6,9 @@
 
 #include <utility>
 
+#include "base/check.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
-#include "base/logging.h"
 #include "base/sequenced_task_runner.h"
 #include "base/threading/thread_restrictions.h"
 #include "components/bookmarks/browser/bookmark_model.h"

ios/chrome/browser/browser_state/off_the_record_chrome_browser_state_impl.cc

@@ -4,8 +4,8 @@
 
 #include "ios/chrome/browser/browser_state/off_the_record_chrome_browser_state_impl.h"
 
-#include "base/logging.h"
 #include "base/metrics/histogram_functions.h"
+#include "base/notreached.h"
 #include "base/sequenced_task_runner.h"
 #include "base/task/post_task.h"
 #include "components/keyed_service/ios/browser_state_dependency_manager.h"

ios/chrome/browser/history/history_client_impl.cc

@@ -6,7 +6,7 @@
 
 #include "base/bind.h"
 #include "base/callback.h"
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/bookmarks/browser/model_loader.h"
 #include "components/history/core/browser/history_service.h"

ios/chrome/browser/overlays/overlay_callback_manager_impl.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/overlays/overlay_callback_manager_impl.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 
 OverlayCallbackManagerImpl::OverlayCallbackManagerImpl() = default;
 

ios/chrome/browser/overlays/overlay_dispatch_callback.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/overlays/public/overlay_dispatch_callback.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "ios/chrome/browser/overlays/public/overlay_response_support.h"
 
 OverlayDispatchCallback::OverlayDispatchCallback(

ios/chrome/browser/overlays/overlay_request_support.cc

@@ -4,7 +4,6 @@
 
 #include "ios/chrome/browser/overlays/public/overlay_request_support.h"
 
-#include "base/logging.h"
 #include "base/no_destructor.h"
 
 namespace {

ios/chrome/browser/overlays/overlay_response_support.cc

@@ -4,7 +4,6 @@
 
 #include "ios/chrome/browser/overlays/public/overlay_response_support.h"
 
-#include "base/logging.h"
 #include "base/no_destructor.h"
 
 namespace {

ios/chrome/browser/overlays/test/fake_overlay_request_callback_installer.cc

@@ -6,7 +6,7 @@
 
 #include "base/bind.h"
 #include "base/callback.h"
-#include "base/logging.h"
+#include "base/check.h"
 #include "ios/chrome/browser/overlays/public/overlay_callback_manager.h"
 #include "ios/chrome/browser/overlays/public/overlay_dispatch_callback.h"
 #include "ios/chrome/browser/overlays/public/overlay_response_support.h"

ios/chrome/browser/prefs/ios_chrome_pref_service_factory.cc

@@ -7,8 +7,8 @@
 #include <vector>
 
 #include "base/bind.h"
+#include "base/check.h"
 #include "base/feature_list.h"
-#include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "components/policy/core/browser/browser_policy_connector.h"

ios/chrome/browser/reading_list/offline_url_utils.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/reading_list/offline_url_utils.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"

ios/chrome/browser/rlz/rlz_tracker_delegate_impl.cc

@@ -5,8 +5,9 @@
 #include "ios/chrome/browser/rlz/rlz_tracker_delegate_impl.h"
 
 #include "base/bind.h"
+#include "base/check.h"
 #include "base/command_line.h"
-#include "base/logging.h"
+#include "base/notreached.h"
 #include "components/omnibox/browser/omnibox_event_global_tracker.h"
 #include "components/omnibox/browser/omnibox_log.h"
 #include "components/search_engines/template_url.h"

ios/chrome/browser/search_engines/template_url_service_client_impl.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/search_engines/template_url_service_client_impl.h"
 
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "base/time/time.h"
 #include "components/history/core/browser/history_service.h"
 #include "components/history/core/browser/history_types.h"

ios/chrome/browser/search_engines/ui_thread_search_terms_data.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/browser/search_engines/ui_thread_search_terms_data.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/strings/string16.h"
 #include "components/google/core/common/google_util.h"
 #include "components/omnibox/browser/omnibox_field_trial.h"

ios/chrome/browser/ui/badges/badge_type_util.cc

@@ -4,7 +4,9 @@
 
 #include "ios/chrome/browser/ui/badges/badge_type_util.h"
 
-#include "base/logging.h"
+#include <ostream>
+
+#include "base/notreached.h"
 
 BadgeType BadgeTypeForInfobarType(InfobarType infobar_type) {
   switch (infobar_type) {

ios/chrome/browser/webdata_services/web_data_service_factory.cc

@@ -6,8 +6,8 @@
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/check.h"
 #include "base/files/file_path.h"
-#include "base/logging.h"
 #include "base/no_destructor.h"
 #include "base/task/post_task.h"
 #include "components/autofill/core/browser/webdata/autofill_webdata_service.h"

ios/chrome/common/x_callback_url.cc

@@ -4,7 +4,7 @@
 
 #include "ios/chrome/common/x_callback_url.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "net/base/escape.h"

ios/net/chunked_data_stream_uploader.cc

@@ -4,7 +4,7 @@
 
 #include "ios/net/chunked_data_stream_uploader.h"
 
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 

ios/net/cookies/cookie_cache.cc

@@ -6,7 +6,6 @@
 
 #include <algorithm>
 
-#include "base/logging.h"
 #include "net/cookies/cookie_options.h"
 
 namespace net {

ios/web/common/referrer_util.cc

@@ -4,7 +4,7 @@
 
 #include "ios/web/common/referrer_util.h"
 
-#include "base/logging.h"
+#include "base/notreached.h"
 #include "ios/web/public/navigation/referrer.h"
 #include "url/gurl.h"
 

ios/web/history_state_util.cc

@@ -4,7 +4,7 @@
 
 #include "ios/web/history_state_util.h"
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "url/gurl.h"
 
 namespace web {

ios/web/security/cert_policy.cc

@@ -4,7 +4,6 @@
 
 #include "ios/web/public/security/cert_policy.h"
 
-#include "base/logging.h"
 #include "net/cert/x509_certificate.h"
 
 namespace web {

ios/web/security/certificate_policy_cache.cc

@@ -4,7 +4,7 @@
 
 #include "ios/web/public/security/certificate_policy_cache.h"
 
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "ios/web/public/thread/web_thread.h"
 
 namespace web {

ios/web/webui/web_ui_ios_message_handler.cc

@@ -4,7 +4,7 @@
 
 #include "ios/web/public/webui/web_ui_ios_message_handler.h"
 
-#include "base/logging.h"
+#include "base/notreached.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"