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

Unified Diff: chrome/browser/resources/settings/people_page/sync_browser_proxy.js

Issue 2825493003: MD Settings: change outlinks to actually use <a> (Closed)
Patch Set: merge Created 3 years, 8 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: chrome/browser/resources/settings/people_page/sync_browser_proxy.js
diff --git a/chrome/browser/resources/settings/people_page/sync_browser_proxy.js b/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
index 1d332be06d7a4515a43a18cbf691e2a2ba50aa15..572dcb4f9c04bd9ee3d49f9f89f2bac1eba626f4 100644
--- a/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
+++ b/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
@@ -232,7 +232,6 @@ cr.define('settings', function() {
openActivityControlsUrl: function() {
chrome.metricsPrivate.recordUserAction(
'Signin_AccountSettings_GoogleActivityControlsClicked');
- window.open(loadTimeData.getString('activityControlsUrl'));
}
};

Powered by Google App Engine
This is Rietveld 408576698