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

Unified Diff: chrome/browser/resources/settings/site_settings/site_list.js

Issue 2409003002: [MD settings] flag to enable site settings All sites and Site details pages (Closed)
Patch Set: merge with master; fix typo Created 4 years, 2 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/site_settings/site_list.js
diff --git a/chrome/browser/resources/settings/site_settings/site_list.js b/chrome/browser/resources/settings/site_settings/site_list.js
index f2fcb7c1602ae812b7a6d06487c11f0aba0b208e..41f844e50bf027e34072ff405f0adb9c232de9c3 100644
--- a/chrome/browser/resources/settings/site_settings/site_list.js
+++ b/chrome/browser/resources/settings/site_settings/site_list.js
@@ -26,6 +26,14 @@ Polymer({
behaviors: [SiteSettingsBehavior, WebUIListenerBehavior],
properties: {
+ /** @private */
+ enableSiteSettings_: {
+ type: Boolean,
+ value: function() {
+ return loadTimeData.getBoolean('enableSiteSettings');
+ },
+ },
+
/**
* The site that was selected by the user in the dropdown list.
* @type {SiteException}
@@ -418,6 +426,8 @@ Polymer({
* @private
*/
onOriginTap_: function(event) {
+ if (!this.enableSiteSettings_)
+ return;
this.selectedSite = event.model.item;
settings.navigateTo(settings.Route.SITE_SETTINGS_SITE_DETAILS,
new URLSearchParams('site=' + this.selectedSite.origin));

Powered by Google App Engine
This is Rietveld 408576698