[heap] Cleanup some includes

Bug: 
Change-Id: I857f8b3f992faf950917060b3fe705bdd45c3ada
Reviewed-on: https://chromium-review.googlesource.com/638339Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47653}

src/heap/concurrent-marking.cc

@@ -16,7 +16,6 @@
 #include "src/heap/objects-visiting.h"
 #include "src/heap/worklist.h"
 #include "src/isolate.h"
-#include "src/locked-queue-inl.h"
 #include "src/utils-inl.h"
 #include "src/utils.h"
 #include "src/v8.h"

src/heap/heap.cc

@@ -53,7 +53,6 @@
 #include "src/utils-inl.h"
 #include "src/utils.h"
 #include "src/v8.h"
-#include "src/v8threads.h"
 #include "src/vm-state-inl.h"
 
 namespace v8 {

src/heap/heap.h

@@ -6,6 +6,7 @@
 #define V8_HEAP_HEAP_H_
 
 #include <cmath>
+#include <map>
 #include <unordered_map>
 #include <vector>
 
@@ -15,7 +16,6 @@
 #include "src/allocation.h"
 #include "src/assert-scope.h"
 #include "src/base/atomic-utils.h"
-#include "src/debug/debug-interface.h"
 #include "src/globals.h"
 #include "src/heap-symbols.h"
 #include "src/objects.h"
@@ -24,6 +24,11 @@
 #include "src/visitors.h"
 
 namespace v8 {
+
+namespace debug {
+typedef void (*OutOfMemoryCallback)(void* data);
+}  // namespace debug
+
 namespace internal {
 
 namespace heap {
@@ -355,7 +360,6 @@ using v8::MemoryPressureLevel;
     heap->incremental_marking()->RecordWrites(array);                  \
   } while (false)
 
-// Forward declarations.
 class AllocationObserver;
 class ArrayBufferTracker;
 class ConcurrentMarking;

src/heap/incremental-marking.h

@@ -6,18 +6,17 @@
 #define V8_HEAP_INCREMENTAL_MARKING_H_
 
 #include "src/cancelable-task.h"
-#include "src/execution.h"
 #include "src/heap/heap.h"
 #include "src/heap/incremental-marking-job.h"
 #include "src/heap/mark-compact.h"
-#include "src/heap/spaces.h"
-#include "src/objects.h"
 
 namespace v8 {
 namespace internal {
 
-// Forward declarations.
+class HeapObject;
 class MarkBit;
+class Map;
+class Object;
 class PagedSpace;
 
 enum class StepOrigin { kV8, kTask };

src/heap/invalidated-slots.h

@@ -10,7 +10,6 @@
 
 #include "src/allocation.h"
 #include "src/base/atomic-utils.h"
-#include "src/base/bits.h"
 #include "src/utils.h"
 
 namespace v8 {

src/heap/mark-compact-inl.h

@@ -5,9 +5,9 @@
 #ifndef V8_HEAP_MARK_COMPACT_INL_H_
 #define V8_HEAP_MARK_COMPACT_INL_H_
 
+#include "src/base/bits.h"
 #include "src/heap/mark-compact.h"
 #include "src/heap/remembered-set.h"
-#include "src/isolate.h"
 
 namespace v8 {
 namespace internal {

src/heap/mark-compact.cc

@@ -6,44 +6,33 @@
 
 #include <unordered_map>
 
-#include "src/base/atomicops.h"
-#include "src/base/bits.h"
-#include "src/base/sys-info.h"
 #include "src/cancelable-task.h"
 #include "src/code-stubs.h"
 #include "src/compilation-cache.h"
 #include "src/deoptimizer.h"
 #include "src/execution.h"
 #include "src/frames-inl.h"
-#include "src/gdb-jit.h"
 #include "src/global-handles.h"
 #include "src/heap/array-buffer-tracker-inl.h"
-#include "src/heap/array-buffer-tracker.h"
 #include "src/heap/concurrent-marking.h"
 #include "src/heap/gc-tracer.h"
 #include "src/heap/incremental-marking.h"
 #include "src/heap/invalidated-slots-inl.h"
-#include "src/heap/invalidated-slots.h"
 #include "src/heap/item-parallel-job.h"
 #include "src/heap/local-allocator.h"
 #include "src/heap/mark-compact-inl.h"
 #include "src/heap/object-stats.h"
 #include "src/heap/objects-visiting-inl.h"
-#include "src/heap/objects-visiting.h"
 #include "src/heap/spaces-inl.h"
 #include "src/heap/worklist.h"
-#include "src/ic/ic.h"
 #include "src/ic/stub-cache.h"
-#include "src/tracing/tracing-category-observer.h"
 #include "src/transitions-inl.h"
 #include "src/utils-inl.h"
 #include "src/v8.h"
-#include "src/v8threads.h"
 
 namespace v8 {
 namespace internal {
 
-
 const char* Marking::kWhiteBitPattern = "00";
 const char* Marking::kBlackBitPattern = "11";
 const char* Marking::kGreyBitPattern = "10";

src/heap/mark-compact.h

@@ -8,7 +8,6 @@
 #include <deque>
 #include <vector>
 
-#include "src/base/bits.h"
 #include "src/heap/marking.h"
 #include "src/heap/sequential-marking-deque.h"
 #include "src/heap/spaces.h"

src/heap/object-stats.cc

@@ -9,7 +9,6 @@
 #include "src/counters.h"
 #include "src/heap/heap-inl.h"
 #include "src/isolate.h"
-#include "src/macro-assembler.h"
 #include "src/objects/code-cache-inl.h"
 #include "src/objects/compilation-cache-inl.h"
 #include "src/utils.h"

src/heap/objects-visiting.h

@@ -6,9 +6,10 @@
 #define V8_OBJECTS_VISITING_H_
 
 #include "src/allocation.h"
-#include "src/heap/heap.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/string.h"
 
 namespace v8 {

src/heap/scavenge-job.cc

@@ -5,6 +5,7 @@
 #include "src/heap/scavenge-job.h"
 
 #include "src/base/platform/time.h"
+#include "src/heap/gc-tracer.h"
 #include "src/heap/heap-inl.h"
 #include "src/heap/heap.h"
 #include "src/isolate.h"

src/heap/scavenge-job.h

@@ -7,7 +7,6 @@
 
 #include "src/cancelable-task.h"
 #include "src/globals.h"
-#include "src/heap/gc-tracer.h"
 
 namespace v8 {
 namespace internal {
@@ -15,7 +14,6 @@ namespace internal {
 class Heap;
 class Isolate;
 
-
 // This class posts idle tasks and performs scavenges in the idle tasks.
 class V8_EXPORT_PRIVATE ScavengeJob {
  public:

src/heap/spaces.cc

@@ -16,7 +16,6 @@
 #include "src/heap/incremental-marking.h"
 #include "src/heap/mark-compact.h"
 #include "src/heap/slot-set.h"
-#include "src/macro-assembler.h"
 #include "src/msan.h"
 #include "src/objects-inl.h"
 #include "src/snapshot/snapshot.h"

src/heap/spaces.h

@@ -14,8 +14,6 @@
 
 #include "src/allocation.h"
 #include "src/base/atomic-utils.h"
-#include "src/base/atomicops.h"
-#include "src/base/bits.h"
 #include "src/base/iterator.h"
 #include "src/base/platform/mutex.h"
 #include "src/cancelable-task.h"

src/visitors.h

@@ -5,7 +5,7 @@
 #ifndef V8_VISITORS_H_
 #define V8_VISITORS_H_
 
-#include "src/allocation.h"
+#include "src/globals.h"
 
 namespace v8 {
 namespace internal {