[webkit-changes] [185007] trunk/Source/WebCore

Fri, 29 May 2015 15:01:52 -0700

Title: [185007] trunk/Source/WebCore
Revision
185007
Author
[email protected]
Date
2015-05-29 15:01:21 -0700 (Fri, 29 May 2015)

Log Message

NavigationAction constructor cleanup.
https://bugs.webkit.org/show_bug.cgi?id=145472

Reviewed by Alex Christensen.

No new tests (Refactor, no change in behavior).

* loader/NavigationAction.cpp:
(WebCore::NavigationAction::NavigationAction): Delegate many of the various constructors
  to a single full-featured constructor.
* loader/NavigationAction.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (185006 => 185007)


--- trunk/Source/WebCore/ChangeLog	2015-05-29 21:53:30 UTC (rev 185006)
+++ trunk/Source/WebCore/ChangeLog	2015-05-29 22:01:21 UTC (rev 185007)
@@ -1,3 +1,17 @@
+2015-05-29  Brady Eidson  <[email protected]>
+
+        NavigationAction constructor cleanup.
+        https://bugs.webkit.org/show_bug.cgi?id=145472
+
+        Reviewed by Alex Christensen.
+
+        No new tests (Refactor, no change in behavior).
+
+        * loader/NavigationAction.cpp:
+        (WebCore::NavigationAction::NavigationAction): Delegate many of the various constructors
+          to a single full-featured constructor.
+        * loader/NavigationAction.h:
+
 2015-05-29  Matt Rajca  <[email protected]>
 
         Rename MediaSession and MediaSessionClient to PlatformMediaSession and PlatformMediaSessionClient so we can use MediaSession for the formal implementation of the Media Session spec.

Modified: trunk/Source/WebCore/loader/NavigationAction.cpp (185006 => 185007)


--- trunk/Source/WebCore/loader/NavigationAction.cpp	2015-05-29 21:53:30 UTC (rev 185006)
+++ trunk/Source/WebCore/loader/NavigationAction.cpp	2015-05-29 22:01:21 UTC (rev 185007)
@@ -48,55 +48,47 @@
     return NavigationType::Other;
 }
 
-NavigationAction::NavigationAction()
-    : m_type(NavigationType::Other)
+NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, NavigationType type, PassRefPtr<Event> event, ShouldOpenExternalURLsPolicy shouldOpenExternalURLsPolicy)
+    : m_resourceRequest(resourceRequest)
+    , m_type(type)
+    , m_event(event)
     , m_processingUserGesture(ScriptController::processingUserGesture())
+    , m_shouldOpenExternalURLsPolicy(shouldOpenExternalURLsPolicy)
 {
 }
 
+NavigationAction::NavigationAction()
+    : NavigationAction(ResourceRequest(), NavigationType::Other, nullptr, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
+{
+}
+
 NavigationAction::NavigationAction(const ResourceRequest& resourceRequest)
-    : m_resourceRequest(resourceRequest)
-    , m_type(NavigationType::Other)
-    , m_processingUserGesture(ScriptController::processingUserGesture())
+    : NavigationAction(resourceRequest, NavigationType::Other, nullptr, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
 {
 }
 
 NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, NavigationType type)
-    : m_resourceRequest(resourceRequest)
-    , m_type(type)
-    , m_processingUserGesture(ScriptController::processingUserGesture())
+    : NavigationAction(resourceRequest, type, nullptr, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
 {
 }
 
-NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, FrameLoadType frameLoadType,
-        bool isFormSubmission)
-    : m_resourceRequest(resourceRequest)
-    , m_type(navigationType(frameLoadType, isFormSubmission, 0))
-    , m_processingUserGesture(ScriptController::processingUserGesture())
+NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, FrameLoadType frameLoadType, bool isFormSubmission)
+    : NavigationAction(resourceRequest, navigationType(frameLoadType, isFormSubmission, 0), nullptr, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
 {
 }
 
 NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, NavigationType type, PassRefPtr<Event> event)
-    : m_resourceRequest(resourceRequest)
-    , m_type(type)
-    , m_event(event)
-    , m_processingUserGesture(ScriptController::processingUserGesture())
+    : NavigationAction(resourceRequest, type, event, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
 {
 }
 
 NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, NavigationType type, ShouldOpenExternalURLsPolicy shouldOpenExternalURLsPolicy)
-    : m_resourceRequest(resourceRequest)
-    , m_type(type)
-    , m_shouldOpenExternalURLsPolicy(shouldOpenExternalURLsPolicy)
+    : NavigationAction(resourceRequest, type, nullptr, shouldOpenExternalURLsPolicy)
 {
 }
 
-NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, FrameLoadType frameLoadType,
-        bool isFormSubmission, PassRefPtr<Event> event)
-    : m_resourceRequest(resourceRequest)
-    , m_type(navigationType(frameLoadType, isFormSubmission, event))
-    , m_event(event)
-    , m_processingUserGesture(ScriptController::processingUserGesture())
+NavigationAction::NavigationAction(const ResourceRequest& resourceRequest, FrameLoadType frameLoadType, bool isFormSubmission, PassRefPtr<Event> event)
+    : NavigationAction(resourceRequest, navigationType(frameLoadType, isFormSubmission, event), event, ShouldOpenExternalURLsPolicy::ShouldNotAllow)
 {
 }

Modified: trunk/Source/WebCore/loader/NavigationAction.h (185006 => 185007)


--- trunk/Source/WebCore/loader/NavigationAction.h	2015-05-29 21:53:30 UTC (rev 185006)
+++ trunk/Source/WebCore/loader/NavigationAction.h	2015-05-29 22:01:21 UTC (rev 185007)
@@ -44,6 +44,7 @@
     WEBCORE_EXPORT NavigationAction(const ResourceRequest&, NavigationType);
     WEBCORE_EXPORT NavigationAction(const ResourceRequest&, FrameLoadType, bool isFormSubmission);
     NavigationAction(const ResourceRequest&, NavigationType, PassRefPtr<Event>);
+    NavigationAction(const ResourceRequest&, NavigationType, PassRefPtr<Event>, ShouldOpenExternalURLsPolicy);
     NavigationAction(const ResourceRequest&, NavigationType, ShouldOpenExternalURLsPolicy);
     NavigationAction(const ResourceRequest&, FrameLoadType, bool isFormSubmission, PassRefPtr<Event>);
 
@@ -65,7 +66,7 @@
     NavigationType m_type;
     RefPtr<Event> m_event;
     bool m_processingUserGesture;
-    ShouldOpenExternalURLsPolicy m_shouldOpenExternalURLsPolicy { ShouldOpenExternalURLsPolicy::ShouldNotAllow };
+    ShouldOpenExternalURLsPolicy m_shouldOpenExternalURLsPolicy;
 };
 
 }

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to