<https://webkit.org/b/131136> Remove FrameLoadRequest's m_lockHistory member since...
authormrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Apr 2014 00:56:09 +0000 (00:56 +0000)
committermrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Apr 2014 00:56:09 +0000 (00:56 +0000)
Reviewed by Andreas Kling.

* loader/FrameLoadRequest.cpp:
(WebCore::FrameLoadRequest::FrameLoadRequest):
* loader/FrameLoadRequest.h:
(WebCore::FrameLoadRequest::FrameLoadRequest):
(WebCore::FrameLoadRequest::lockHistory): Deleted.
(WebCore::FrameLoadRequest::setLockHistory): Deleted.
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::load): Remove an if whose body was never executed.

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

Source/WebCore/ChangeLog
Source/WebCore/loader/FrameLoadRequest.cpp
Source/WebCore/loader/FrameLoadRequest.h
Source/WebCore/loader/FrameLoader.cpp

index 276fde1..587a239 100644 (file)
@@ -1,5 +1,20 @@
 2014-04-02  Mark Rowe  <mrowe@apple.com>
 
+        <https://webkit.org/b/131136> Remove FrameLoadRequest's m_lockHistory member since it's always false.
+
+        Reviewed by Andreas Kling.
+
+        * loader/FrameLoadRequest.cpp:
+        (WebCore::FrameLoadRequest::FrameLoadRequest):
+        * loader/FrameLoadRequest.h:
+        (WebCore::FrameLoadRequest::FrameLoadRequest):
+        (WebCore::FrameLoadRequest::lockHistory): Deleted.
+        (WebCore::FrameLoadRequest::setLockHistory): Deleted.
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::load): Remove an if whose body was never executed.
+
+2014-04-02  Mark Rowe  <mrowe@apple.com>
+
         <https://webkit.org/b/131135> Introduce LockHistory and LockBackForwardList enums to use in place of bools.
 
         These arguments are often passed using literals at the call site, where the use of bools severely hinders
index 0146788..6739cff 100644 (file)
@@ -39,7 +39,6 @@ namespace WebCore {
 FrameLoadRequest::FrameLoadRequest(Frame* frame, const ResourceRequest& resourceRequest, const SubstituteData& substituteData)
     : m_requester(frame->document()->securityOrigin())
     , m_resourceRequest(resourceRequest)
-    , m_lockHistory(false)
     , m_shouldCheckNewWindowPolicy(false)
     , m_substituteData(substituteData)
 {
index 398f33f..1a2bbf8 100644 (file)
@@ -37,7 +37,6 @@ struct FrameLoadRequest {
 public:
     explicit FrameLoadRequest(SecurityOrigin* requester)
         : m_requester(requester)
-        , m_lockHistory(false)
         , m_shouldCheckNewWindowPolicy(false)
     {
     }
@@ -45,7 +44,6 @@ public:
     FrameLoadRequest(SecurityOrigin* requester, const ResourceRequest& resourceRequest)
         : m_requester(requester)
         , m_resourceRequest(resourceRequest)
-        , m_lockHistory(false)
         , m_shouldCheckNewWindowPolicy(false)
     {
     }
@@ -54,7 +52,6 @@ public:
         : m_requester(requester)
         , m_resourceRequest(resourceRequest)
         , m_frameName(frameName)
-        , m_lockHistory(false)
         , m_shouldCheckNewWindowPolicy(false)
     {
     }
@@ -71,9 +68,6 @@ public:
     const String& frameName() const { return m_frameName; }
     void setFrameName(const String& frameName) { m_frameName = frameName; }
 
-    void setLockHistory(bool lockHistory) { m_lockHistory = lockHistory; }
-    bool lockHistory() const { return m_lockHistory; }
-
     void setShouldCheckNewWindowPolicy(bool checkPolicy) { m_shouldCheckNewWindowPolicy = checkPolicy; }
     bool shouldCheckNewWindowPolicy() const { return m_shouldCheckNewWindowPolicy; }
 
@@ -85,7 +79,6 @@ private:
     RefPtr<SecurityOrigin> m_requester;
     ResourceRequest m_resourceRequest;
     String m_frameName;
-    bool m_lockHistory;
     bool m_shouldCheckNewWindowPolicy;
     SubstituteData m_substituteData;
 };
index acef730..36d65a4 100644 (file)
@@ -1357,8 +1357,6 @@ void FrameLoader::load(const FrameLoadRequest& passedRequest)
         request.setSubstituteData(defaultSubstituteDataForURL(request.resourceRequest().url()));
 
     RefPtr<DocumentLoader> loader = m_client.createDocumentLoader(request.resourceRequest(), request.substituteData());
-    if (request.lockHistory() && m_documentLoader)
-        loader->setClientRedirectSourceForHistory(m_documentLoader->didCreateGlobalHistoryEntry() ? m_documentLoader->urlForHistory().string() : m_documentLoader->clientRedirectSourceForHistory());
     load(loader.get());
 }