From 6f81b801142a2b8a3befc5d17dd2282215571fc2 Mon Sep 17 00:00:00 2001
From: Pavel Shliak <shlyakpavel@gmail.com>
Date: Sun, 8 Dec 2024 21:26:06 +0400
Subject: [PATCH] Everywhere: Include HashMap only where it's actually used

---
 AK/FlyString.cpp                                               | 2 +-
 Libraries/LibCore/EventReceiver.h                              | 2 +-
 Libraries/LibCore/FileWatcher.h                                | 1 +
 Libraries/LibGC/ConservativeVector.h                           | 1 -
 Libraries/LibJS/AST.cpp                                        | 1 -
 Libraries/LibJS/AST.h                                          | 1 -
 Libraries/LibJS/Runtime/MapIterator.h                          | 1 -
 Libraries/LibJS/Runtime/MapPrototype.cpp                       | 1 -
 Libraries/LibJS/Runtime/Object.h                               | 1 -
 Libraries/LibRegex/RegexMatch.h                                | 1 -
 Libraries/LibRegex/RegexParser.h                               | 1 +
 Libraries/LibUnicode/CurrencyCode.cpp                          | 1 +
 Libraries/LibUnicode/CurrencyCode.h                            | 1 -
 Libraries/LibWeb/Bindings/Intrinsics.cpp                       | 1 -
 Libraries/LibWeb/Crypto/SubtleCrypto.h                         | 1 -
 Libraries/LibWeb/HTML/BrowsingContextGroup.h                   | 1 -
 Libraries/LibWeb/IndexedDB/Internal/Database.h                 | 1 -
 Libraries/LibWeb/Loader/Resource.h                             | 1 -
 Libraries/LibWeb/Worker/WebWorkerClient.h                      | 1 -
 Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm | 1 +
 Libraries/LibWebView/EventLoop/EventLoopImplementationQt.cpp   | 1 +
 Libraries/LibXML/DOM/Node.cpp                                  | 1 -
 Services/RequestServer/Request.h                               | 1 -
 Services/WebContent/WebDriverConnection.h                      | 1 -
 Tests/AK/TestJSON.cpp                                          | 1 -
 UI/Qt/WebContentView.h                                         | 1 -
 26 files changed, 7 insertions(+), 21 deletions(-)

diff --git a/AK/FlyString.cpp b/AK/FlyString.cpp
index 13f08ddfc4e..2f644de8847 100644
--- a/AK/FlyString.cpp
+++ b/AK/FlyString.cpp
@@ -6,7 +6,7 @@
 
 #include <AK/DeprecatedFlyString.h>
 #include <AK/FlyString.h>
-#include <AK/HashMap.h>
+#include <AK/HashTable.h>
 #include <AK/Singleton.h>
 #include <AK/String.h>
 #include <AK/StringData.h>
diff --git a/Libraries/LibCore/EventReceiver.h b/Libraries/LibCore/EventReceiver.h
index 32b44c83e42..ac95020ac87 100644
--- a/Libraries/LibCore/EventReceiver.h
+++ b/Libraries/LibCore/EventReceiver.h
@@ -10,11 +10,11 @@
 #include <AK/ByteString.h>
 #include <AK/Forward.h>
 #include <AK/Function.h>
-#include <AK/HashMap.h>
 #include <AK/Noncopyable.h>
 #include <AK/OwnPtr.h>
 #include <AK/StringView.h>
 #include <AK/TypeCasts.h>
+#include <AK/Vector.h>
 #include <AK/Weakable.h>
 #include <LibCore/Forward.h>
 
diff --git a/Libraries/LibCore/FileWatcher.h b/Libraries/LibCore/FileWatcher.h
index ea05ad0b841..2df8203989e 100644
--- a/Libraries/LibCore/FileWatcher.h
+++ b/Libraries/LibCore/FileWatcher.h
@@ -10,6 +10,7 @@
 #include <AK/ByteString.h>
 #include <AK/EnumBits.h>
 #include <AK/Function.h>
