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

Unified Diff: content/browser/frame_host/navigator_impl.cc

Issue 2368183004: Move redirect_chain from NavigationEntry to FrameNavigationEntry. (Closed)
Patch Set: Nit Created 4 years, 2 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
Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index 2413c24c86b35b2976ad5cda6e61e2ada6ac7c98..dcc3d2a3711b39cf8ef94047179533cd702be09a 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -821,8 +821,9 @@ void NavigatorImpl::RequestTransferURL(
}
entry->AddOrUpdateFrameEntry(
node, -1, -1, nullptr,
- static_cast<SiteInstanceImpl*>(source_site_instance), dest_url,
- referrer_to_use, PageState(), method, -1);
+ static_cast<SiteInstanceImpl*>(source_site_instance),
+ dest_url, referrer_to_use, redirect_chain, PageState(), method,
+ -1);
} else {
// Main frame case.
entry = NavigationEntryImpl::FromNavigationEntry(
@@ -831,9 +832,9 @@ void NavigatorImpl::RequestTransferURL(
std::string(), controller_->GetBrowserContext()));
entry->root_node()->frame_entry->set_source_site_instance(
static_cast<SiteInstanceImpl*>(source_site_instance));
+ entry->SetRedirectChain(redirect_chain);
}
- entry->SetRedirectChain(redirect_chain);
// Don't allow an entry replacement if there is no entry to replace.
// http://crbug.com/457149
if (should_replace_current_entry && controller_->GetEntryCount() > 0)

Powered by Google App Engine
This is Rietveld 408576698