PlzNavigate: update navigation only if it took place.

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 1772d017f5f785f0f157a9a55a58e61930579d6d..21ff72fc163dfa940fe0225b69ca7ad9334b696a 100644
--- a/third_party/WebKit/Source/web/WebDataSourceImpl.cpp
+++ b/third_party/WebKit/Source/web/WebDataSourceImpl.cpp
@@ -77,10 +77,12 @@ void WebDataSourceImpl::appendRedirect(const WebURL& url)
 
 void WebDataSourceImpl::updateNavigation(double redirectStartTime, double redirectEndTime, double fetchStartTime, const WebVector<WebURL>& redirectChain)
 {
-    for (size_t i = 0; i + 1 < redirectChain.size(); ++i)
-        didRedirect(redirectChain[i], redirectChain[i + 1]);
-    timing().setRedirectStart(redirectStartTime);
-    timing().setRedirectEnd(redirectEndTime);
+    if (redirectChain.size() >= 2) {

[clamy, 2016/09/23 15:35:38]

This does not seem related to the other issue and the CL description. Please put
it in another CL if this relates to another issue, or remove it if it is no
longer needed.

[arthursonzogni, 2016/09/26 08:30:32]

There are related because they cause the same assertion violation.
Without it, only one of TracingBrowserTest still pass and the renderer crash.

Nevertheless, I think it may be better to do two different patchs.

+        for (size_t i = 0; i + 1 < redirectChain.size(); ++i)
+            didRedirect(redirectChain[i], redirectChain[i + 1]);
+        timing().setRedirectStart(redirectStartTime);
+        timing().setRedirectEnd(redirectEndTime);
+    }
     timing().setFetchStart(fetchStartTime);
 }