Consolidate policy refresh scheduling |
|||||
Issue descriptionWe have CloudPolicyRefreshScheduler and built-in scheduling in ActiveDirectoryPolicyManager. We should consolidate this into a single scheduler.
,
Jun 21 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/dd04f963fac90b8d3502d8e96340b07be463b760 commit dd04f963fac90b8d3502d8e96340b07be463b760 Author: tnagel <tnagel@chromium.org> Date: Wed Jun 21 17:46:25 2017 Extract AD policy scheduler into separate class The new code should be easier to test, easier to extend and easier to re-use. BUG=734776 TEST=added unittests for scheduler Review-Url: https://codereview.chromium.org/2942373002 Cr-Commit-Position: refs/heads/master@{#481237} [modify] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/chrome/browser/chromeos/policy/active_directory_policy_manager.cc [modify] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/chrome/browser/chromeos/policy/active_directory_policy_manager.h [modify] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/components/policy/core/common/BUILD.gn [add] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/components/policy/core/common/policy_scheduler.cc [add] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/components/policy/core/common/policy_scheduler.h [add] https://crrev.com/dd04f963fac90b8d3502d8e96340b07be463b760/components/policy/core/common/policy_scheduler_unittest.cc
,
Jul 26 2017
Next step is to merge CloudPolicyRefreshScheduler and PolicyScheduler.
,
Jul 26 2017
For example, PolicyScheduler does not account for time the device has slept and thus would benefit from that piece of CloudPolicyRefreshScheduler functionality.
,
Jul 28 2017
,
Sep 15 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by tnagel@chromium.org
, Jun 19 2017