Remove/replace unnecessary includes of logging.h (services/)

If the file just needs the CHECK/CHECK_OP/NOTREACHED
macros, use the appropriate header for that instead.
Or if logging.h is not needed at all, remove it.

This is both a nice cleanup (logging.h is a big header,
and including it unnecessarily has compile-time costs),
and part of the final step towards making logging.h no
longer include check.h and the others.

TBR=jorgelo,meacer

Bug: 1031540
Change-Id: Iaa243f8ff7ccfd865f26b765606314c453b90329
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254665
Commit-Queue: Hans Wennborg <hans@chromium.org>
Auto-Submit: Hans Wennborg <hans@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#780734}

services/audio/group_coordinator.h

@@ -10,7 +10,7 @@
 #include <vector>
 
 #include "base/callback.h"
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/macros.h"
 #include "base/sequence_checker.h"
 #include "base/stl_util.h"

services/device/geolocation/wifi_data_provider_common.h

@@ -10,7 +10,6 @@
 
 #include <memory>
 
-#include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/strings/string16.h"

services/device/hid/hid_service.h

@@ -11,7 +11,7 @@
 #include <vector>
 
 #include "base/bind_helpers.h"
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"

services/device/media_transfer_protocol/mtp_device_manager.h

@@ -13,7 +13,6 @@
 #include "base/containers/flat_map.h"
 #include "base/containers/flat_set.h"
 #include "base/containers/queue.h"
-#include "base/logging.h"
 #include "base/memory/weak_ptr.h"
 #include "base/sequenced_task_runner.h"
 #include "base/threading/thread_checker.h"

services/device/public/cpp/generic_sensor/platform_sensor_configuration.h

@@ -5,7 +5,6 @@
 #ifndef SERVICES_DEVICE_PUBLIC_CPP_GENERIC_SENSOR_PLATFORM_SENSOR_CONFIGURATION_H_
 #define SERVICES_DEVICE_PUBLIC_CPP_GENERIC_SENSOR_PLATFORM_SENSOR_CONFIGURATION_H_
 
-#include "base/logging.h"
 
 namespace device {
 

services/device/usb/usb_service.h

@@ -12,7 +12,7 @@
 #include <vector>
 
 #include "base/bind_helpers.h"
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/observer_list.h"

services/network/cert_verifier_with_trust_anchors.cc

@@ -7,7 +7,7 @@
 #include <utility>
 
 #include "base/bind.h"
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "net/base/net_errors.h"
 #include "net/cert/caching_cert_verifier.h"
 #include "net/cert/cert_verifier.h"

services/network/public/cpp/data_element.h

@@ -14,11 +14,11 @@
 #include <string>
 #include <vector>
 
+#include "base/check_op.h"
 #include "base/component_export.h"
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/gtest_prod_util.h"
-#include "base/logging.h"
 #include "base/time/time.h"
 #include "mojo/public/cpp/bindings/enum_traits.h"
 #include "mojo/public/cpp/bindings/pending_remote.h"

services/network/public/cpp/spki_hash_set.cc

@@ -3,6 +3,9 @@
 // found in the LICENSE file.
 
 #include "services/network/public/cpp/spki_hash_set.h"
+
+#include "base/check_op.h"
+#include "base/logging.h"
 #include "net/base/hash_value.h"
 
 namespace network {

services/proxy_resolver/mojo_proxy_resolver_v8_tracing_bindings.h

@@ -9,7 +9,7 @@
 #include <string>
 #include <utility>
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_checker.h"
 #include "net/base/address_family.h"

services/service_manager/sandbox/linux/sandbox_linux.h

@@ -9,7 +9,7 @@
 #include <string>
 #include <vector>
 
-#include "base/logging.h"
+#include "base/check_op.h"
 #include "base/macros.h"
 #include "base/posix/global_descriptors.h"
 #include "sandbox/linux/syscall_broker/broker_command.h"

services/service_manager/sandbox/win/sandbox_diagnostics.h

@@ -14,7 +14,6 @@
 #include <vector>
 
 #include "base/callback.h"
-#include "base/logging.h"
 #include "base/sequenced_task_runner.h"
 #include "base/values.h"
 #include "sandbox/constants.h"

services/service_manager/service_instance.h

@@ -12,7 +12,6 @@
 #include <string>
 
 #include "base/containers/unique_ptr_adapters.h"
-#include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/optional.h"

services/tracing/public/cpp/perfetto/system_trace_writer.h

@@ -8,8 +8,8 @@
 #include <list>
 
 #include "base/bind.h"
+#include "base/check.h"
 #include "base/component_export.h"
-#include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/memory/weak_ptr.h"

services/viz/public/cpp/compositing/quads_mojom_traits.h

@@ -5,8 +5,9 @@
 #ifndef SERVICES_VIZ_PUBLIC_CPP_COMPOSITING_QUADS_MOJOM_TRAITS_H_
 #define SERVICES_VIZ_PUBLIC_CPP_COMPOSITING_QUADS_MOJOM_TRAITS_H_
 
+#include "base/check.h"
 #include "base/containers/span.h"
-#include "base/logging.h"
+#include "base/notreached.h"
 #include "base/unguessable_token.h"
 #include "components/viz/common/quads/debug_border_draw_quad.h"
 #include "components/viz/common/quads/picture_draw_quad.h"

services/viz/public/cpp/compositing/render_pass_mojom_traits.h

@@ -6,8 +6,9 @@
 #define SERVICES_VIZ_PUBLIC_CPP_COMPOSITING_RENDER_PASS_MOJOM_TRAITS_H_
 
 #include <memory>
+#include <vector>
 
-#include "base/logging.h"
+#include "base/check.h"
 #include "components/viz/common/quads/render_pass.h"
 #include "services/viz/public/cpp/compositing/copy_output_request_mojom_traits.h"
 #include "services/viz/public/cpp/compositing/quads_mojom_traits.h"