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

Unified Diff: components/policy/resources/policy_templates.json

Issue 309553011: Enable policy support for registering protocol handler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Issue_116119_pre
Patch Set: Created 6 years, 7 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: components/policy/resources/policy_templates.json
diff --git a/components/policy/resources/policy_templates.json b/components/policy/resources/policy_templates.json
index 50b442a570e486387beff931a1c939d43b0ecd26..c999baeac9cad6e2d9db6298c251eb9511ca3f4e 100644
--- a/components/policy/resources/policy_templates.json
+++ b/components/policy/resources/policy_templates.json
@@ -118,7 +118,7 @@
# persistent IDs for all fields (but not for groups!) are needed. These are
# specified by the 'id' keys of each policy. NEVER CHANGE EXISTING IDs,
# because doing so would break the deployed wire format!
-# For your editing convenience: highest ID currently used: 266
+# For your editing convenience: highest ID currently used: 268
#
# Placeholders:
# The following placeholder strings are automatically substituted:
@@ -2668,6 +2668,80 @@
If this policy is left not set the global default value will be used for all sites either from the 'DefaultPopupsSetting' policy if it is set, or the user's personal configuration otherwise.''',
},
{
+ 'name': 'RegisteredProtocolHandlers',
+ 'type': 'list',
+ 'schema': {
+ 'type': 'array',
+ 'items': {
+ 'type': 'object',
+ 'properties': {
+ 'default': {
+ 'description': 'A boolean flag indicating if the protocol handler should be set as the default.',
+ 'type': 'boolean'
+ },
+ 'protocol': {
+ 'description': 'The protocol for the protocol handler.',
+ 'type': 'string'
+ },
+ 'url': {
+ 'description': 'The URL of the protocol handler.',
+ 'type': 'string'
+ }
+ },
+ 'required': ['protocol', 'url']
+ }
+ },
+ 'supported_on': [
bartfab (slow) 2014/06/03 11:58:21 Nit: Would the supported_on entries not have all f
kaliamoorthi 2014/06/04 18:01:58 Done.
+ 'chrome.*:37-',
+ 'chrome_os:37-',
+ 'ios:37-',
+ 'android:37-',
+ ],
+ 'features': {
+ 'dynamic_refresh': False,
bartfab (slow) 2014/06/03 11:58:21 Why can this not be dynamically changed?
kaliamoorthi 2014/06/04 18:01:58 We discussed this, I'll file a separate bug.
+ 'per_profile': True,
+ },
+ 'example_value': [{'protocol': 'mailto', 'url': 'https://mail.google.com/mail/?extsrc=mailto&url=%s'}],
+ 'id': 267,
+ 'caption': '''Register protocol handlers''',
+ 'desc': '''Allows you to register a list of protocol handlers.''',
bartfab (slow) 2014/06/03 11:58:21 You should briefly document the format of the list
kaliamoorthi 2014/06/04 18:01:58 Done.
+ },
+ {
+ 'name': 'IgnoredProtocolHandlers',
+ 'type': 'list',
+ 'schema': {
+ 'type': 'array',
+ 'items': {
+ 'type': 'object',
+ 'properties': {
+ 'protocol': {
+ 'description': 'The protocol for the protocol handler.',
+ 'type': 'string'
+ },
+ 'url': {
+ 'description': 'The URL of the protocol handler.',
+ 'type': 'string'
+ }
+ },
+ 'required': ['protocol', 'url']
+ }
+ },
+ 'supported_on': [
+ 'chrome.*:37-',
+ 'chrome_os:37-',
+ 'ios:37-',
+ 'android:37-',
+ ],
+ 'features': {
+ 'dynamic_refresh': False,
bartfab (slow) 2014/06/03 11:58:21 Why can this not be dynamically changed?
kaliamoorthi 2014/06/04 18:01:58 We discussed this, I'll file a separate bug.
+ 'per_profile': True,
+ },
+ 'example_value': [{'protocol': 'mailto', 'url': 'https://mail.google.com/mail/?extsrc=mailto&url=%s'}],
+ 'id': 268,
+ 'caption': '''Ignore protocol handlers''',
+ 'desc': '''Allows you to ignore a list of protocol handlers.''',
bartfab (slow) 2014/06/03 11:58:21 What does this mean? The documentation is not clea
kaliamoorthi 2014/06/04 18:01:58 I have expanded this.
+ },
+ {
'name': 'PopupsBlockedForUrls',
'type': 'list',
'schema': {

Powered by Google App Engine
This is Rietveld 408576698