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

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

Issue 2249873003: Settings: Fix Site Details subpage routing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix comment typo 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_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 b3b57a5eaddb649233d4ae5c553af4b1d587e54d..d6bb2c43d74f0db5598b729288cd8aabe44bc89a 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
@@ -150,44 +150,6 @@ var SiteSettingsBehaviorImpl = {
},
/**
- * A utility function to lookup the 'details' route for a category name.
- * @param {string} category The category ID to look up.
- * @return {!settings.Route}
- * @protected
- */
- computeCategoryDetailsRoute: function(category) {
- switch (category) {
- case settings.ContentSettingsTypes.AUTOMATIC_DOWNLOADS:
- return settings.Route.SITE_SETTINGS_AUTOMATIC_DOWNLOADS_DETAILS;
- case settings.ContentSettingsTypes.BACKGROUND_SYNC:
- return settings.Route.SITE_SETTINGS_BACKGROUND_SYNC_DETAILS;
- case settings.ContentSettingsTypes.CAMERA:
- return settings.Route.SITE_SETTINGS_CAMERA_DETAILS;
- case settings.ContentSettingsTypes.COOKIES:
- return settings.Route.SITE_SETTINGS_COOKIES_DETAILS;
- case settings.ContentSettingsTypes.GEOLOCATION:
- return settings.Route.SITE_SETTINGS_LOCATION_DETAILS;
- case settings.ContentSettingsTypes.IMAGES:
- return settings.Route.SITE_SETTINGS_IMAGES_DETAILS;
- case settings.ContentSettingsTypes.JAVASCRIPT:
- return settings.Route.SITE_SETTINGS_JAVASCRIPT_DETAILS;
- case settings.ContentSettingsTypes.KEYGEN:
- return settings.Route.SITE_SETTINGS_KEYGEN_DETAILS;
- case settings.ContentSettingsTypes.MIC:
- return settings.Route.SITE_SETTINGS_MICROPHONE_DETAILS;
- case settings.ContentSettingsTypes.NOTIFICATIONS:
- return settings.Route.SITE_SETTINGS_NOTIFICATIONS_DETAILS;
- case settings.ContentSettingsTypes.PLUGINS:
- return settings.Route.SITE_SETTINGS_PLUGINS_DETAILS;
- case settings.ContentSettingsTypes.POPUPS:
- return settings.Route.SITE_SETTINGS_POPUPS_DETAILS;
- case settings.ContentSettingsTypes.UNSANDBOXED_PLUGINS:
- return settings.Route.SITE_SETTINGS_UNSANDBOXED_PLUGINS_DETAILS;
- }
- assertNotReached();
- },
-
- /**
* A utility function to compute the icon to use for the category, both for
* the overall category as well as the individual permission in the details
* for a site.
« no previous file with comments | « chrome/browser/resources/settings/site_settings/site_list.js ('k') | chrome/test/data/webui/settings/route_tests.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698