+#include <AK/HashMap.h>
 #include <AK/Noncopyable.h>
 #include <AK/NonnullRefPtr.h>
 #include <AK/RefCounted.h>
diff --git a/Libraries/LibGC/ConservativeVector.h b/Libraries/LibGC/ConservativeVector.h
index 150525a7cea..8277e6bcbcd 100644
--- a/Libraries/LibGC/ConservativeVector.h
+++ b/Libraries/LibGC/ConservativeVector.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <AK/IntrusiveList.h>
 #include <AK/Vector.h>
 #include <LibGC/Cell.h>
diff --git a/Libraries/LibJS/AST.cpp b/Libraries/LibJS/AST.cpp
index 6764b1ac872..d08433cc55c 100644
--- a/Libraries/LibJS/AST.cpp
+++ b/Libraries/LibJS/AST.cpp
@@ -7,7 +7,6 @@
  */
 
 #include <AK/Demangle.h>
-#include <AK/HashMap.h>
 #include <AK/HashTable.h>
 #include <AK/QuickSort.h>
 #include <AK/ScopeGuard.h>
diff --git a/Libraries/LibJS/AST.h b/Libraries/LibJS/AST.h
index dec69d2ec59..cd3fe0b6c15 100644
--- a/Libraries/LibJS/AST.h
+++ b/Libraries/LibJS/AST.h
@@ -10,7 +10,6 @@
 
 #include <AK/ByteString.h>
 #include <AK/DeprecatedFlyString.h>
-#include <AK/HashMap.h>
 #include <AK/OwnPtr.h>
 #include <AK/RefPtr.h>
 #include <AK/Variant.h>
diff --git a/Libraries/LibJS/Runtime/MapIterator.h b/Libraries/LibJS/Runtime/MapIterator.h
index 224dec301fb..e2d4f1fd31a 100644
--- a/Libraries/LibJS/Runtime/MapIterator.h
+++ b/Libraries/LibJS/Runtime/MapIterator.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <LibJS/Runtime/Map.h>
 #include <LibJS/Runtime/Object.h>
 
diff --git a/Libraries/LibJS/Runtime/MapPrototype.cpp b/Libraries/LibJS/Runtime/MapPrototype.cpp
index 048e891f7fe..bddd63d01c0 100644
--- a/Libraries/LibJS/Runtime/MapPrototype.cpp
+++ b/Libraries/LibJS/Runtime/MapPrototype.cpp
@@ -4,7 +4,6 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
-#include <AK/HashMap.h>
 #include <AK/TypeCasts.h>
 #include <LibJS/Runtime/AbstractOperations.h>
 #include <LibJS/Runtime/KeyedCollections.h>
diff --git a/Libraries/LibJS/Runtime/Object.h b/Libraries/LibJS/Runtime/Object.h
index 606d86fb9e6..e1032884c66 100644
--- a/Libraries/LibJS/Runtime/Object.h
+++ b/Libraries/LibJS/Runtime/Object.h
@@ -8,7 +8,6 @@
 #pragma once
 
 #include <AK/Badge.h>
-#include <AK/HashMap.h>
 #include <AK/StringView.h>
 #include <LibGC/CellAllocator.h>
 #include <LibGC/MarkedVector.h>
diff --git a/Libraries/LibRegex/RegexMatch.h b/Libraries/LibRegex/RegexMatch.h
index bdeb3baaaef..1593647fbe8 100644
--- a/Libraries/LibRegex/RegexMatch.h
+++ b/Libraries/LibRegex/RegexMatch.h
@@ -13,7 +13,6 @@
 #include <AK/ByteString.h>
 #include <AK/COWVector.h>
 #include <AK/DeprecatedFlyString.h>
-#include <AK/HashMap.h>
 #include <AK/MemMem.h>
 #include <AK/RedBlackTree.h>
 #include <AK/StringBuilder.h>
