CryptAuthDeviceManager: trust incoming CryptAuth enable feature bits |
||||
Issue descriptionDeviceManager as-is errors out if a feature is marked as enabled but not supported [1]. Even in this weird edge case, DeviceManager should still mark the feature as enabled locally. 1) https://cs.chromium.org/chromium/src/components/cryptauth/cryptauth_device_manager_impl.cc?q=cryptauthdevicemanagerimpl&sq=package:chromium&dr=CSs&l=144 https://cs.chromium.org/chromium/src/chromeos/services/device_sync/cryptauth_device_manager_impl.cc?q=cryptauth_device_manager_impl&sq=package:chromium&dr=C&l=161
,
Dec 28
,
Dec 28
,
Jan 7
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7bad39df276337bd8962dc69b3e1760d365062a8 commit 7bad39df276337bd8962dc69b3e1760d365062a8 Author: Regan Hsu <hsuregan@chromium.org> Date: Mon Jan 07 19:27:37 2019 [CrOS MultiDevice] Set feature state enabled on edge case. DeviceManager as-is errors out if a feature is marked as enabled but not supported. Even in this weird edge case, DeviceManager should still mark the feature as enabled locally. Bug: 900787 Change-Id: I442db075f520d3e5edf0cafa697b3dcdea2325dd Reviewed-on: https://chromium-review.googlesource.com/c/1392254 Reviewed-by: Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Regan Hsu <hsuregan@chromium.org> Cr-Commit-Position: refs/heads/master@{#620422} [modify] https://crrev.com/7bad39df276337bd8962dc69b3e1760d365062a8/chromeos/services/device_sync/cryptauth_device_manager_impl.cc
,
Jan 7
|
||||
►
Sign in to add a comment |
||||
Comment 1 by jlklein@chromium.org
, Dec 28