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

Side by Side Diff: content/browser/frame_host/navigation_handle_impl.cc

Issue 2951483002: TEST Check that PageTransition keeps bf flag
Patch Set: Created 3 years, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/navigation_handle_impl.h" 5 #include "content/browser/frame_host/navigation_handle_impl.h"
6 6
7 #include <iterator> 7 #include <iterator>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/metrics/histogram_macros.h" 10 #include "base/metrics/histogram_macros.h"
(...skipping 725 matching lines...) Expand 10 before | Expand all | Expand 10 after
736 const FrameHostMsg_DidCommitProvisionalLoad_Params& params, 736 const FrameHostMsg_DidCommitProvisionalLoad_Params& params,
737 bool navigation_entry_committed, 737 bool navigation_entry_committed,
738 bool did_replace_entry, 738 bool did_replace_entry,
739 const GURL& previous_url, 739 const GURL& previous_url,
740 NavigationType navigation_type, 740 NavigationType navigation_type,
741 RenderFrameHostImpl* render_frame_host) { 741 RenderFrameHostImpl* render_frame_host) {
742 DCHECK(!render_frame_host_ || render_frame_host_ == render_frame_host); 742 DCHECK(!render_frame_host_ || render_frame_host_ == render_frame_host);
743 DCHECK_EQ(frame_tree_node_, render_frame_host->frame_tree_node()); 743 DCHECK_EQ(frame_tree_node_, render_frame_host->frame_tree_node());
744 CHECK_EQ(url_, params.url); 744 CHECK_EQ(url_, params.url);
745 745
746 CHECK_EQ(transition_ & ui::PAGE_TRANSITION_FORWARD_BACK,
747 params.transition & ui::PAGE_TRANSITION_FORWARD_BACK);
nasko 2017/06/23 13:40:38 One of the cases where this is hit is for history
748
746 did_replace_entry_ = did_replace_entry; 749 did_replace_entry_ = did_replace_entry;
747 method_ = params.method; 750 method_ = params.method;
748 has_user_gesture_ = (params.gesture == NavigationGestureUser); 751 has_user_gesture_ = (params.gesture == NavigationGestureUser);
749 transition_ = params.transition; 752 transition_ = params.transition;
750 should_update_history_ = params.should_update_history; 753 should_update_history_ = params.should_update_history;
751 render_frame_host_ = render_frame_host; 754 render_frame_host_ = render_frame_host;
752 previous_url_ = previous_url; 755 previous_url_ = previous_url;
753 base_url_ = params.base_url; 756 base_url_ = params.base_url;
754 socket_address_ = params.socket_address; 757 socket_address_ = params.socket_address;
755 navigation_type_ = navigation_type; 758 navigation_type_ = navigation_type;
(...skipping 402 matching lines...) Expand 10 before | Expand all | Expand 10 after
1158 if (new_site_url == site_url_) 1161 if (new_site_url == site_url_)
1159 return; 1162 return;
1160 1163
1161 // When redirecting cross-site, stop telling the speculative 1164 // When redirecting cross-site, stop telling the speculative
1162 // RenderProcessHost to expect a navigation commit. 1165 // RenderProcessHost to expect a navigation commit.
1163 SetExpectedProcess(nullptr); 1166 SetExpectedProcess(nullptr);
1164 site_url_ = new_site_url; 1167 site_url_ = new_site_url;
1165 } 1168 }
1166 1169
1167 } // namespace content 1170 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698