Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(180)

Unified Diff: third_party/WebKit/Source/web/WebDataSourceImpl.cpp

Issue 2653953005: PlzNavigate: transmit redirect info to the renderer side (Closed)
Patch Set: Rebase + addressed comments Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/web/WebDataSourceImpl.h ('k') | third_party/WebKit/public/web/WebDataSource.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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]);
+ if (hasRedirect) {
timing().setRedirectStart(redirectStartTime);
timing().setRedirectEnd(redirectEndTime);
}
« no previous file with comments | « third_party/WebKit/Source/web/WebDataSourceImpl.h ('k') | third_party/WebKit/public/web/WebDataSource.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698