Commit cd2aedf2 authored by Toby Huang's avatar Toby Huang Committed by Commit Bot

Change kSupervisedUserApprovedExtensions to SYNCABLE_PREF

The pref kSupervisedUserApprovedExtensions contains extension ids that
the parent has approved for the supervised user. We want these
approvals to sync across ChromeOS devices. As a SYNCABLE_OS_PREF,
kSupervisedUserApprovedExtensions does not sync from M83 to M84 due to
the SplitSettingsSync feature in M84. Changing this pref to a
SYNCABLE_PREF fixes this syncing issue from M83->M84.

Moreover, this pref should be a browser pref instead of an OS pref to
be more consistent with the rest of extensions.

Bug: 1068794
Change-Id: I8ae34dbd1e12d312ff72f062c9e5026bb8e6a525
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2147853Reviewed-by: default avatarJames Cook <jamescook@chromium.org>
Reviewed-by: default avatarAga Wronska <agawronska@chromium.org>
Commit-Queue: Toby Huang <tobyhuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#758654}
parent f7789d6a
...@@ -129,7 +129,7 @@ void SupervisedUserService::RegisterProfilePrefs( ...@@ -129,7 +129,7 @@ void SupervisedUserService::RegisterProfilePrefs(
prefs::kSupervisedUserExtensionsMayRequestPermissions, false); prefs::kSupervisedUserExtensionsMayRequestPermissions, false);
registry->RegisterDictionaryPref( registry->RegisterDictionaryPref(
prefs::kSupervisedUserApprovedExtensions, prefs::kSupervisedUserApprovedExtensions,
user_prefs::PrefRegistrySyncable::SYNCABLE_OS_PREF); user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
#endif #endif
registry->RegisterDictionaryPref(prefs::kSupervisedUserManualHosts); registry->RegisterDictionaryPref(prefs::kSupervisedUserManualHosts);
registry->RegisterDictionaryPref(prefs::kSupervisedUserManualURLs); registry->RegisterDictionaryPref(prefs::kSupervisedUserManualURLs);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment