Don't stop the documentLoader on navigations.

Source/core/loader/FrameLoader.h

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) Research In Motion Limited 2009. All rights reserved.
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
@@ skipping 166 matching lines @@
     void applyUserAgent(ResourceRequest&);
 
     bool shouldInterruptLoadForXFrameOptions(const String&, const KURL&, unsigned long requestIdentifier);
 
     bool allAncestorsAreComplete() const; // including this
 
     bool shouldClose();
 
     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);
 
     HistoryItem* currentItem() const { return m_currentItem.get(); }
     void saveDocumentAndScrollState();
     void clearScrollPositionAndViewState();
 
     enum RestorePolicy {
         StandardRestore,
@@ skipping 29 matching lines @@
     void detachChildren();
     void closeAndRemoveChild(LocalFrame*);
     void detachClient();
 
     void setHistoryItemStateForCommit(HistoryCommitType, bool isPushOrReplaceState = false, PassRefPtr<SerializedScriptValue> = nullptr);
 
     void loadInSameDocument(const KURL&, PassRefPtr<SerializedScriptValue> stateObject, UpdateBackForwardListPolicy, ClientRedirectPolicy);
 
     void scheduleCheckCompleted();
     void startCheckCompleteTimer();
+    void checkCurrentDocumentLoaderNeedsStop();
 
     LocalFrame* m_frame;
     FrameLoaderClient* m_client;
 
     // FIXME: These should be OwnPtr<T> to reduce build times and simplify
     // header dependencies unless performance testing proves otherwise.
     // Some of these could be lazily created for memory savings on devices.
     mutable FrameLoaderStateMachine m_stateMachine;
     mutable MixedContentChecker m_mixedContentChecker;
 
@@ skipping 12 matching lines @@
     RefPtr<DocumentLoader> m_policyDocumentLoader;
     OwnPtr<FetchContext> m_fetchContext;
 
     RefPtr<HistoryItem> m_currentItem;
     RefPtr<HistoryItem> m_provisionalItem;
 
     bool m_inStopAllLoaders;
 
     // 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;
 
     bool m_didAccessInitialDocument;
     Timer<FrameLoader> m_didAccessInitialDocumentTimer;
 
     SandboxFlags m_forcedSandboxFlags;
 };
 
 } // namespace WebCore
 
 #endif // FrameLoader_h