diff --git a/Libraries/LibRegex/RegexParser.h b/Libraries/LibRegex/RegexParser.h
index d5634e89246..6ac48c00531 100644
--- a/Libraries/LibRegex/RegexParser.h
+++ b/Libraries/LibRegex/RegexParser.h
@@ -12,6 +12,7 @@
 #include "RegexOptions.h"
 
 #include <AK/Forward.h>
+#include <AK/HashMap.h>
 #include <AK/Types.h>
 #include <AK/Vector.h>
 #include <LibUnicode/Forward.h>
diff --git a/Libraries/LibUnicode/CurrencyCode.cpp b/Libraries/LibUnicode/CurrencyCode.cpp
index 1457f0594b8..e6a1326318e 100644
--- a/Libraries/LibUnicode/CurrencyCode.cpp
+++ b/Libraries/LibUnicode/CurrencyCode.cpp
@@ -4,6 +4,7 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
+#include <AK/HashMap.h>
 #include <LibUnicode/CurrencyCode.h>
 
 namespace Unicode {
diff --git a/Libraries/LibUnicode/CurrencyCode.h b/Libraries/LibUnicode/CurrencyCode.h
index 11ca5f1c3d9..c33d31e8e36 100644
--- a/Libraries/LibUnicode/CurrencyCode.h
+++ b/Libraries/LibUnicode/CurrencyCode.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <AK/Optional.h>
 #include <AK/Span.h>
 #include <AK/StringView.h>
diff --git a/Libraries/LibWeb/Bindings/Intrinsics.cpp b/Libraries/LibWeb/Bindings/Intrinsics.cpp
index e077138ed57..de2585589d2 100644
--- a/Libraries/LibWeb/Bindings/Intrinsics.cpp
+++ b/Libraries/LibWeb/Bindings/Intrinsics.cpp
@@ -4,7 +4,6 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
-#include <AK/HashMap.h>
 #include <LibJS/Forward.h>
 #include <LibJS/Runtime/NativeFunction.h>
 #include <LibJS/Runtime/Object.h>
diff --git a/Libraries/LibWeb/Crypto/SubtleCrypto.h b/Libraries/LibWeb/Crypto/SubtleCrypto.h
index 6cc934a973d..f8ca8e032ed 100644
--- a/Libraries/LibWeb/Crypto/SubtleCrypto.h
+++ b/Libraries/LibWeb/Crypto/SubtleCrypto.h
@@ -7,7 +7,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <AK/String.h>
 #include <AK/Vector.h>
 #include <LibJS/Forward.h>
diff --git a/Libraries/LibWeb/HTML/BrowsingContextGroup.h b/Libraries/LibWeb/HTML/BrowsingContextGroup.h
index 1669518b036..583b024e7ac 100644
--- a/Libraries/LibWeb/HTML/BrowsingContextGroup.h
+++ b/Libraries/LibWeb/HTML/BrowsingContextGroup.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <AK/NonnullRefPtr.h>
 #include <AK/WeakPtr.h>
 #include <LibJS/Heap/Cell.h>
diff --git a/Libraries/LibWeb/IndexedDB/Internal/Database.h b/Libraries/LibWeb/IndexedDB/Internal/Database.h
index 4b2de0f8a32..8550eaa59ac 100644
--- a/Libraries/LibWeb/IndexedDB/Internal/Database.h
+++ b/Libraries/LibWeb/IndexedDB/Internal/Database.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <LibGC/Ptr.h>
 #include <LibWeb/DOM/EventTarget.h>
 #include <LibWeb/IndexedDB/IDBDatabase.h>
diff --git a/Libraries/LibWeb/Loader/Resource.h b/Libraries/LibWeb/Loader/Resource.h
index d06c06d951d..bd475594c5d 100644
--- a/Libraries/LibWeb/Loader/Resource.h
+++ b/Libraries/LibWeb/Loader/Resource.h
@@ -7,7 +7,6 @@
 #pragma once
 
 #include <AK/ByteBuffer.h>
-#include <AK/HashMap.h>
 #include <AK/HashTable.h>
 #include <AK/Noncopyable.h>
 #include <AK/RefCounted.h>
diff --git a/Libraries/LibWeb/Worker/WebWorkerClient.h b/Libraries/LibWeb/Worker/WebWorkerClient.h
index e1f4df0f3c4..6e40be7e187 100644
--- a/Libraries/LibWeb/Worker/WebWorkerClient.h
+++ b/Libraries/LibWeb/Worker/WebWorkerClient.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <LibIPC/ConnectionToServer.h>
 #include <LibWeb/Worker/WebWorkerClientEndpoint.h>
 #include <LibWeb/Worker/WebWorkerServerEndpoint.h>
diff --git a/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm b/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm
index 4362367d273..8652d2c5267 100644
--- a/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm
+++ b/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm
@@ -5,6 +5,7 @@
  */
 
 #include <AK/Assertions.h>
+#include <AK/HashMap.h>
 #include <AK/IDAllocator.h>
 #include <AK/Singleton.h>
 #include <AK/TemporaryChange.h>
diff --git a/Libraries/LibWebView/EventLoop/EventLoopImplementationQt.cpp b/Libraries/LibWebView/EventLoop/EventLoopImplementationQt.cpp
index 15d64919c18..18a7fb6bd70 100644
--- a/Libraries/LibWebView/EventLoop/EventLoopImplementationQt.cpp
+++ b/Libraries/LibWebView/EventLoop/EventLoopImplementationQt.cpp
@@ -4,6 +4,7 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
+#include <AK/HashMap.h>
 #include <AK/IDAllocator.h>
 #include <AK/Singleton.h>
 #include <AK/TemporaryChange.h>
diff --git a/Libraries/LibXML/DOM/Node.cpp b/Libraries/LibXML/DOM/Node.cpp
index 0fb50ead62a..088088fa3f4 100644
--- a/Libraries/LibXML/DOM/Node.cpp
+++ b/Libraries/LibXML/DOM/Node.cpp
@@ -4,7 +4,6 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
-#include <AK/HashMap.h>
 #include <AK/NonnullOwnPtr.h>
 #include <LibXML/DOM/Node.h>
 
diff --git a/Services/RequestServer/Request.h b/Services/RequestServer/Request.h
index a166dd608f4..ade90b26262 100644
--- a/Services/RequestServer/Request.h
+++ b/Services/RequestServer/Request.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <AK/HashMap.h>
 #include <AK/NonnullOwnPtr.h>
 #include <AK/Optional.h>
 #include <AK/RefCounted.h>
diff --git a/Services/WebContent/WebDriverConnection.h b/Services/WebContent/WebDriverConnection.h
index a5d7380df47..c1789c222df 100644
--- a/Services/WebContent/WebDriverConnection.h
+++ b/Services/WebContent/WebDriverConnection.h
@@ -9,7 +9,6 @@
 #pragma once
 
 #include <AK/ByteString.h>
-#include <AK/HashMap.h>
 #include <AK/String.h>
 #include <LibGC/MarkedVector.h>
 #include <LibGfx/Rect.h>
diff --git a/Tests/AK/TestJSON.cpp b/Tests/AK/TestJSON.cpp
index 3178f2dd64c..438f9901672 100644
--- a/Tests/AK/TestJSON.cpp
+++ b/Tests/AK/TestJSON.cpp
@@ -7,7 +7,6 @@
 #include <LibTest/TestCase.h>
 
 #include <AK/ByteString.h>
-#include <AK/HashMap.h>
 #include <AK/JsonObject.h>
 #include <AK/JsonValue.h>
 #include <AK/StringBuilder.h>
diff --git a/UI/Qt/WebContentView.h b/UI/Qt/WebContentView.h
index 0d20a368830..377c32a0df3 100644
--- a/UI/Qt/WebContentView.h
+++ b/UI/Qt/WebContentView.h
@@ -9,7 +9,6 @@
 
 #include <AK/ByteString.h>
 #include <AK/Function.h>
-#include <AK/HashMap.h>
 #include <AK/OwnPtr.h>
 #include <LibGfx/Forward.h>
 #include <LibGfx/Rect.h>