Don't stop the documentLoader on navigations.

Source/core/loader/FrameLoader.h

Index: Source/core/loader/FrameLoader.h
diff --git a/Source/core/loader/FrameLoader.h b/Source/core/loader/FrameLoader.h
index edbb861e1dd17adad700279148c12c127db6dc6c..aebab1fcda168f364a84dccce43de941c066df5c 100644
--- a/Source/core/loader/FrameLoader.h
+++ b/Source/core/loader/FrameLoader.h
@@ -184,6 +184,8 @@ public:
 
     void started();
 
+    void onResponseReceived();

[Nate Chapin, 2014/03/10 16:12:44]

Nit: responseReceived(), to match the name on DocumentLoader.

+
     bool allowPlugins(ReasonForCallingAllowPlugins);
 
     void updateForSameDocumentNavigation(const KURL&, SameDocumentNavigationSource, PassRefPtr<SerializedScriptValue>, UpdateBackForwardListPolicy);
@@ -233,6 +235,7 @@ private:
 
     void scheduleCheckCompleted();
     void startCheckCompleteTimer();
+    void checkCurrentDocumentLoaderNeedsStop();
 
     LocalFrame* m_frame;
     FrameLoaderClient* m_client;
@@ -265,6 +268,7 @@ private:
 
     // FIXME: This is only used in checkCompleted(). Figure out a way to disentangle it.
     bool m_isComplete;
+    bool m_currentLoaderNeedsStop;

[Nate Chapin, 2014/03/10 16:12:44]

currentLoader isn't a particularly clear name to me. I'd favor either changing
its name to match one of the DocumentLoaders, or a comment explaining what
exactly it means.

Based on how it's used, it appears to be true when a provisional loader receives
a response, and a committed loader is in progress?

 
     Timer<FrameLoader> m_checkTimer;
     bool m_shouldCallCheckCompleted;