Delete LegacyPolicyCacheLoader. |
||||||||||
Issue description// TODO(mnissler): Remove all the legacy policy support members below after // the number of pre-M20 clients drops back to zero. https://cs.chromium.org/chromium/src/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.h?rcl=0&l=131 I guess it's safe to remove the LegacyPolicyCacheLoader (whatever that is about)?
,
Oct 14 2016
Assigned to dskaram for triage.
,
Oct 17 2016
No need to bother David with a minor code health issue.
,
Nov 4 2016
,
Nov 7 2016
,
Nov 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/21108b94206bdb524c67b3d4b765b6965cecc4bd commit 21108b94206bdb524c67b3d4b765b6965cecc4bd Author: pmarko <pmarko@chromium.org> Date: Tue Nov 22 08:58:53 2016 Remove legacy policy cache support on chromeos Remove obsolete loading of legacy policy cache files from the chrome profile (created before session_manager was used to store policy). Also remove unit tests which were testing the migration path. BUG= 655956 TEST=ran all components_unittests and unit_tests Review-Url: https://codereview.chromium.org/2487703002 Cr-Commit-Position: refs/heads/master@{#433818} [modify] https://crrev.com/21108b94206bdb524c67b3d4b765b6965cecc4bd/chrome/browser/chromeos/policy/user_cloud_policy_manager_factory_chromeos.cc [modify] https://crrev.com/21108b94206bdb524c67b3d4b765b6965cecc4bd/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc [modify] https://crrev.com/21108b94206bdb524c67b3d4b765b6965cecc4bd/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.h [modify] https://crrev.com/21108b94206bdb524c67b3d4b765b6965cecc4bd/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos_unittest.cc
,
Nov 23 2016
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Jul 6 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by mnissler@chromium.org
, Oct 14 2016