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

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

Issue 2272213005: Site Settings Desktop: Add links to Adobe settings and Learn more. (Closed)
Patch Set: Address feedback Created 4 years, 4 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_settings_category.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_category.js b/chrome/browser/resources/settings/site_settings/site_settings_category.js
index c1aa1baa373cc1c4eb9a2766fba1a57dcb5ead6c..4dab58b9f203404d92a53802abbcf69678df520e 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_category.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_category.js
@@ -114,4 +114,26 @@ Polymer({
this.categoryEnabled = enabled;
}.bind(this));
},
+
+ /**
+ * Returns whether this is the Plugins category.
+ * @param {string} category The current category.
+ * @return {boolean} Whether this is the Plugins category.
+ * @private
+ */
+ isPluginCategory_: function(category) {
+ return category == settings.ContentSettingsTypes.PLUGINS;
+ },
+
+ /** @private */
+ onLearnMoreClicked_: function() {
+ window.open(
+ 'https://support.google.com/chrome/?p=settings_manage_exceptions');
+ },
+
+ /** @private */
+ onAdobeFlashStorageClicked_: function() {
+ window.open('https://www.macromedia.com/support/' +
+ 'documentation/en/flashplayer/help/settings_manager07.html');
+ },
});

Powered by Google App Engine
This is Rietveld 408576698