Commit 26fd541c authored by Chris Sharp's avatar Chris Sharp Committed by Commit Bot

Reland "Update schema for PolicyRefreshRate and MaxInvalidationFetchDelay"

This is a reland of dc5bcc30

BYPASS_POLICY_COMPATIBILITY_CHECK=Min and max values already specified,
                                   they just were missing from the schema
TBR=pmarko@chromium.org

Original change's description:
> Update schema for PolicyRefreshRate and MaxInvalidationFetchDelay
>
> Include the min and max values already specified in the policy
> descriptions and in the code handling the policies.
>
> The limits for the PolicyRefreshRate policy are specified in
> components/policy/core/common/cloud/cloud_policy_refresh_scheduler.cc.
>
> The limits for the MaxInvalidationFetchDelay policy are specified in
> chrome/browser/policy/cloud/cloud_policy_invalidator.cc.
>
> BYPASS_POLICY_COMPATIBILITY_CHECK=Min and max values already specified,
>                                   they just were missing from the schema
>
> Change-Id: Ia1b3904dd1f377d5bc1ee7c2742f3eaeb3f34969
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2349211
> Commit-Queue: Chris Sharp <csharp@chromium.org>
> Reviewed-by: Pavol Marko <pmarko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#797407}

Change-Id: I8260c5e00aabac51e92bd51f057b4648f913767a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2354458Reviewed-by: default avatarChris Sharp <csharp@chromium.org>
Commit-Queue: Chris Sharp <csharp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799277}
parent f02cc013
...@@ -7039,7 +7039,7 @@ ...@@ -7039,7 +7039,7 @@
'name': 'PolicyRefreshRate', 'name': 'PolicyRefreshRate',
'owners': ['file://components/policy/resources/OWNERS'], 'owners': ['file://components/policy/resources/OWNERS'],
'type': 'int', 'type': 'int',
'schema': { 'type': 'integer' }, 'schema': { 'type': 'integer', 'minimum': 1800000, 'maximum': 86400000 },
'supported_on': ['chrome_os:11-', 'chrome.*:79-'], 'supported_on': ['chrome_os:11-', 'chrome.*:79-'],
'features': { 'features': {
'dynamic_refresh': True, 'dynamic_refresh': True,
...@@ -7061,7 +7061,7 @@ ...@@ -7061,7 +7061,7 @@
'name': 'MaxInvalidationFetchDelay', 'name': 'MaxInvalidationFetchDelay',
'owners': ['stepco@chromium.org', 'poromov@chromium.org'], 'owners': ['stepco@chromium.org', 'poromov@chromium.org'],
'type': 'int', 'type': 'int',
'schema': { 'type': 'integer' }, 'schema': { 'type': 'integer', 'minimum': 1000, 'maximum': 30000 },
'supported_on': ['chrome.*:30-', 'chrome_os:30-'], 'supported_on': ['chrome.*:30-', 'chrome_os:30-'],
'features': { 'features': {
'dynamic_refresh': True, 'dynamic_refresh': True,
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment