PlzNavigate: transmit redirect info to the renderer side

third_party/WebKit/Source/web/WebDataSourceImpl.cpp

Index: third_party/WebKit/Source/web/WebDataSourceImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebDataSourceImpl.cpp b/third_party/WebKit/Source/web/WebDataSourceImpl.cpp
index 10f872153156450b9b7a9fc1e1fb0bd04a815644..0934e36da9c6152bde203cbf5f60b3d4fe9129a0 100644
--- a/third_party/WebKit/Source/web/WebDataSourceImpl.cpp
+++ b/third_party/WebKit/Source/web/WebDataSourceImpl.cpp
@@ -74,16 +74,13 @@ void WebDataSourceImpl::appendRedirect(const WebURL& url) {
   DocumentLoader::appendRedirect(url);
 }
 
-void WebDataSourceImpl::updateNavigation(
-    double redirectStartTime,
-    double redirectEndTime,
-    double fetchStartTime,
-    const WebVector<WebURL>& redirectChain) {
+void WebDataSourceImpl::updateNavigation(double redirectStartTime,
+                                         double redirectEndTime,
+                                         double fetchStartTime,
+                                         bool hasRedirect) {
   // Updates the redirection timing if there is at least one redirection
   // (between two URLs).
-  if (redirectChain.size() >= 2) {
-    for (size_t i = 0; i + 1 < redirectChain.size(); ++i)
-      didRedirect(redirectChain[i], redirectChain[i + 1]);

[Nate Chapin, 2017/02/17 18:25:28]

I think didRedirect now just has a single caller, in
DocumentLoader::redirectReceived. Could we merge it into redirectReceived?

[clamy, 2017/02/20 15:42:01]

Done.

+  if (hasRedirect) {
     timing().setRedirectStart(redirectStartTime);
     timing().setRedirectEnd(redirectEndTime);
   }