WTF: use Forward.h when appropriate instead of Vector.h
authorjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Sep 2017 21:28:55 +0000 (21:28 +0000)
committerjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Sep 2017 21:28:55 +0000 (21:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=176984

Reviewed by Saam Barati.

There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.

Source/JavaScriptCore:

* bytecode/HandlerInfo.h:
* heap/GCIncomingRefCounted.h:
* heap/GCSegmentedArray.h:
* wasm/js/JSWebAssemblyModule.h:

Source/WebCore:

* Modules/entriesapi/FileSystemEntriesCallback.h:
* Modules/indexeddb/IDBEventDispatcher.h:
* Modules/indexeddb/IDBFactory.h:
* Modules/indexeddb/client/IDBConnectionProxy.h:
* Modules/indexeddb/server/IDBConnectionToClientDelegate.h:
* Modules/webdatabase/DatabaseTask.h:
* Modules/websockets/WebSocketChannelClient.h:
* contentextensions/CombinedURLFilters.h:
* crypto/SerializedCryptoKeyWrap.h:
* css/InspectorCSSOMWrappers.h:
* css/PageRuleCollector.h:
* css/parser/CSSParserTokenRange.h:
* dom/DocumentTouch.h:
* dom/MutationCallback.h:
* editing/EditingStyle.h:
* editing/SpellChecker.h:
* editing/markup.h:
* fileapi/ThreadableBlobRegistry.h:
* html/FileListCreator.h:
* inspector/WebHeapAgent.h:
* loader/ContentFilter.cpp:
(WebCore::ContentFilter::ContentFilter):
* loader/ContentFilter.h:
* loader/CookieJar.h:
* loader/FrameLoaderClient.h:
* loader/LoaderStrategy.h:
* loader/SubframeLoader.h:
* page/ChromeClient.h:
* page/FrameSnapshotting.h:
* page/IntersectionObserverCallback.h:
* page/PageSerializer.h:
* page/UserContentURLPattern.h:
* page/scrolling/AxisScrollSnapOffsets.h:
* page/win/FrameWin.h:
* platform/CookiesStrategy.h:
* platform/KeyedCoding.h:
* platform/PasteboardStrategy.h:
* platform/SSLKeyGenerator.h:
* platform/ScrollableArea.h:
* platform/encryptedmedia/CDMFactory.h:
* platform/gamepad/EmptyGamepadProvider.cpp:
* platform/gamepad/GamepadProvider.h:
* platform/gamepad/GamepadProviderClient.h:
* platform/gamepad/PlatformGamepad.h:
* platform/graphics/GeometryUtilities.cpp:
* platform/graphics/GeometryUtilities.h:
* platform/graphics/Icon.h:
* platform/graphics/LayoutRect.h:
* platform/graphics/Path.h:
* platform/graphics/WOFFFileFormat.h:
* platform/graphics/avfoundation/MediaSampleAVFObjC.h:
* platform/graphics/avfoundation/objc/AVAssetTrackUtilities.h:
* platform/graphics/ca/PlatformCAAnimation.h:
* platform/graphics/ca/win/PlatformCALayerWinInternal.h:
* platform/graphics/opentype/OpenTypeMathData.h:
* platform/image-encoders/JPEGImageEncoder.h:
* platform/image-encoders/PNGImageEncoder.h:
* platform/network/BlobRegistry.h:
* platform/network/HTTPParsers.h:
* platform/network/PlatformCookieJar.h:
* platform/network/cf/DownloadBundle.h:
* platform/network/curl/CurlCacheEntry.h:
* platform/network/curl/DownloadBundle.h:
* platform/text/LineEnding.h:
* platform/text/QuotedPrintable.cpp:
* platform/text/QuotedPrintable.h:
* rendering/FlexibleBoxAlgorithm.h:
* rendering/style/QuotesData.h:
* rendering/svg/SVGSubpathData.h:
* storage/StorageEventDispatcher.h:
* style/StyleInvalidator.h:
* style/StyleRelations.h:
* svg/SVGAltGlyphDefElement.h:
* svg/SVGAltGlyphItemElement.h:

Source/WebDriver:

* Session.h:

Source/WebKit:

* Scripts/webkit/LegacyMessages-expected.h:
* Scripts/webkit/Messages-expected.h:
* Shared/API/APIData.h:
* UIProcess/API/APIContextMenuClient.h:
* UIProcess/API/APIFormClient.h:
* UIProcess/API/APIHTTPCookieStore.h:
* UIProcess/API/APINotificationProvider.h:
* UIProcess/API/glib/WebKitNotificationProvider.h:
* UIProcess/Notifications/WebNotificationProvider.h:
* UIProcess/TextCheckerCompletion.h:
* UIProcess/WebCookieManagerProxy.h:
* UIProcess/WebOpenPanelResultListenerProxy.h:
* UIProcess/WebPasteboardProxy.h:
* UIProcess/WebPopupMenuProxy.h:
* UIProcess/WebTextCheckerClient.h:
* WebProcess/Cookies/WebCookieManager.h:
* WebProcess/Gamepad/WebGamepadProvider.h:
* WebProcess/InjectedBundle/API/APIInjectedBundleEditorClient.h:
* WebProcess/InjectedBundle/API/APIInjectedBundlePageContextMenuClient.h:
* WebProcess/InjectedBundle/API/APIInjectedBundlePageLoaderClient.h:
* WebProcess/InjectedBundle/APIInjectedBundleFormClient.h:
* WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
* WebProcess/Network/webrtc/WebRTCMonitor.h:
* WebProcess/Network/webrtc/WebRTCResolver.h:
* WebProcess/WebPage/DrawingArea.h:
* WebProcess/WebPage/WebOpenPanelResultListener.h:

Source/WTF:

* wtf/Forward.h:
* wtf/Vector.h:
* wtf/text/StringVector.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@222113 268f45cc-cd09-0410-ab3c-d52691b4dbfc

113 files changed:
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/bytecode/HandlerInfo.h
Source/JavaScriptCore/heap/GCIncomingRefCounted.h
Source/JavaScriptCore/heap/GCSegmentedArray.h
Source/JavaScriptCore/wasm/js/JSWebAssemblyModule.h
Source/WTF/ChangeLog
Source/WTF/wtf/Forward.h
Source/WTF/wtf/Vector.h
Source/WTF/wtf/text/StringVector.h
Source/WebCore/ChangeLog
Source/WebCore/Modules/entriesapi/FileSystemEntriesCallback.h
Source/WebCore/Modules/indexeddb/IDBEventDispatcher.h
Source/WebCore/Modules/indexeddb/IDBFactory.h
Source/WebCore/Modules/indexeddb/client/IDBConnectionProxy.h
Source/WebCore/Modules/indexeddb/server/IDBConnectionToClientDelegate.h
Source/WebCore/Modules/webdatabase/DatabaseTask.h
Source/WebCore/Modules/websockets/WebSocketChannelClient.h
Source/WebCore/contentextensions/CombinedURLFilters.h
Source/WebCore/crypto/SerializedCryptoKeyWrap.h
Source/WebCore/css/InspectorCSSOMWrappers.h
Source/WebCore/css/PageRuleCollector.h
Source/WebCore/css/parser/CSSParserTokenRange.h
Source/WebCore/dom/DocumentTouch.h
Source/WebCore/dom/MutationCallback.h
Source/WebCore/editing/EditingStyle.h
Source/WebCore/editing/SpellChecker.h
Source/WebCore/editing/markup.h
Source/WebCore/fileapi/ThreadableBlobRegistry.h
Source/WebCore/html/FileListCreator.h
Source/WebCore/inspector/WebHeapAgent.h
Source/WebCore/loader/ContentFilter.cpp
Source/WebCore/loader/ContentFilter.h
Source/WebCore/loader/CookieJar.h
Source/WebCore/loader/FrameLoaderClient.h
Source/WebCore/loader/LoaderStrategy.h
Source/WebCore/loader/SubframeLoader.h
Source/WebCore/page/ChromeClient.h
Source/WebCore/page/FrameSnapshotting.h
Source/WebCore/page/IntersectionObserverCallback.h
Source/WebCore/page/PageSerializer.h
Source/WebCore/page/UserContentURLPattern.h
Source/WebCore/page/scrolling/AxisScrollSnapOffsets.h
Source/WebCore/page/win/FrameWin.h
Source/WebCore/platform/CookiesStrategy.h
Source/WebCore/platform/KeyedCoding.h
Source/WebCore/platform/PasteboardStrategy.h
Source/WebCore/platform/SSLKeyGenerator.h
Source/WebCore/platform/ScrollableArea.h
Source/WebCore/platform/encryptedmedia/CDMFactory.cpp
Source/WebCore/platform/encryptedmedia/CDMFactory.h
Source/WebCore/platform/gamepad/EmptyGamepadProvider.cpp
Source/WebCore/platform/gamepad/GamepadProvider.h
Source/WebCore/platform/gamepad/GamepadProviderClient.h
Source/WebCore/platform/gamepad/PlatformGamepad.h
Source/WebCore/platform/graphics/GeometryUtilities.cpp
Source/WebCore/platform/graphics/GeometryUtilities.h
Source/WebCore/platform/graphics/Icon.h
Source/WebCore/platform/graphics/LayoutRect.h
Source/WebCore/platform/graphics/Path.h
Source/WebCore/platform/graphics/WOFFFileFormat.h
Source/WebCore/platform/graphics/avfoundation/MediaSampleAVFObjC.h
Source/WebCore/platform/graphics/avfoundation/objc/AVAssetTrackUtilities.h
Source/WebCore/platform/graphics/ca/PlatformCAAnimation.h
Source/WebCore/platform/graphics/ca/win/PlatformCALayerWinInternal.h
Source/WebCore/platform/graphics/opentype/OpenTypeMathData.h
Source/WebCore/platform/image-encoders/JPEGImageEncoder.h
Source/WebCore/platform/image-encoders/PNGImageEncoder.h
Source/WebCore/platform/network/BlobRegistry.h
Source/WebCore/platform/network/HTTPParsers.h
Source/WebCore/platform/network/PlatformCookieJar.h
Source/WebCore/platform/network/cf/DownloadBundle.h
Source/WebCore/platform/network/curl/CurlCacheEntry.h
Source/WebCore/platform/network/curl/DownloadBundle.h
Source/WebCore/platform/text/LineEnding.h
Source/WebCore/platform/text/QuotedPrintable.cpp
Source/WebCore/platform/text/QuotedPrintable.h
Source/WebCore/rendering/FlexibleBoxAlgorithm.h
Source/WebCore/rendering/style/QuotesData.h
Source/WebCore/rendering/svg/SVGSubpathData.h
Source/WebCore/storage/StorageEventDispatcher.h
Source/WebCore/style/StyleInvalidator.h
Source/WebCore/style/StyleRelations.h
Source/WebCore/svg/SVGAltGlyphDefElement.h
Source/WebCore/svg/SVGAltGlyphItemElement.h
Source/WebDriver/ChangeLog
Source/WebDriver/Session.h
Source/WebKit/ChangeLog
Source/WebKit/Scripts/webkit/LegacyMessages-expected.h
Source/WebKit/Scripts/webkit/Messages-expected.h
Source/WebKit/Shared/API/APIData.h
Source/WebKit/UIProcess/API/APIContextMenuClient.h
Source/WebKit/UIProcess/API/APIFormClient.h
Source/WebKit/UIProcess/API/APIHTTPCookieStore.h
Source/WebKit/UIProcess/API/APINotificationProvider.h
Source/WebKit/UIProcess/API/glib/WebKitNotificationProvider.h
Source/WebKit/UIProcess/Notifications/WebNotificationProvider.h
Source/WebKit/UIProcess/TextCheckerCompletion.h
Source/WebKit/UIProcess/WebCookieManagerProxy.h
Source/WebKit/UIProcess/WebOpenPanelResultListenerProxy.h
Source/WebKit/UIProcess/WebPasteboardProxy.h
Source/WebKit/UIProcess/WebPopupMenuProxy.h
Source/WebKit/UIProcess/WebTextCheckerClient.h
Source/WebKit/WebProcess/Cookies/WebCookieManager.h
Source/WebKit/WebProcess/Gamepad/WebGamepadProvider.h
Source/WebKit/WebProcess/InjectedBundle/API/APIInjectedBundleEditorClient.h
Source/WebKit/WebProcess/InjectedBundle/API/APIInjectedBundlePageContextMenuClient.h
Source/WebKit/WebProcess/InjectedBundle/API/APIInjectedBundlePageLoaderClient.h
Source/WebKit/WebProcess/InjectedBundle/APIInjectedBundleFormClient.h
Source/WebKit/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h
Source/WebKit/WebProcess/Network/webrtc/WebRTCMonitor.h
Source/WebKit/WebProcess/Network/webrtc/WebRTCResolver.h
Source/WebKit/WebProcess/WebPage/DrawingArea.h
Source/WebKit/WebProcess/WebPage/WebOpenPanelResultListener.h

index 069dc15..d5d111d 100644 (file)
@@ -1,3 +1,17 @@
+2017-09-15  JF Bastien  <jfbastien@apple.com>
+
+        WTF: use Forward.h when appropriate instead of Vector.h
+        https://bugs.webkit.org/show_bug.cgi?id=176984
+
+        Reviewed by Saam Barati.
+
+        There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.
+
+        * bytecode/HandlerInfo.h:
+        * heap/GCIncomingRefCounted.h:
+        * heap/GCSegmentedArray.h:
+        * wasm/js/JSWebAssemblyModule.h:
+
 2017-09-14  Saam Barati  <sbarati@apple.com>
 
         We should have a way of preventing a caller from making a tail call and we should use it for ProxyObject instead of using build flags
index ad7e74c..3e6001a 100644 (file)
@@ -26,7 +26,7 @@
 #pragma once
 
 #include "CodeLocation.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace JSC {
 
index 6c8b73e..1d336bf 100644 (file)
@@ -26,7 +26,7 @@
 #pragma once
 
 #include <wtf/DeferrableRefCounted.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace JSC {
 
index c0aac2c..21beec5 100644 (file)
@@ -26,7 +26,7 @@
 #pragma once
 
 #include <wtf/DoublyLinkedList.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace JSC {
 
index 8361682..3dc67a3 100644 (file)
@@ -32,7 +32,7 @@
 #include "UnconditionalFinalizer.h"
 #include "WasmModule.h"
 #include <wtf/Bag.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace JSC {
 
index bbc2f93..6ff0d98 100644 (file)
@@ -1,3 +1,16 @@
+2017-09-15  JF Bastien  <jfbastien@apple.com>
+
+        WTF: use Forward.h when appropriate instead of Vector.h
+        https://bugs.webkit.org/show_bug.cgi?id=176984
+
+        Reviewed by Saam Barati.
+
+        There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.
+
+        * wtf/Forward.h:
+        * wtf/Vector.h:
+        * wtf/text/StringVector.h:
+
 2017-09-15  Keith Miller  <keith_miller@apple.com>
 
         generate-unified-source-bundles.rb shouldn't write a file that isn't going to change
index 7f3ee42..08f6cb4 100644 (file)
@@ -28,6 +28,9 @@ template<typename T> class optional;
 
 namespace WTF {
 
+struct FastMalloc;
+class CrashOnOverflow;
+
 template<typename T> class Function;
 template<typename T> class LazyNeverDestroyed;
 template<typename T> class NeverDestroyed;
@@ -37,7 +40,7 @@ template<typename T> class RefPtr;
 template<typename T> class StringBuffer;
 
 template<typename... T> class Variant;
-template<typename T, size_t inlineCapacity, typename OverflowHandler, size_t minCapacity, typename Malloc> class Vector;
+template<typename T, size_t inlineCapacity = 0, typename OverflowHandler = CrashOnOverflow, size_t minCapacity = 16, typename Malloc = FastMalloc> class Vector;
 
 class AtomicString;
 class AtomicStringImpl;
index 98d2374..2aebe71 100644 (file)
@@ -28,6 +28,7 @@
 #include <utility>
 #include <wtf/CheckedArithmetic.h>
 #include <wtf/FastMalloc.h>
+#include <wtf/Forward.h>
 #include <wtf/MallocPtr.h>
 #include <wtf/Noncopyable.h>
 #include <wtf/NotFound.h>
@@ -575,7 +576,8 @@ struct UnsafeVectorOverflow {
     }
 };
 
-template<typename T, size_t inlineCapacity = 0, typename OverflowHandler = CrashOnOverflow, size_t minCapacity = 16, typename Malloc = FastMalloc>
+// Template default values are in Forward.h.
+template<typename T, size_t inlineCapacity, typename OverflowHandler, size_t minCapacity, typename Malloc>
 class Vector : private VectorBuffer<T, inlineCapacity, Malloc> {
     WTF_MAKE_FAST_ALLOCATED;
 private:
index 63c611c..d7b23f9 100644 (file)
@@ -25,7 +25,7 @@
 
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/StringMalloc.h>
 
 namespace WTF {
index 1a76ade..0a073a1 100644 (file)
@@ -1,3 +1,87 @@
+2017-09-15  JF Bastien  <jfbastien@apple.com>
+
+        WTF: use Forward.h when appropriate instead of Vector.h
+        https://bugs.webkit.org/show_bug.cgi?id=176984
+
+        Reviewed by Saam Barati.
+
+        There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.
+
+        * Modules/entriesapi/FileSystemEntriesCallback.h:
+        * Modules/indexeddb/IDBEventDispatcher.h:
+        * Modules/indexeddb/IDBFactory.h:
+        * Modules/indexeddb/client/IDBConnectionProxy.h:
+        * Modules/indexeddb/server/IDBConnectionToClientDelegate.h:
+        * Modules/webdatabase/DatabaseTask.h:
+        * Modules/websockets/WebSocketChannelClient.h:
+        * contentextensions/CombinedURLFilters.h:
+        * crypto/SerializedCryptoKeyWrap.h:
+        * css/InspectorCSSOMWrappers.h:
+        * css/PageRuleCollector.h:
+        * css/parser/CSSParserTokenRange.h:
+        * dom/DocumentTouch.h:
+        * dom/MutationCallback.h:
+        * editing/EditingStyle.h:
+        * editing/SpellChecker.h:
+        * editing/markup.h:
+        * fileapi/ThreadableBlobRegistry.h:
+        * html/FileListCreator.h:
+        * inspector/WebHeapAgent.h:
+        * loader/ContentFilter.cpp:
+        (WebCore::ContentFilter::ContentFilter):
+        * loader/ContentFilter.h:
+        * loader/CookieJar.h:
+        * loader/FrameLoaderClient.h:
+        * loader/LoaderStrategy.h:
+        * loader/SubframeLoader.h:
+        * page/ChromeClient.h:
+        * page/FrameSnapshotting.h:
+        * page/IntersectionObserverCallback.h:
+        * page/PageSerializer.h:
+        * page/UserContentURLPattern.h:
+        * page/scrolling/AxisScrollSnapOffsets.h:
+        * page/win/FrameWin.h:
+        * platform/CookiesStrategy.h:
+        * platform/KeyedCoding.h:
+        * platform/PasteboardStrategy.h:
+        * platform/SSLKeyGenerator.h:
+        * platform/ScrollableArea.h:
+        * platform/encryptedmedia/CDMFactory.h:
+        * platform/gamepad/EmptyGamepadProvider.cpp:
+        * platform/gamepad/GamepadProvider.h:
+        * platform/gamepad/GamepadProviderClient.h:
+        * platform/gamepad/PlatformGamepad.h:
+        * platform/graphics/GeometryUtilities.cpp:
+        * platform/graphics/GeometryUtilities.h:
+        * platform/graphics/Icon.h:
+        * platform/graphics/LayoutRect.h:
+        * platform/graphics/Path.h:
+        * platform/graphics/WOFFFileFormat.h:
+        * platform/graphics/avfoundation/MediaSampleAVFObjC.h:
+        * platform/graphics/avfoundation/objc/AVAssetTrackUtilities.h:
+        * platform/graphics/ca/PlatformCAAnimation.h:
+        * platform/graphics/ca/win/PlatformCALayerWinInternal.h:
+        * platform/graphics/opentype/OpenTypeMathData.h:
+        * platform/image-encoders/JPEGImageEncoder.h:
+        * platform/image-encoders/PNGImageEncoder.h:
+        * platform/network/BlobRegistry.h:
+        * platform/network/HTTPParsers.h:
+        * platform/network/PlatformCookieJar.h:
+        * platform/network/cf/DownloadBundle.h:
+        * platform/network/curl/CurlCacheEntry.h:
+        * platform/network/curl/DownloadBundle.h:
+        * platform/text/LineEnding.h:
+        * platform/text/QuotedPrintable.cpp:
+        * platform/text/QuotedPrintable.h:
+        * rendering/FlexibleBoxAlgorithm.h:
+        * rendering/style/QuotesData.h:
+        * rendering/svg/SVGSubpathData.h:
+        * storage/StorageEventDispatcher.h:
+        * style/StyleInvalidator.h:
+        * style/StyleRelations.h:
+        * svg/SVGAltGlyphDefElement.h:
+        * svg/SVGAltGlyphItemElement.h:
+
 2017-09-15  Youenn Fablet  <youenn@apple.com>
 
         ASSERTION FAILED: writtenAudioDuration >= readAudioDuration in com.apple.WebCore:WebCore::RealtimeOutgoingAudioSource::isReachingBufferedAudioDataHighLimit() + 222
index 36578eb..912a4da 100644 (file)
@@ -29,7 +29,6 @@
 #include "CallbackResult.h"
 #include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 47e5e92..97cb7d6 100644 (file)
@@ -30,7 +30,7 @@
 
 #if ENABLE(INDEXED_DATABASE)
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 2969e5e..1f04d13 100644 (file)
@@ -32,7 +32,6 @@
 #include <wtf/Forward.h>
 #include <wtf/Ref.h>
 #include <wtf/ThreadSafeRefCounted.h>
-#include <wtf/Vector.h>
 
 namespace JSC {
 class ExecState;
index cc244d4..b7728c0 100644 (file)
 #include "TransactionOperation.h"
 #include <wtf/CrossThreadQueue.h>
 #include <wtf/CrossThreadTask.h>
+#include <wtf/Forward.h>
 #include <wtf/Function.h>
 #include <wtf/HashMap.h>
 #include <wtf/MainThread.h>
 #include <wtf/RefPtr.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index cc94653..16d4007 100644 (file)
@@ -27,7 +27,7 @@
 
 #if ENABLE(INDEXED_DATABASE)
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 65275be..1da1b1c 100644 (file)
@@ -30,8 +30,8 @@
 
 #include "ExceptionOr.h"
 #include <wtf/Condition.h>
+#include <wtf/Forward.h>
 #include <wtf/Lock.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 5f58d76..8cac905 100644 (file)
@@ -32,7 +32,6 @@
 #include "NFA.h"
 #include <wtf/Function.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 02cf82c..d9786ea 100644 (file)
@@ -25,7 +25,7 @@
 
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 #if ENABLE(SUBTLE_CRYPTO)
index ace93da..4052fea 100644 (file)
@@ -26,7 +26,6 @@
 #include <wtf/HashMap.h>
 #include <wtf/HashSet.h>
 #include <wtf/RefPtr.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 828bc79..7839871 100644 (file)
@@ -23,7 +23,7 @@
 
 #include "DocumentRuleSets.h"
 #include "StyleResolver.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 8a9dd43..735d023 100644 (file)
@@ -30,7 +30,7 @@
 #pragma once
 
 #include "CSSParserToken.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 89b723a..e0daf50 100644 (file)
@@ -30,7 +30,7 @@
 #elif ENABLE(TOUCH_EVENTS)
 
 #include <functional>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 8859d53..3d44293 100644 (file)
@@ -32,9 +32,9 @@
 
 #include "ActiveDOMCallback.h"
 #include "CallbackResult.h"
+#include <wtf/Forward.h>
 #include <wtf/Ref.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 596c08e..4773e07 100644 (file)
@@ -39,7 +39,6 @@
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
 #include <wtf/TriState.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 7180c33..a23c18c 100644 (file)
@@ -30,9 +30,9 @@
 #include "TextChecking.h"
 #include "Timer.h"
 #include <wtf/Deque.h>
+#include <wtf/Forward.h>
 #include <wtf/RefPtr.h>
 #include <wtf/Noncopyable.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index f95b8ce..381c9cb 100644 (file)
@@ -29,7 +29,6 @@
 #include "FragmentScriptingPermission.h"
 #include "HTMLInterchange.h"
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 552a5d9..4246d7e 100644 (file)
@@ -32,7 +32,6 @@
 #pragma once
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 53f7926..31c74cf 100644 (file)
 
 #pragma once
 
+#include <wtf/Forward.h>
 #include <wtf/Function.h>
 #include <wtf/Ref.h>
 #include <wtf/ThreadSafeRefCounted.h>
-#include <wtf/Vector.h>
 #include <wtf/WorkQueue.h>
 
 namespace WebCore {
index 30a8896..b503a59 100644 (file)
@@ -26,7 +26,7 @@
 #pragma once
 
 #include <inspector/agents/InspectorHeapAgent.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index c1a8c85..46680ab 100644 (file)
@@ -81,7 +81,7 @@ std::unique_ptr<ContentFilter> ContentFilter::create(DocumentLoader& documentLoa
     return std::make_unique<ContentFilter>(WTFMove(filters), documentLoader);
 }
 
-ContentFilter::ContentFilter(Container contentFilters, DocumentLoader& documentLoader)
+ContentFilter::ContentFilter(Container&& contentFilters, DocumentLoader& documentLoader)
     : m_contentFilters { WTFMove(contentFilters) }
     , m_documentLoader { documentLoader }
 {
index 8b48166..23ed048 100644 (file)
@@ -31,7 +31,7 @@
 #include "PlatformContentFilter.h"
 #include "ResourceError.h"
 #include <functional>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
@@ -75,7 +75,7 @@ private:
 
     using Container = Vector<std::unique_ptr<PlatformContentFilter>>;
     friend std::unique_ptr<ContentFilter> std::make_unique<ContentFilter>(Container&&, DocumentLoader&);
-    ContentFilter(Container, DocumentLoader&);
+    ContentFilter(Container&&, DocumentLoader&);
 
     template <typename Function> void forEachContentFilterUntilBlocked(Function&&);
     void didDecide(State);
index 128491b..d79a09b 100644 (file)
@@ -26,7 +26,6 @@
 #pragma once
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index a7cf99c..4f822a5 100644 (file)
@@ -34,7 +34,6 @@
 #include "LinkIcon.h"
 #include <functional>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 #if ENABLE(CONTENT_FILTERING)
index d6590a2..5281c2c 100644 (file)
@@ -28,8 +28,8 @@
 #include "ResourceHandleTypes.h"
 #include "ResourceLoadPriority.h"
 #include "ResourceLoaderOptions.h"
+#include <wtf/Forward.h>
 #include <wtf/SHA1.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 12bf771..4896d62 100644 (file)
@@ -33,7 +33,6 @@
 #include "FrameLoaderTypes.h"
 #include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 8cf1899..dcd400a 100644 (file)
@@ -44,7 +44,6 @@
 #include <runtime/ConsoleTypes.h>
 #include <wtf/Forward.h>
 #include <wtf/Seconds.h>
-#include <wtf/Vector.h>
 
 #if ENABLE(WIRELESS_PLAYBACK_TARGET)
 #include "MediaPlaybackTargetContext.h"
index fb3dd03..bdbc7ea 100644 (file)
@@ -30,7 +30,7 @@
 #pragma once
 
 #include <memory>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
@@ -52,7 +52,7 @@ enum {
 typedef unsigned SnapshotOptions;
 
 WEBCORE_EXPORT std::unique_ptr<ImageBuffer> snapshotFrameRect(Frame&, const IntRect&, SnapshotOptions = SnapshotOptionsNone);
-std::unique_ptr<ImageBuffer> snapshotFrameRectWithClip(Frame&, const IntRect&, const Vector<FloatRect>& clipRects = { }, SnapshotOptions = SnapshotOptionsNone);
+std::unique_ptr<ImageBuffer> snapshotFrameRectWithClip(Frame&, const IntRect&, const Vector<FloatRect>& clipRects, SnapshotOptions = SnapshotOptionsNone);
 std::unique_ptr<ImageBuffer> snapshotNode(Frame&, Node&);
 WEBCORE_EXPORT std::unique_ptr<ImageBuffer> snapshotSelection(Frame&, SnapshotOptions = SnapshotOptionsNone);
 
index 8592bc6..c08f9a4 100644 (file)
@@ -29,8 +29,8 @@
 
 #include "ActiveDOMCallback.h"
 #include "CallbackResult.h"
+#include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index d5743bc..3b6e182 100644 (file)
@@ -32,9 +32,9 @@
 
 #include "URLHash.h"
 #include "SharedBuffer.h"
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ListHashSet.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index e9ffe1c..933f69d 100644 (file)
@@ -25,7 +25,7 @@
  
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 6b6b2bc..e1393fb 100644 (file)
@@ -30,7 +30,7 @@
 #include "LayoutUnit.h"
 #include "ScrollSnapOffsetsInfo.h"
 #include "ScrollTypes.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 25413d1..5c5e900 100644 (file)
@@ -25,7 +25,7 @@
 
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/win/GDIObject.h>
 
 namespace WebCore {
index c24dddc..a71dfae 100644 (file)
@@ -27,7 +27,7 @@
 
 #include <pal/SessionID.h>
 #include <wtf/EnumTraits.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 3e1fd83..631ab04 100644 (file)
@@ -28,7 +28,6 @@
 #include <functional>
 #include <wtf/Deque.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 07ae08a..d7ab9a6 100644 (file)
@@ -27,7 +27,6 @@
 #define PasteboardStrategy_h
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 7d6ce2a..e7d6c88 100644 (file)
@@ -26,7 +26,7 @@
 #ifndef SSLKeyGenerator_h
 #define SSLKeyGenerator_h
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 77ff588..4b820fe 100644 (file)
@@ -27,7 +27,7 @@
 
 #include "ScrollSnapOffsetsInfo.h"
 #include "Scrollbar.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/WeakPtr.h>
 
 namespace WebCore {
index b286a2e..575135f 100644 (file)
@@ -32,6 +32,7 @@
 
 #include <mutex>
 #include <wtf/NeverDestroyed.h>
+#include <wtf/Vector.h>
 
 namespace WebCore {
 
index bd48367..44eaab0 100644 (file)
@@ -31,7 +31,6 @@
 
 #include <memory>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 62401c8..2c4f12d 100644 (file)
@@ -29,6 +29,7 @@
 #if ENABLE(GAMEPAD)
 
 #include <wtf/NeverDestroyed.h>
+#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 1d50f2c..f050241 100644 (file)
@@ -27,8 +27,8 @@
 
 #if ENABLE(GAMEPAD)
 
+#include <wtf/Forward.h>
 #include <wtf/HashSet.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 63ea46c..bbbf597 100644 (file)
@@ -27,7 +27,7 @@
 
 #if ENABLE(GAMEPAD)
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 2b8d7a9..eb25fc3 100644 (file)
@@ -27,7 +27,7 @@
 
 #if ENABLE(GAMEPAD)
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 8a72165..0276511 100644 (file)
@@ -25,6 +25,7 @@
 
 #include "config.h"
 #include "GeometryUtilities.h"
+#include <wtf/Vector.h>
 
 namespace WebCore {
 
index ef2e9ae..aa5c76b 100644 (file)
@@ -27,7 +27,7 @@
 
 #include "FloatRect.h"
 #include "IntRect.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 7b6a304..860ea69 100644 (file)
@@ -25,7 +25,6 @@
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
 #include <wtf/RetainPtr.h>
-#include <wtf/Vector.h>
 
 #if PLATFORM(IOS)
 #include "NativeImage.h"
index 9d191f0..8bbc739 100644 (file)
@@ -34,7 +34,7 @@
 #include "IntRect.h"
 #include "LayoutPoint.h"
 #include "LengthBox.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WTF {
 class TextStream;
index 8b2e7e2..92f6f25 100644 (file)
@@ -33,7 +33,6 @@
 #include <wtf/FastMalloc.h>
 #include <wtf/Function.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 #if USE(CG)
 
index 44c3824..fa09d9e 100644 (file)
@@ -26,7 +26,7 @@
 #ifndef WOFFFileFormat_h
 #define WOFFFileFormat_h
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 2132a72..e334ca4 100644 (file)
@@ -29,7 +29,7 @@
 #include "MediaTimeAVFoundation.h"
 
 #include <runtime/Uint8ClampedArray.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
     
index c18b1c5..099988c 100644 (file)
@@ -28,7 +28,7 @@
 #if ENABLE(VIDEO) && USE(AVFOUNDATION)
 
 #include "ContentType.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 OBJC_CLASS AVAssetTrack;
 
index 68d4d5e..799c361 100644 (file)
@@ -30,9 +30,9 @@
 #include "FilterOperation.h"
 #include "FloatPoint3D.h"
 #include "TransformationMatrix.h"
+#include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
 #include <wtf/TypeCasts.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index a562523..cd78148 100644 (file)
@@ -27,8 +27,8 @@
 #define PlatformCALayerWinInternal_h
 
 #include <CoreGraphics/CGGeometry.h>
+#include <wtf/Forward.h>
 #include <wtf/RetainPtr.h>
-#include <wtf/Vector.h>
 
 typedef struct _CACFLayer *CACFLayerRef;
 typedef struct CGContext *CGContextRef;
@@ -41,7 +41,7 @@ class PlatformCALayer;
 class TileController;
 class TiledBacking;
 
-typedef Vector<RefPtr<PlatformCALayer> > PlatformCALayerList;
+typedef Vector<RefPtr<PlatformCALayer>> PlatformCALayerList;
 
 class PlatformCALayerWinInternal {
 public:
index dc34bbb..72327d2 100644 (file)
 #pragma once
 
 #include "Glyph.h"
+#include <wtf/Forward.h>
 #include <wtf/Ref.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
+#include <wtf/RefPtr.h>
 
 #if !ENABLE(OPENTYPE_MATH) && USE(HARFBUZZ)
 #include <hb-ot.h>
index 9f31a91..22bb805 100644 (file)
@@ -20,8 +20,8 @@
 
 #pragma once
 
+#include <wtf/Forward.h>
 #include <wtf/Optional.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 0c09086..08f0b5a 100644 (file)
@@ -21,7 +21,7 @@
 #ifndef PNGImageEncoder_h
 #define PNGImageEncoder_h
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index b121675..9ff600c 100644 (file)
@@ -34,7 +34,6 @@
 #include <functional>
 #include <wtf/Forward.h>
 #include <wtf/Function.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 685e83c..6a82ad2 100644 (file)
@@ -34,7 +34,6 @@
 #include <wtf/Forward.h>
 #include <wtf/HashSet.h>
 #include <wtf/Optional.h>
-#include <wtf/Vector.h>
 #include <wtf/text/StringHash.h>
 #include <wtf/text/WTFString.h>
 
index e8ce508..047273f 100644 (file)
@@ -28,7 +28,6 @@
 #include <chrono>
 #include <wtf/Forward.h>
 #include <wtf/HashSet.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index f33d8a5..8de2245 100644 (file)
@@ -27,7 +27,6 @@
 #define DownloadBundle_h
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 namespace DownloadBundle {
index 4af0464..3b0b205 100644 (file)
@@ -32,9 +32,9 @@
 #include "ResourceHandle.h"
 #include "ResourceRequest.h"
 #include "ResourceResponse.h"
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ListHashSet.h>
-#include <wtf/Vector.h>
 #include <wtf/text/CString.h>
 #include <wtf/text/WTFString.h>
 
index f33d8a5..8de2245 100644 (file)
@@ -27,7 +27,6 @@
 #define DownloadBundle_h
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 namespace DownloadBundle {
index a41cb80..ea44362 100644 (file)
@@ -33,7 +33,6 @@
 #define LineEnding_h
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 47eb3ae..5286de4 100644 (file)
@@ -32,6 +32,7 @@
 #include "QuotedPrintable.h"
 
 #include <wtf/ASCIICType.h>
+#include <wtf/Vector.h>
 
 namespace WebCore {
 
index c3cc337..ab692a2 100644 (file)
@@ -31,7 +31,7 @@
 #ifndef QuotedPrintable_h
 #define QuotedPrintable_h
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 8d7d6a3..4289200 100644 (file)
@@ -32,8 +32,8 @@
 
 #include "LayoutUnit.h"
 #include "RenderStyle.h"
+#include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index 3c743ae..d6778e6 100644 (file)
@@ -21,8 +21,8 @@
 
 #pragma once
 
+#include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 073afd4..0ffbeed 100644 (file)
@@ -20,7 +20,7 @@
 #pragma once
 
 #include "Path.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 6fa9510..8cc4d0b 100644 (file)
@@ -30,7 +30,6 @@
 #pragma once
 
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 
index ec74ba9..7ec5ea0 100644 (file)
@@ -25,7 +25,7 @@
 
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 831099d..8a29281 100644 (file)
@@ -25,7 +25,7 @@
 
 #pragma once
 
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 40996a5..ccd43d7 100644 (file)
@@ -22,7 +22,7 @@
 #if ENABLE(SVG_FONTS)
 
 #include "SVGElement.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 840a928..b8c70f0 100644 (file)
@@ -22,7 +22,7 @@
 #if ENABLE(SVG_FONTS)
 
 #include "SVGElement.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebCore {
 
index 84e8446..156e82a 100644 (file)
@@ -1,3 +1,14 @@
+2017-09-15  JF Bastien  <jfbastien@apple.com>
+
+        WTF: use Forward.h when appropriate instead of Vector.h
+        https://bugs.webkit.org/show_bug.cgi?id=176984
+
+        Reviewed by Saam Barati.
+
+        There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.
+
+        * Session.h:
+
 2017-09-09  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         WebDriver: ensure we close all windows handles when closing the session
index 68cae7e..196aefa 100644 (file)
@@ -30,7 +30,6 @@
 #include <wtf/Function.h>
 #include <wtf/OptionSet.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace Inspector {
index dad7abb..a402f1e 100644 (file)
@@ -1,3 +1,39 @@
+2017-09-15  JF Bastien  <jfbastien@apple.com>
+
+        WTF: use Forward.h when appropriate instead of Vector.h
+        https://bugs.webkit.org/show_bug.cgi?id=176984
+
+        Reviewed by Saam Barati.
+
+        There's no need to include Vector.h when Forward.h will suffice. All we need is to move the template default parameters from Vector, and then the forward declaration can be used in so many new places: if a header only takes Vector by reference, rvalue reference, pointer, returns any of these, or has them as members then the header doesn't need to see the definition because the declaration will suffice.
+
+        * Scripts/webkit/LegacyMessages-expected.h:
+        * Scripts/webkit/Messages-expected.h:
+        * Shared/API/APIData.h:
+        * UIProcess/API/APIContextMenuClient.h:
+        * UIProcess/API/APIFormClient.h:
+        * UIProcess/API/APIHTTPCookieStore.h:
+        * UIProcess/API/APINotificationProvider.h:
+        * UIProcess/API/glib/WebKitNotificationProvider.h:
+        * UIProcess/Notifications/WebNotificationProvider.h:
+        * UIProcess/TextCheckerCompletion.h:
+        * UIProcess/WebCookieManagerProxy.h:
+        * UIProcess/WebOpenPanelResultListenerProxy.h:
+        * UIProcess/WebPasteboardProxy.h:
+        * UIProcess/WebPopupMenuProxy.h:
+        * UIProcess/WebTextCheckerClient.h:
+        * WebProcess/Cookies/WebCookieManager.h:
+        * WebProcess/Gamepad/WebGamepadProvider.h:
+        * WebProcess/InjectedBundle/API/APIInjectedBundleEditorClient.h:
+        * WebProcess/InjectedBundle/API/APIInjectedBundlePageContextMenuClient.h:
+        * WebProcess/InjectedBundle/API/APIInjectedBundlePageLoaderClient.h:
+        * WebProcess/InjectedBundle/APIInjectedBundleFormClient.h:
+        * WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
+        * WebProcess/Network/webrtc/WebRTCMonitor.h:
+        * WebProcess/Network/webrtc/WebRTCResolver.h:
+        * WebProcess/WebPage/DrawingArea.h:
+        * WebProcess/WebPage/WebOpenPanelResultListener.h:
+
 2017-09-15  Tim Horton  <timothy_horton@apple.com>
 
         Fix the macOS CMake build
index 2a0994c..9c7e31b 100644 (file)
@@ -33,9 +33,9 @@
 #include <WebCore/KeyboardEvent.h>
 #include <WebCore/PluginData.h>
 #include <utility>
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ThreadSafeRefCounted.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace IPC {
index 2a0994c..9c7e31b 100644 (file)
@@ -33,9 +33,9 @@
 #include <WebCore/KeyboardEvent.h>
 #include <WebCore/PluginData.h>
 #include <utility>
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ThreadSafeRefCounted.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace IPC {
index 066fa44..16600cb 100644 (file)
@@ -29,7 +29,6 @@
 #include "APIObject.h"
 #include "DataReference.h"
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace IPC {
 class Decoder;
index 8b2d244..73646bb 100644 (file)
@@ -31,8 +31,8 @@
 #include "WebContextMenuListenerProxy.h"
 #include "WebHitTestResultData.h"
 #include <WebKit/WKBase.h>
+#include <wtf/Forward.h>
 #include <wtf/RefPtr.h>
-#include <wtf/Vector.h>
 
 OBJC_CLASS NSMenu;
 
index 44c005b..777aea2 100644 (file)
@@ -28,7 +28,6 @@
 
 #include "WebFormSubmissionListenerProxy.h"
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebKit {
 class WebFrameProxy;
index 89e8dc5..bb0d1cb 100644 (file)
@@ -28,9 +28,9 @@
 #include "APIObject.h"
 #include "HTTPCookieAcceptPolicy.h"
 #include <WebCore/Cookie.h>
+#include <wtf/Forward.h>
 #include <wtf/Function.h>
 #include <wtf/HashSet.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 class URL;
index 5c8c9bd..9be8e8c 100644 (file)
@@ -27,7 +27,6 @@
 
 #include <wtf/Forward.h>
 #include <wtf/HashMap.h>
-#include <wtf/Vector.h>
 #include <wtf/text/StringHash.h>
 
 namespace WebKit {
index 7367235..9e58bcb 100644 (file)
@@ -21,8 +21,8 @@
 
 #include "WebKitNotification.h"
 #include "WebKitWebContext.h"
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
-#include <wtf/Vector.h>
 #include <wtf/glib/GRefPtr.h>
 #include <wtf/text/StringHash.h>
 
index 95de910..64614dc 100644 (file)
@@ -30,7 +30,6 @@
 #include "APINotificationProvider.h"
 #include "WKNotificationProvider.h"
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace API {
 template<> struct ClientTraits<WKNotificationProviderBase> {
index 0523abe..0eb59be 100644 (file)
@@ -27,7 +27,7 @@
 
 #include "WebPageProxy.h"
 #include <WebCore/TextChecking.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace WebKit {
 
index eae5d24..aaea890 100644 (file)
@@ -31,8 +31,8 @@
 #include "WebContextSupplement.h"
 #include "WebCookieManagerProxyClient.h"
 #include <pal/SessionID.h>
+#include <wtf/Forward.h>
 #include <wtf/RefPtr.h>
-#include <wtf/Vector.h>
 
 #if USE(SOUP)
 #include "SoupCookiePersistentStorageType.h"
index 0449f58..82aa46a 100644 (file)
@@ -28,7 +28,6 @@
 #include "APIObject.h"
 #include <wtf/Forward.h>
 #include <wtf/RefPtr.h>
-#include <wtf/Vector.h>
 
 namespace API {
 class Array;
index 3f5efa9..0955727 100644 (file)
@@ -29,7 +29,6 @@
 #include "SharedMemory.h"
 #include <wtf/Forward.h>
 #include <wtf/HashSet.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 class Color;
index bc16a0f..61dee2e 100644 (file)
@@ -27,8 +27,8 @@
 #define WebPopupMenuProxy_h
 
 #include <WebCore/WritingMode.h>
+#include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
     class IntRect;
index 49275ff..1fd4897 100644 (file)
@@ -30,7 +30,6 @@
 #include "WKTextChecker.h"
 #include <WebCore/TextCheckerClient.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace API {
 template<> struct ClientTraits<WKTextCheckerClientBase> {
index 89ff04a..a5ebd4d 100644 (file)
@@ -36,7 +36,6 @@
 #include <stdint.h>
 #include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
-#include <wtf/Vector.h>
 
 #if USE(SOUP)
 #include "SoupCookiePersistentStorageType.h"
index 513bfff..edab7c3 100644 (file)
@@ -30,6 +30,7 @@
 #include <WebCore/GamepadProvider.h>
 #include <wtf/Forward.h>
 #include <wtf/HashSet.h>
+#include <wtf/Vector.h>
 
 namespace WebKit {
 
index 2606f38..7774f75 100644 (file)
@@ -28,7 +28,6 @@
 #include <WebCore/EditorInsertAction.h>
 #include <WebCore/TextAffinity.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 class CSSStyleDeclaration;
index 11d36d6..fe739b2 100644 (file)
@@ -28,7 +28,6 @@
 #include "SameDocumentNavigationType.h"
 #include <WebCore/LayoutMilestones.h>
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index e4080f7..dcb0cfc 100644 (file)
@@ -30,7 +30,6 @@
 #include "APIInjectedBundlePageLoaderClient.h"
 #include "WKBundlePageLoaderClient.h"
 #include <wtf/Forward.h>
-#include <wtf/Vector.h>
 
 namespace API {
 class Object;
index dc15249..27012e2 100644 (file)
@@ -29,7 +29,7 @@
 
 #include "RTCNetwork.h"
 #include <WebCore/LibWebRTCProvider.h>
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace IPC {
 class Connection;
index 1b82eec..547f01c 100644 (file)
@@ -28,7 +28,7 @@
 #if USE(LIBWEBRTC)
 
 #include "RTCNetwork.h"
-#include <wtf/Vector.h>
+#include <wtf/Forward.h>
 
 namespace IPC {
 class Connection;
index a7da382..88e90e8 100644 (file)
@@ -38,7 +38,6 @@
 #include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
 #include <wtf/TypeCasts.h>
-#include <wtf/Vector.h>
 
 namespace IPC {
 class Connection;
index f47f02c..33d7388 100644 (file)
@@ -28,7 +28,6 @@
 #include <wtf/Forward.h>
 #include <wtf/Ref.h>
 #include <wtf/RefCounted.h>
-#include <wtf/Vector.h>
 
 namespace WebCore {
 class FileChooser;