[iwyu] Fix includes in accounting allocator

R=sigurds@chromium.org

Bug: v8:7490, v8:8834
Change-Id: I5d94c2f262826977bbf15ad32a10875f01304039
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1505432Reviewed-by: Sigurd Schneider <sigurds@chromium.org>
Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#60126}

src/heap/heap.h

@@ -29,6 +29,7 @@
 #include "src/objects/string-table.h"
 #include "src/roots.h"
 #include "src/visitors.h"
+#include "testing/gtest/include/gtest/gtest_prod.h"
 
 namespace v8 {
 

src/isolate-data.h

@@ -11,6 +11,7 @@
 #include "src/roots.h"
 #include "src/thread-local-top.h"
 #include "src/utils.h"
+#include "testing/gtest/include/gtest/gtest_prod.h"
 
 namespace v8 {
 namespace internal {

src/objects/shared-function-info.h

@@ -14,6 +14,7 @@
 #include "src/objects/slots.h"
 #include "src/objects/smi.h"
 #include "src/objects/struct.h"
+#include "testing/gtest/include/gtest/gtest_prod.h"
 #include "torque-generated/class-definitions-from-dsl.h"
 
 // Has to be the last include (doesn't have include guards):

src/profiler/profile-generator.h

@@ -16,6 +16,7 @@
 
 #include "include/v8-profiler.h"
 #include "src/allocation.h"
+#include "src/base/platform/time.h"
 #include "src/builtins/builtins.h"
 #include "src/code-events.h"
 #include "src/profiler/strings-storage.h"

src/zone/accounting-allocator.cc

@@ -4,15 +4,9 @@
 
 #include "src/zone/accounting-allocator.h"
 
-#include <cstdlib>
-
-#if V8_LIBC_BIONIC
-#include <malloc.h>  // NOLINT
-#endif
-
 #include "src/allocation.h"
-#include "src/asan.h"
-#include "src/msan.h"
+#include "src/base/logging.h"
+#include "src/zone/zone-segment.h"
 
 namespace v8 {
 namespace internal {

src/zone/accounting-allocator.h

@@ -7,16 +7,14 @@
 
 #include <atomic>
 
-#include "include/v8-platform.h"
-#include "include/v8.h"
 #include "src/base/macros.h"
-#include "src/base/platform/time.h"
-#include "src/zone/zone-segment.h"
-#include "testing/gtest/include/gtest/gtest_prod.h"  // nogncheck
 
 namespace v8 {
 namespace internal {
 
+class Segment;
+class Zone;
+
 class V8_EXPORT_PRIVATE AccountingAllocator {
  public:
   AccountingAllocator() = default;

src/zone/zone.h

@@ -13,6 +13,7 @@
 #include "src/base/logging.h"
 #include "src/globals.h"
 #include "src/zone/accounting-allocator.h"
+#include "src/zone/zone-segment.h"
 
 #ifndef ZONE_NAME
 #define STRINGIFY(x) #x