Change FrameLoadRequest from a struct to a class
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Jun 2017 00:27:51 +0000 (00:27 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Jun 2017 00:27:51 +0000 (00:27 +0000)
FrameLoadRequest is underutilizing the purpose of a struct - default visibility of
members is public, as FrameLoadRequest explicitly groups its members under public:
or private: sections. Maybe in the future we can make FrameLoadRequest a struct
with only public members. For now, we should consider FrameLoadRequest a class.

* loader/FormSubmission.h:
* loader/FrameLoadRequest.h:
* loader/FrameLoader.h:
* page/Chrome.h:
* page/ChromeClient.h:
* replay/UserInputBridge.h:

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

Source/WebCore/ChangeLog
Source/WebCore/loader/FormSubmission.h
Source/WebCore/loader/FrameLoadRequest.h
Source/WebCore/loader/FrameLoader.h
Source/WebCore/page/Chrome.h
Source/WebCore/page/ChromeClient.h
Source/WebCore/replay/UserInputBridge.h

index a5f4ff5..2979384 100644 (file)
@@ -1,3 +1,19 @@
+2017-06-21  Daniel Bates  <dabates@apple.com>
+
+        Change FrameLoadRequest from a struct to a class
+
+        FrameLoadRequest is underutilizing the purpose of a struct - default visibility of
+        members is public, as FrameLoadRequest explicitly groups its members under public:
+        or private: sections. Maybe in the future we can make FrameLoadRequest a struct
+        with only public members. For now, we should consider FrameLoadRequest a class.
+
+        * loader/FormSubmission.h:
+        * loader/FrameLoadRequest.h:
+        * loader/FrameLoader.h:
+        * page/Chrome.h:
+        * page/ChromeClient.h:
+        * replay/UserInputBridge.h:
+
 2017-06-21  Daewoong Jang  <daewoong.jang@navercorp.com>
 
         Compiler error while building with !HAVE(ACCESSIBILITY)
index 252412c..867e63f 100644 (file)
@@ -38,8 +38,7 @@ namespace WebCore {
 
 class Event;
 class FormData;
-
-struct FrameLoadRequest;
+class FrameLoadRequest;
 
 class FormSubmission : public RefCounted<FormSubmission> {
 public:
index bdb73e8..fbfc201 100644 (file)
 #include "SubstituteData.h"
 
 namespace WebCore {
+
 class Frame;
 
-struct FrameLoadRequest {
+class FrameLoadRequest {
 public:
     FrameLoadRequest(SecurityOrigin& requester, const ResourceRequest& resourceRequest, const String& frameName, LockHistory lockHistory, LockBackForwardList lockBackForwardList, ShouldSendReferrer shouldSendReferrer, AllowNavigationToInvalidURL allowNavigationToInvalidURL, NewFrameOpenerPolicy newFrameOpenerPolicy, ShouldOpenExternalURLsPolicy shouldOpenExternalURLsPolicy, ShouldReplaceDocumentIfJavaScriptURL shouldReplaceDocumentIfJavaScriptURL = ReplaceDocumentIfJavaScriptURL, const AtomicString& downloadAttribute = { })
         : m_requester { makeRef(requester) }
index c6190db..1d02029 100644 (file)
@@ -61,6 +61,7 @@ class DocumentLoader;
 class Event;
 class FormState;
 class FormSubmission;
+class FrameLoadRequest;
 class FrameLoaderClient;
 class FrameNetworkingContext;
 class HistoryController;
@@ -78,7 +79,6 @@ class SharedBuffer;
 class SubframeLoader;
 class SubstituteData;
 
-struct FrameLoadRequest;
 struct WindowFeatures;
 
 WEBCORE_EXPORT bool isBackForwardLoadType(FrameLoadType);
index 8ebdb5d..9d763b9 100644 (file)
@@ -41,6 +41,7 @@ class DateTimeChooserClient;
 class FileChooser;
 class FileIconLoader;
 class FloatRect;
+class FrameLoadRequest;
 class Element;
 class Frame;
 class Geolocation;
@@ -55,7 +56,6 @@ class PopupOpeningObserver;
 class SearchPopupMenu;
 
 struct DateTimeChooserParameters;
-struct FrameLoadRequest;
 struct ViewportArguments;
 struct WindowFeatures;
     
index 839ae39..83090ae 100644 (file)
@@ -74,6 +74,7 @@ class FileChooser;
 class FileIconLoader;
 class FloatRect;
 class Frame;
+class FrameLoadRequest;
 class Geolocation;
 class GraphicsLayer;
 class GraphicsLayerFactory;
@@ -96,7 +97,6 @@ class MediaPlayerRequestInstallMissingPluginsCallback;
 #endif
 
 struct DateTimeChooserParameters;
-struct FrameLoadRequest;
 struct GraphicsDeviceAdapter;
 struct ViewportArguments;
 struct WindowFeatures;
index d17b886..26f4a36 100644 (file)
@@ -38,9 +38,8 @@ class InputCursor;
 
 namespace WebCore {
 
-struct FrameLoadRequest;
-
 class Frame;
+class FrameLoadRequest;
 class Page;
 class PlatformKeyboardEvent;
 class PlatformMouseEvent;