Fix for the RedirectTest.ClientServerServer test.

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 800e38378e8e596430318731ed891abd4ee6ce16..754b4ab583d419827cc97d8ab8429faaeee757d9 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
@@ -1675,8 +1675,10 @@ void FrameLoader::startLoad(FrameLoadRequest& frameLoadRequest,
     client()->dispatchWillSubmitForm(frameLoadRequest.form());
 
   m_progressTracker->progressStarted();
+  // The document URL needs to be added to the head of the list as that is
+  // where the redirects originated.
   if (m_provisionalDocumentLoader->isClientRedirect())
-    m_provisionalDocumentLoader->appendRedirect(m_frame->document()->url());
+    m_provisionalDocumentLoader->prependRedirect(m_frame->document()->url());

[nasko, 2016/10/31 18:45:50]

Why is there another URL in the redirect chain? The m_provisionalDocumentLoader
is constructed on 1658 and the m_redirectChain should be empty at this point,
which is why just calling appendRedirect should've just worked.

[ananta, 2016/10/31 19:11:33]

The FrameLoaderClientImpl::createDocumentLoader function internally calls
RenderFrameImpl::didCreateDataSource which adds redirects to the list. It is not
empty as a result.

https://cs.chromium.org/chromium/src/content/renderer/render_frame_impl.cc?q=...

   m_provisionalDocumentLoader->appendRedirect(
       m_provisionalDocumentLoader->request().url());
   double triggeringEventTime =