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

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

Issue 746993003: Only cancel navigation on user gesture (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Try2 : Only cancel when it is caused by user action Created 6 years 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 39956621dfd5f5cfadb5b0ad0fe123726f7e58ce..06637dc84c7115bba153ffc122394baa69994e53 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -517,14 +517,16 @@ void NavigatorImpl::DidNavigate(
}
if (!use_site_per_process)
- frame_tree->root()->render_manager()->DidNavigateFrame(render_frame_host);
+ frame_tree->root()->render_manager()->DidNavigateFrame(
+ render_frame_host, input_params.gesture == NavigationGestureUser);
Charlie Reis 2014/12/03 21:32:19 Please use params (here and below), not input_para
}
// When using --site-per-process, we notify the RFHM for all navigations,
// not just main frame navigations.
if (use_site_per_process) {
FrameTreeNode* frame = render_frame_host->frame_tree_node();
- frame->render_manager()->DidNavigateFrame(render_frame_host);
+ frame->render_manager()->DidNavigateFrame(
+ render_frame_host, input_params.gesture == NavigationGestureUser);
}
// Update the site of the SiteInstance if it doesn't have one yet, unless

Powered by Google App Engine
This is Rietveld 408576698