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

Unified Diff: chrome/browser/resources/settings/settings_page/settings_router.js

Issue 2131953002: Site Settings Desktop: Implement protocol handler section. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test 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/settings_page/settings_router.js
diff --git a/chrome/browser/resources/settings/settings_page/settings_router.js b/chrome/browser/resources/settings/settings_page/settings_router.js
index cf987729e2bfd677309fd865ec477f71a459f761..d26933e73018a81a7c08b9a040b9f1d8c298584a 100644
--- a/chrome/browser/resources/settings/settings_page/settings_router.js
+++ b/chrome/browser/resources/settings/settings_page/settings_router.js
@@ -283,22 +283,28 @@ Polymer({
subpage: ['site-settings', 'site-settings-category-images'],
},
{
- url: '/siteSettings/keygen',
+ url: '/siteSettings/handlers',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-keygen'],
+ subpage: ['site-settings', 'protocol-handlers'],
},
{
- url: '/siteSettings/location',
+ url: '/siteSettings/javascript',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-location'],
+ subpage: ['site-settings', 'site-settings-category-javascript'],
},
{
- url: '/siteSettings/javascript',
+ url: '/siteSettings/keygen',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-javascript'],
+ subpage: ['site-settings', 'site-settings-category-keygen'],
+ },
+ {
+ url: '/siteSettings/location',
+ page: 'advanced',
+ section: 'privacy',
+ subpage: ['site-settings', 'site-settings-category-location'],
},
{
url: '/siteSettings/microphone',
@@ -380,24 +386,24 @@ Polymer({
'site-details'],
},
{
- url: '/siteSettings/keygen/details',
+ url: '/siteSettings/javascript/details',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-keygen',
+ subpage: ['site-settings', 'site-settings-category-javascript',
'site-details'],
},
{
- url: '/siteSettings/location/details',
+ url: '/siteSettings/keygen/details',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-location',
+ subpage: ['site-settings', 'site-settings-category-keygen',
'site-details'],
},
{
- url: '/siteSettings/javascript/details',
+ url: '/siteSettings/location/details',
page: 'advanced',
section: 'privacy',
- subpage: ['site-settings', 'site-settings-category-javascript',
+ subpage: ['site-settings', 'site-settings-category-location',
'site-details'],
},
{

Powered by Google App Engine
This is Rietveld 408576698