The delay [1] needs to be removed, but we also need to make sure that the session doesn't start before policy has been received. [1] https://cs.chromium.org/chromium/src/chrome/browser/chromeos/policy/active_directory_policy_manager.cc?type=cs&l=27
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/adacf601d936418bd1604e62c32f43258e2d7946 commit adacf601d936418bd1604e62c32f43258e2d7946 Author: tnagel <tnagel@chromium.org> Date: Wed Jun 14 13:40:50 2017 Chromad: Drop delay for policy refresh on startup Policy fetch used to be slow and thus was delayed to prevent blocking login. Now that policy fetch has become faster, this CL removes the delay. BUG= 732801 Review-Url: https://codereview.chromium.org/2937953002 Cr-Commit-Position: refs/heads/master@{#479371} [modify] https://crrev.com/adacf601d936418bd1604e62c32f43258e2d7946/chrome/browser/chromeos/policy/active_directory_policy_manager.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cab862ac8252d68584eaf94f9b19e639f03200d7 commit cab862ac8252d68584eaf94f9b19e639f03200d7 Author: tnagel <tnagel@chromium.org> Date: Wed Jul 19 08:19:55 2017 Interface and implementation are modeled after UserCloudPolicyManagerChromeOS. Also added some slight modifications/explanations to the latter. BUG= 732801 TEST=added unit tests Review-Url: https://codereview.chromium.org/2954293002 Cr-Commit-Position: refs/heads/master@{#487791} [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/BUILD.gn [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/active_directory_policy_manager.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/active_directory_policy_manager.h [add] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/active_directory_policy_manager_unittest.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos_unittest.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chrome/browser/chromeos/policy/user_policy_manager_factory_chromeos.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chromeos/dbus/dbus_thread_manager.cc [modify] https://crrev.com/cab862ac8252d68584eaf94f9b19e639f03200d7/chromeos/dbus/dbus_thread_manager.h
Comment 1 by bugdroid1@chromium.org
, Jun 14 2017