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

Unified Diff: components/navigation_interception/navigation_params.cc

Issue 1091253008: Fix an issue that external protocol in subframes are not handled on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase and fix test Created 5 years, 7 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: components/navigation_interception/navigation_params.cc
diff --git a/components/navigation_interception/navigation_params.cc b/components/navigation_interception/navigation_params.cc
index 83579fb3855382d35b4d085d0ea8f41b9f3bf6f6..54288f76c724c537460a84a629462c5587a06d47 100644
--- a/components/navigation_interception/navigation_params.cc
+++ b/components/navigation_interception/navigation_params.cc
@@ -16,14 +16,16 @@ NavigationParams::NavigationParams(const GURL& url,
bool is_post,
ui::PageTransition transition_type,
bool is_redirect,
- bool is_external_protocol)
+ bool is_external_protocol,
+ bool is_main_frame)
: url_(url),
referrer_(referrer),
has_user_gesture_(has_user_gesture),
is_post_(is_post),
transition_type_(transition_type),
is_redirect_(is_redirect),
- is_external_protocol_(is_external_protocol) {
+ is_external_protocol_(is_external_protocol),
+ is_main_frame_(is_main_frame) {
}
void NavigationParams::operator=(const NavigationParams& rhs) {
@@ -38,6 +40,7 @@ void NavigationParams::Assign(const NavigationParams& other) {
transition_type_ = other.transition_type();
is_redirect_ = other.is_redirect();
is_external_protocol_ = other.is_external_protocol();
+ is_main_frame_ = other.is_main_frame();
}
} // namespace navigation_interception

Powered by Google App Engine
This is Rietveld 408576698