.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/96b23bd9b84ebee3fda633fa9ba640b5ceccb67c commit 96b23bd9b84ebee3fda633fa9ba640b5ceccb67c Author: rsorokin <rsorokin@chromium.org> Date: Tue Nov 15 11:10:46 2016 Add Refresh{Device,User}Policies methods into AuthPolicyClient BUG= 656558 , 655971 Review-Url: https://codereview.chromium.org/2491153003 Cr-Commit-Position: refs/heads/master@{#432163} [modify] https://crrev.com/96b23bd9b84ebee3fda633fa9ba640b5ceccb67c/chromeos/dbus/auth_policy_client.cc [modify] https://crrev.com/96b23bd9b84ebee3fda633fa9ba640b5ceccb67c/chromeos/dbus/auth_policy_client.h [modify] https://crrev.com/96b23bd9b84ebee3fda633fa9ba640b5ceccb67c/chromeos/dbus/fake_auth_policy_client.cc [modify] https://crrev.com/96b23bd9b84ebee3fda633fa9ba640b5ceccb67c/chromeos/dbus/fake_auth_policy_client.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8d2791ecec5187ab2188b123ce50710d14f9b9e4 commit 8d2791ecec5187ab2188b123ce50710d14f9b9e4 Author: tnagel <tnagel@chromium.org> Date: Fri Nov 18 10:43:13 2016 Add DeviceADPolicyManager to provide AD policy. For Active Directory-managed devices, in BrowserPolicyConnectorChromeOS replace the DeviceCloudPolicyManagerChromeOS ConfigurationPolicyProvider by the new DeviceADPolicyManager ConfigurationPolicyProvider and disable DeviceLocalAccountPolicyService and ServerBackedStateKeysBroker. Note that accessors to the disabled services may return nullptr now. Policy refresh is not yet wired through to authpolicyd, instead it triggers another load from session_manager. BUG= 656558 TBR=bartfab (enterprise_platform_keys_private_api.cc) TBR=achuith (chrome/browser/ui/webui/options/chromeos) Review-Url: https://codereview.chromium.org/2486813002 Cr-Commit-Position: refs/heads/master@{#433171} [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/BUILD.gn [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/login/screens/chrome_user_selection_screen.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/login/users/avatar/user_image_manager_browsertest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager_policy_browsertest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h [add] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_active_directory_policy_manager.cc [add] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_cloud_policy_initializer.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_cloud_policy_store_chromeos.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_cloud_policy_store_chromeos.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/policy/enrollment_handler_chromeos.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/device_settings_service.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/device_settings_service.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/install_attributes.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/install_attributes.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/install_attributes_unittest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/session_manager_operation.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/session_manager_operation.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/chromeos/settings/session_manager_operation_unittest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/extensions/api/enterprise_platform_keys_private/enterprise_platform_keys_private_api.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/policy/cloud/policy_header_service_factory.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/policy/profile_policy_connector.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/policy/profile_policy_connector_factory.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/ui/webui/chromeos/login/core_oobe_handler.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/ui/webui/policy_ui_handler.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/chrome/browser/ui/webui/policy_ui_handler.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/components/policy/core/common/cloud/cloud_policy_client.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/components/policy/core/common/policy_map.cc [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/components/policy/core/common/policy_map.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/components/policy/core/common/policy_types.h [modify] https://crrev.com/8d2791ecec5187ab2188b123ce50710d14f9b9e4/components/policy_strings.grdp
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/dd095cf6bd65478ad5e09f63aa0d4a927e85fc89 commit dd095cf6bd65478ad5e09f63aa0d4a927e85fc89 Author: tnagel <tnagel@chromium.org> Date: Fri Nov 18 15:14:07 2016 Fix enrollment crash Fix a crash when hitting the "Done" button after enrollment that was introduced in 8d2791ecec5187ab2188b123ce50710d14f9b9e4. BUG= 656558 Review-Url: https://codereview.chromium.org/2515673002 Cr-Commit-Position: refs/heads/master@{#433199} [modify] https://crrev.com/dd095cf6bd65478ad5e09f63aa0d4a927e85fc89/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0f18dd0634ded648d1577bcf786555801e224401 commit 0f18dd0634ded648d1577bcf786555801e224401 Author: tnagel <tnagel@chromium.org> Date: Mon Nov 21 17:41:21 2016 Request device policy from authpolicyd Wire up the D-Bus call for device policy fetch in DeviceActiveDirectoryPolicyManager::RefreshPolicies(). BUG= 656558 Review-Url: https://codereview.chromium.org/2512733004 Cr-Commit-Position: refs/heads/master@{#433580} [modify] https://crrev.com/0f18dd0634ded648d1577bcf786555801e224401/chrome/browser/chromeos/policy/device_active_directory_policy_manager.cc [modify] https://crrev.com/0f18dd0634ded648d1577bcf786555801e224401/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h
It seems like the remaining task is setting up a timer to fetch policy at regular intervals.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/91cc209158c160afcfd6ddaccdbcc6a201db1f54 commit 91cc209158c160afcfd6ddaccdbcc6a201db1f54 Author: tnagel <tnagel@chromium.org> Date: Tue Nov 22 10:58:31 2016 Minor PolicyStatusProvider fixes Follow-up to https://codereview.chromium.org/2486813002/. BUG= 656558 Review-Url: https://codereview.chromium.org/2519023002 Cr-Commit-Position: refs/heads/master@{#433833} [modify] https://crrev.com/91cc209158c160afcfd6ddaccdbcc6a201db1f54/chrome/browser/ui/webui/policy_ui_handler.cc
The only remaining task, automatic policy refresh is tracked in issue 677304 .
bulk Verify of older or not-user-facing Chromad bugs
Comment 1 by tnagel@chromium.org
, Nov 3 2016