Refactor FrameLoader loading interface

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 2ac74b8ed38c84b0d4aca49c8d1f9eb81f40c468..e29776ee0e191871ee7b8532a4faf7993c44e857 100644
--- a/Source/core/loader/NavigationScheduler.cpp
+++ b/Source/core/loader/NavigationScheduler.cpp
@@ -177,7 +177,13 @@ public:
     virtual void fire(LocalFrame* frame) override
     {
         OwnPtr<UserGestureIndicator> gestureIndicator = createUserGestureIndicator();
-        frame->loader().reload(NormalReload, KURL(), ClientRedirect);
+        if (!frame->loader().currentItem())

[Nate Chapin, 2015/06/02 17:19:35]

Null check the ResourceRequest instead of the current item here? It looks
mysterious if you don't know that reloads are initialized based on data from the
current history item.

[clamy, 2015/06/03 14:39:14]

Done.

+            return;
+        FrameLoadRequest request = FrameLoader::frameRequestForReload(
+            frame->loader().resourceRequestForReload(
+                *frame, FrameLoadTypeReload, KURL(), ClientRedirect),
+            ClientRedirect);
+        frame->loader().load(request, FrameLoadTypeReload);
     }
 
 private: