[cleanup] Remove unnecessary includes

Change-Id: Ic60f74a60480ea4a74e039c5c87e8c1995e60453
Bug: v8:8834
Reviewed-on: https://chromium-review.googlesource.com/c/1473032Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
Cr-Commit-Position: refs/heads/master@{#59603}

src/heap/factory.h

@@ -16,7 +16,6 @@
 #include "src/messages.h"
 #include "src/objects/code.h"
 #include "src/objects/dictionary.h"
-#include "src/objects/hash-table.h"
 #include "src/objects/js-array.h"
 #include "src/objects/js-regexp.h"
 #include "src/objects/ordered-hash-table.h"

src/heap/objects-visiting.h

@@ -5,35 +5,15 @@
 #ifndef V8_HEAP_OBJECTS_VISITING_H_
 #define V8_HEAP_OBJECTS_VISITING_H_
 
-#include "src/allocation.h"
-#include "src/layout-descriptor.h"
-#include "src/objects-body-descriptors.h"
 #include "src/objects.h"
-#include "src/objects/hash-table.h"
-#include "src/objects/ordered-hash-table.h"
-#include "src/objects/string.h"
+#include "src/objects/fixed-array.h"
+#include "src/objects/map.h"
 #include "src/visitors.h"
 
 namespace v8 {
 namespace internal {
 
-class BigInt;
-class BytecodeArray;
-class DataHandler;
-class EmbedderDataArray;
-class JSArrayBuffer;
-class JSDataView;
-class JSRegExp;
-class JSTypedArray;
-class WeakCell;
-class JSWeakRef;
-class JSWeakCollection;
-class NativeContext;
-class UncompiledDataWithoutPreparseData;
-class UncompiledDataWithPreparseData;
-class WasmInstanceObject;
-
-#define TYPED_VISITOR_ID_LIST(V)                                          \
+#define TYPED_VISITOR_ID_LIST_CLASSES(V)                                  \
   V(AllocationSite, AllocationSite)                                       \
   V(BigInt, BigInt)                                                       \
   V(ByteArray, ByteArray)                                                 \
@@ -51,7 +31,6 @@ class WasmInstanceObject;
   V(FeedbackVector, FeedbackVector)                                       \
   V(FixedArray, FixedArray)                                               \
   V(FixedDoubleArray, FixedDoubleArray)                                   \
-  V(FixedFloat64Array, FixedFloat64Array)                                 \
   V(FixedTypedArrayBase, FixedTypedArrayBase)                             \
   V(JSArrayBuffer, JSArrayBuffer)                                         \
   V(JSDataView, JSDataView)                                               \
@@ -82,6 +61,17 @@ class WasmInstanceObject;
   V(UncompiledDataWithPreparseData, UncompiledDataWithPreparseData)       \
   V(WasmInstanceObject, WasmInstanceObject)
 
+#define FORWARD_DECLARE(TypeName, Type) class Type;
+TYPED_VISITOR_ID_LIST_CLASSES(FORWARD_DECLARE)
+#undef FORWARD_DECLARE
+
+#define TYPED_VISITOR_ID_LIST_TYPEDEFS(V) \
+  V(FixedFloat64Array, FixedFloat64Array)
+
+#define TYPED_VISITOR_ID_LIST(V)   \
+  TYPED_VISITOR_ID_LIST_CLASSES(V) \
+  TYPED_VISITOR_ID_LIST_TYPEDEFS(V)
+
 // The base class for visitors that need to dispatch on object type. The default
 // behavior of all visit functions is to iterate body of the given object using
 // the BodyDescriptor of the object.

src/keys.h

@@ -7,7 +7,7 @@
 
 #include "src/objects.h"
 #include "src/objects/hash-table.h"
-#include "src/objects/ordered-hash-table.h"
+#include "src/objects/js-objects.h"
 
 namespace v8 {
 namespace internal {

src/objects/template-objects.h

@@ -5,8 +5,7 @@
 #ifndef V8_OBJECTS_TEMPLATE_OBJECTS_H_
 #define V8_OBJECTS_TEMPLATE_OBJECTS_H_
 
-#include "src/objects.h"
-#include "src/objects/hash-table.h"
+#include "src/objects/fixed-array.h"
 #include "src/objects/struct.h"
 
 // Has to be the last include (doesn't have include guards):