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

Unified Diff: components/security_interstitials/core/controller_client.cc

Issue 2955503002: Make interstitial links open in a new tab (Closed)
Patch Set: Rebase CL 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 side-by-side diff with in-line comments
Download patch
Index: components/security_interstitials/core/controller_client.cc
diff --git a/components/security_interstitials/core/controller_client.cc b/components/security_interstitials/core/controller_client.cc
index 1c61519f275721186db22c47b8fc729489047da6..23bc794ab1c7a819da4952031970e71dba45be1e 100644
--- a/components/security_interstitials/core/controller_client.cc
+++ b/components/security_interstitials/core/controller_client.cc
@@ -45,7 +45,7 @@ void ControllerClient::OpenExtendedReportingPrivacyPolicy() {
GURL privacy_url(kSafeBrowsingPrivacyPolicyUrl);
privacy_url =
google_util::AppendGoogleLocaleParam(privacy_url, GetApplicationLocale());
- OpenUrlInCurrentTab(privacy_url);
+ OpenUrlInNewForegroundTab(privacy_url);
}
void ControllerClient::OpenExtendedReportingWhitepaper() {
@@ -53,7 +53,7 @@ void ControllerClient::OpenExtendedReportingWhitepaper() {
GURL whitepaper_url(kSafeBrowsingWhitePaperUrl);
whitepaper_url = google_util::AppendGoogleLocaleParam(whitepaper_url,
GetApplicationLocale());
- OpenUrlInCurrentTab(whitepaper_url);
+ OpenUrlInNewForegroundTab(whitepaper_url);
}
} // namespace security_interstitials

Powered by Google App Engine
This is Rietveld 408576698