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

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

Issue 1477773003: Use dom-if to hide settings pages and show explicitly. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Feedback Created 5 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: chrome/browser/resources/settings/site_settings/site_settings_behavior.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
index a72c5c5735a67fb12ac1daa69e1ccb8ef1192a0e..baeefec48a3667e843368423ae66b55c27569c0f 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
@@ -8,6 +8,16 @@
/** @polymerBehavior */
var SiteSettingsBehaviorImpl = {
+ properties: {
+ /**
+ * The ID of the category this element is displaying data for.
+ */
+ category: {
+ type: Number,
+ value: settings.ContentSettingsTypes.GEOLOCATION
michaelpg 2015/12/02 19:22:52 No reason to make geolocation the default category
stevenjb 2015/12/02 21:57:10 Done.
+ },
+ },
+
/**
* Returns whether the category default is set to enabled or not.
* @param {number} category The category to check.

Powered by Google App Engine
This is Rietveld 408576698