Remove header OptionSet.h from FrameLoaderTypes.h
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Jun 2017 00:50:53 +0000 (00:50 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Jun 2017 00:50:53 +0000 (00:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=173489

Reviewed by Joseph Pecoraro.

Although the header FrameLoaderTypes.h defines exactly one enum class, ReloadOption, whose
enumerators conform the power of two prerequisite for parameterizing an OptionSet for it
FrameLoaderTypes.h does not actually make use of the functionality provided by header
OptionSet.h. And not all source files that include FrameLoaderTypes.h need to use an OptionSet.

* loader/DocumentLoader.h: Include <wtf/OptionSet.h>.
* loader/FrameLoader.h: Ditto.
* loader/FrameLoaderTypes.h: Remove header <wtf/OptionSet.h>.
* replay/UserInputBridge.h: Include <wtf/Forward.h>.

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

Source/WebCore/ChangeLog
Source/WebCore/loader/DocumentLoader.h
Source/WebCore/loader/FrameLoader.h
Source/WebCore/loader/FrameLoaderTypes.h
Source/WebCore/replay/UserInputBridge.h

index a608b98..e6216b9 100644 (file)
@@ -1,3 +1,20 @@
+2017-06-16  Daniel Bates  <dabates@apple.com>
+
+        Remove header OptionSet.h from FrameLoaderTypes.h
+        https://bugs.webkit.org/show_bug.cgi?id=173489
+
+        Reviewed by Joseph Pecoraro.
+
+        Although the header FrameLoaderTypes.h defines exactly one enum class, ReloadOption, whose
+        enumerators conform the power of two prerequisite for parameterizing an OptionSet for it
+        FrameLoaderTypes.h does not actually make use of the functionality provided by header
+        OptionSet.h. And not all source files that include FrameLoaderTypes.h need to use an OptionSet.
+
+        * loader/DocumentLoader.h: Include <wtf/OptionSet.h>.
+        * loader/FrameLoader.h: Ditto.
+        * loader/FrameLoaderTypes.h: Remove header <wtf/OptionSet.h>.
+        * replay/UserInputBridge.h: Include <wtf/Forward.h>.
+
 2017-06-16  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         [iOS DnD] Upstream iOS drag and drop implementation into OpenSource WebKit
index 2e6c7b5..1ea2023 100644 (file)
@@ -45,6 +45,7 @@
 #include "SubstituteData.h"
 #include "Timer.h"
 #include <wtf/HashSet.h>
+#include <wtf/OptionSet.h>
 #include <wtf/RefPtr.h>
 #include <wtf/Vector.h>
 
index fd4b264..c6190db 100644 (file)
@@ -44,6 +44,7 @@
 #include "Timer.h"
 #include <wtf/Forward.h>
 #include <wtf/HashSet.h>
+#include <wtf/OptionSet.h>
 #include <wtf/Optional.h>
 
 namespace WebCore {
index cbd9700..84ea25b 100644 (file)
@@ -28,8 +28,6 @@
 
 #pragma once
 
-#include <wtf/OptionSet.h>
-
 namespace WebCore {
 
 enum FrameState {
index 1654772..d17b886 100644 (file)
@@ -29,6 +29,7 @@
 
 #include "FrameLoaderTypes.h"
 #include "ScrollTypes.h"
+#include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
 
 namespace JSC {