[EasyUnlock] Add easy_unlock.enabled pref and sync it to local state

Desktop / Chromium - Tim Song [chromium.org] - 9 August 2017 02:13 EDT

Previously, the login screen looks at the presence of the hardlock local pref to determine if the user has set up EasyUnlock. This was a very unreliable and unintuitive thing to rely on, so we now explicitly have an easy_unlock.enabled pref.

For easy of merging, this CL also fixes a notification bug when setting up EasyUnlock.

BUG=746945,751808

Change-Id: I437bcd3dc5f9d5e113b22a9861467f35ec1e7d62 Reviewed-on: https://chromium-review.googlesource.com/599349 Commit-Queue: Tim Song

a3e8ece [EasyUnlock] Add easy_unlock.enabled pref and sync it to local state.
chrome/browser/signin/easy_unlock_service.cc | 4 +--
.../browser/signin/easy_unlock_service_regular.cc | 37 +++++++++++++++++++---
.../browser/signin/easy_unlock_service_regular.h | 1 +
.../signin/easy_unlock_service_signin_chromeos.cc | 22 +++++++++++--
.../signin/easy_unlock_service_signin_chromeos.h | 1 +
.../proximity_auth_local_state_pref_manager.cc | 16 ++++++++++
.../proximity_auth_local_state_pref_manager.h | 2 ++
...imity_auth_local_state_pref_manager_unittest.cc | 24 ++++++++++++++
.../proximity_auth/proximity_auth_pref_manager.h | 10 +++++-
.../proximity_auth/proximity_auth_pref_names.cc | 4 +++
.../proximity_auth/proximity_auth_pref_names.h | 1 +
.../proximity_auth_profile_pref_manager.cc | 13 ++++++++
.../proximity_auth_profile_pref_manager.h | 2 ++
...proximity_auth_profile_pref_manager_unittest.cc | 15 +++++++++
14 files changed, 140 insertions(+), 12 deletions(-)

Upstream: git.chromium.org


  • Share