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

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

Issue 2156413002: Settings Router Refactor: Migrate to settings.Route.navigateTo calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge origin/master Created 4 years, 5 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 7b053968a15a6caddb74709d2b33897fe9006cfe..75637fd3948fc77f5dc41e6a3a49c9286588e91b 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
@@ -106,6 +106,84 @@ var SiteSettingsBehaviorImpl = {
},
/**
+ * A utility function to lookup the route for a category name.
+ * @param {string} category The category ID to look up.
+ * @return {!settings.Route}
+ * @protected
+ */
+ computeCategoryRoute: function(category) {
+ switch (category) {
+ case settings.ContentSettingsTypes.AUTOMATIC_DOWNLOADS:
+ return settings.Route.SITE_SETTINGS_AUTOMATIC_DOWNLOADS;
+ case settings.ContentSettingsTypes.BACKGROUND_SYNC:
+ return settings.Route.SITE_SETTINGS_BACKGROUND_SYNC;
+ case settings.ContentSettingsTypes.CAMERA:
+ return settings.Route.SITE_SETTINGS_CAMERA;
+ case settings.ContentSettingsTypes.COOKIES:
+ return settings.Route.SITE_SETTINGS_COOKIES;
+ case settings.ContentSettingsTypes.GEOLOCATION:
+ return settings.Route.SITE_SETTINGS_LOCATION;
+ case settings.ContentSettingsTypes.IMAGES:
+ return settings.Route.SITE_SETTINGS_IMAGES;
+ case settings.ContentSettingsTypes.JAVASCRIPT:
+ return settings.Route.SITE_SETTINGS_JAVASCRIPT;
+ case settings.ContentSettingsTypes.KEYGEN:
+ return settings.Route.SITE_SETTINGS_KEYGEN;
+ case settings.ContentSettingsTypes.MIC:
+ return settings.Route.SITE_SETTINGS_MICROPHONE;
+ case settings.ContentSettingsTypes.NOTIFICATIONS:
+ return settings.Route.SITE_SETTINGS_NOTIFICATIONS;
+ case settings.ContentSettingsTypes.PLUGINS:
+ return settings.Route.SITE_SETTINGS_PLUGINS;
+ case settings.ContentSettingsTypes.POPUPS:
+ return settings.Route.SITE_SETTINGS_POPUPS;
+ case settings.ContentSettingsTypes.UNSANDBOXED_PLUGINS:
+ return settings.Route.SITE_SETTINGS_UNSANDBOXED_PLUGINS;
+ default:
Dan Beam 2016/07/23 00:17:12 nit: if this list is exhaustive, can you remove th
tommycli 2016/07/25 16:47:21 Done.
+ assertNotReached();
+ }
+ },
+
+ /**
+ * 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;
+ default:
+ assertNotReached();
Dan Beam 2016/07/23 00:17:12 same
tommycli 2016/07/25 16:47:21 Done.
+ }
+ },
+
+ /**
* 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.

Powered by Google App Engine
This is Rietveld 408576698