Clear provisional item on all server redirects.

third_party/WebKit/Source/core/loader/FrameLoader.cpp

Index: third_party/WebKit/Source/core/loader/FrameLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameLoader.cpp b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
index 832f73f5713e9abbf8b9749c400fdca055fd8399..4f18715f3c67995b26b2447aef9369419684d7cd 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
@@ -358,8 +358,8 @@ void FrameLoader::replaceDocumentWhileExecutingJavaScriptURL(const String& sourc
 void FrameLoader::receivedMainResourceRedirect(const KURL& newURL)
 {
     client()->dispatchDidReceiveServerRedirectForProvisionalLoad();
-    // If a back/forward navigation redirects cross-origin, don't reuse any state from the HistoryItem.
-    if (m_provisionalItem && !SecurityOrigin::create(m_provisionalItem->url())->isSameSchemeHostPort(SecurityOrigin::create(newURL).get()))

[Charlie Reis, 2016/07/20 17:38:23]

Nate, why was this restricted to cross-origin redirects?  Even same-origin
redirects wouldn't want to reuse things from the previous HistoryItem, like
scroll position, form state, DSN (affecting whether it's in-page or not), etc.

+    // If a back/forward navigation redirects, don't reuse any state from the HistoryItem.
+    if (m_provisionalItem)

[Nate Chapin, 2016/07/20 17:42:27]

Nit: the if() isn't really necessary here.

[Charlie Reis, 2016/07/20 18:03:22]

Done.

         m_provisionalItem.clear();
 }