Bug 1752332: Rename ShouldSyncPreference to ShouldSanitizePreference r=KrisWright
This simplifies the number of negations needed, and makes things easy to understand. I think anyway; I know that without renaming it I made several annoying-to-diagnose negation errors... Depends on D141411 Differential Revision: https://phabricator.services.mozilla.com/D141412
This commit is contained in:
@@ -95,7 +95,7 @@ void UtilityProcessManager::OnPreferenceChange(const char16_t* aData) {
|
||||
NS_LossyConvertUTF16toASCII strData(aData);
|
||||
|
||||
// A pref changed. If it is useful to do so, inform child processes.
|
||||
if (!ShouldSyncPreference(strData.Data(), false)) {
|
||||
if (ShouldSanitizePreference(strData.Data(), false)) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user