| Index: reviewbot/handlers/policy_checklist/modification_comment.txt
|
| ===================================================================
|
| --- reviewbot/handlers/policy_checklist/modification_comment.txt (revision 0)
|
| +++ reviewbot/handlers/policy_checklist/modification_comment.txt (revision 0)
|
| @@ -0,0 +1,17 @@
|
| +When modifying an existing policy setting, please consider the following:
|
| +
|
| +- Make sure the policy meta data is up-to-date, in particular
|
| + supported_on, and the feature flags.
|
| +- In general, don't change policy semantics in a way that is
|
| + incompatible (as determined by user/admin-visible behavior) with
|
| + previous semantics. In particular, consider that existing policy
|
| + deployments may affect both old and new browser versions, and both
|
| + should behave according to the admin's intentions.
|
| +- It is OK to expand semantics of policy values as long as the previous
|
| + policy description is compatible with the new behavior.
|
| +- It is OK to update feature implementations and the policy description
|
| + when Chrome changes as long as the intended effect of the policy
|
| + remains intact.
|
| +- The process for removing policies is to deprecate them first, wait a
|
| + few releases (if possible) and then remove them. Make sure you put the
|
| + deprecated flag if you deprecate a policy.
|
|
|
| Property changes on: reviewbot/handlers/policy_checklist/modification_comment.txt
|
| ___________________________________________________________________
|
| Added: svn:eol-style
|
| + LF
|
|
|
|
|