location.reload() after POST should re-POST

Source/core/loader/NavigationScheduler.cpp

Index: Source/core/loader/NavigationScheduler.cpp
diff --git a/Source/core/loader/NavigationScheduler.cpp b/Source/core/loader/NavigationScheduler.cpp
index 25db42fcde25b56361cd136c79ccc91839eace80..52f988414e1ec3e89f058961d47b99d04be40a59 100644
--- a/Source/core/loader/NavigationScheduler.cpp
+++ b/Source/core/loader/NavigationScheduler.cpp
@@ -157,20 +157,17 @@ public:
         : ScheduledURLNavigation(0.0, originDocument, url, referrer, lockBackForwardList, true) { }
 };
 
-class ScheduledRefresh FINAL : public ScheduledURLNavigation {
+class ScheduledRefresh FINAL : public ScheduledNavigation {
 public:
-    ScheduledRefresh(Document* originDocument, const String& url, const Referrer& referrer)
-        : ScheduledURLNavigation(0.0, originDocument, url, referrer, true, true)
+    ScheduledRefresh()
+        : ScheduledNavigation(0.0, true, true)
     {
     }
 
     virtual void fire(LocalFrame* frame) OVERRIDE
     {
         OwnPtr<UserGestureIndicator> gestureIndicator = createUserGestureIndicator();
-        FrameLoadRequest request(originDocument(), ResourceRequest(KURL(ParsedURLString, url()), referrer(), ReloadIgnoringCacheData), "_self");
-        request.setLockBackForwardList(lockBackForwardList());
-        request.setClientRedirect(ClientRedirect);
-        frame->loader().load(request);
+        frame->loader().reload(NormalReload, KURL(), nullAtom, ClientRedirect);

[Nate Chapin, 2014/07/28 20:05:04]

This makes location.reload() behave more like an internal "trusted" reload.
However, it can't override the url or encoding, it retains its user gesture
indicator status, and it will use the current document's referrer. So not much
observable has changed (except inheriting the http method).

     }
 };
 
@@ -343,11 +340,9 @@ void NavigationScheduler::scheduleRefresh()
 {
     if (!shouldScheduleNavigation())
         return;
-    const KURL& url = m_frame->document()->url();
-    if (url.isEmpty())
+    if (m_frame->document()->url().isEmpty())
         return;
-
-    schedule(adoptPtr(new ScheduledRefresh(m_frame->document(), url.string(), Referrer(m_frame->document()->outgoingReferrer(), m_frame->document()->referrerPolicy()))));
+    schedule(adoptPtr(new ScheduledRefresh));
 }
 
 void NavigationScheduler::scheduleHistoryNavigation(int steps)