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

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

Issue 2368183004: Move redirect_chain from NavigationEntry to FrameNavigationEntry. (Closed)
Patch Set: Addressed comments. Created 4 years, 3 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/navigation_entry_impl.cc
diff --git a/content/browser/frame_host/navigation_entry_impl.cc b/content/browser/frame_host/navigation_entry_impl.cc
index fbf56542c1609eae1051a8926899c51de2ef4d93..5c8393836b884e129d99b54646585f8124509b60 100644
--- a/content/browser/frame_host/navigation_entry_impl.cc
+++ b/content/browser/frame_host/navigation_entry_impl.cc
@@ -548,11 +548,11 @@ int NavigationEntryImpl::GetHttpStatusCode() const {
void NavigationEntryImpl::SetRedirectChain(
const std::vector<GURL>& redirect_chain) {
- redirect_chain_ = redirect_chain;
+ root_node()->frame_entry->set_redirect_chain(redirect_chain);
}
const std::vector<GURL>& NavigationEntryImpl::GetRedirectChain() const {
- return redirect_chain_;
+ return root_node()->frame_entry->redirect_chain();
}
bool NavigationEntryImpl::IsRestored() const {
@@ -632,7 +632,6 @@ std::unique_ptr<NavigationEntryImpl> NavigationEntryImpl::CloneAndReplace(
copy->cached_display_title_ = cached_display_title_;
// ResetForCommit: transferred_global_request_id_
// ResetForCommit: should_replace_entry_
- copy->redirect_chain_ = redirect_chain_;
// ResetForCommit: should_clear_history_list_
// ResetForCommit: frame_tree_node_id_
// ResetForCommit: intent_received_timestamp_
@@ -701,7 +700,7 @@ RequestNavigationParams NavigationEntryImpl::ConstructRequestNavigationParams(
// completed navigation (whose previous redirects don't apply).
std::vector<GURL> redirects;
if (ui::PageTransitionIsNewNavigation(GetTransitionType())) {
- redirects = GetRedirectChain();
+ redirects = frame_entry.redirect_chain();
}
int pending_offset_to_send = pending_history_list_offset;
@@ -772,6 +771,7 @@ void NavigationEntryImpl::AddOrUpdateFrameEntry(
scoped_refptr<SiteInstanceImpl> source_site_instance,
const GURL& url,
const Referrer& referrer,
+ const std::vector<GURL>& redirect_chain,
const PageState& page_state,
const std::string& method,
int64_t post_id) {
@@ -795,8 +795,8 @@ void NavigationEntryImpl::AddOrUpdateFrameEntry(
root_node()->frame_entry->UpdateEntry(
frame_tree_node->unique_name(), item_sequence_number,
document_sequence_number, site_instance,
- std::move(source_site_instance), url, referrer, page_state, method,
- post_id);
+ std::move(source_site_instance), url, referrer, redirect_chain,
+ page_state, method, post_id);
return;
}
@@ -821,10 +821,10 @@ void NavigationEntryImpl::AddOrUpdateFrameEntry(
child->children.clear();
// Update the existing FrameNavigationEntry (e.g., for replaceState).
- child->frame_entry->UpdateEntry(unique_name, item_sequence_number,
- document_sequence_number, site_instance,
- std::move(source_site_instance), url,
- referrer, page_state, method, post_id);
+ child->frame_entry->UpdateEntry(
+ unique_name, item_sequence_number, document_sequence_number,
+ site_instance, std::move(source_site_instance), url, referrer,
+ redirect_chain, page_state, method, post_id);
return;
}
}

Powered by Google App Engine
This is Rietveld 408576698