This is an automated email from the git hooks/post-receive script.
pierov pushed a commit to branch geckoview-99.0.1-11.0-1 in repository tor-browser.
commit 89c5488ab6eff5536eb1a835665894720741bd2a Author: Drew Willcoxon adw@mozilla.com AuthorDate: Sat Jan 15 20:22:34 2022 +0000
Bug 1749758 - Disable session store for the Firefox Suggest toggles/checkboxes in about:preferences. r=Gijs, a=RyanVM,dsmith
Please see bug 1749758 comment 6 for background. In summary, session store is restoring the states of the Firefox Suggest `html:input` checkboxes in about:preferences#privacy, which intermittently happens **after** their states are initialized by the preferences code. If a checkbox's session store state does not match the value of its underlying pref, then the checkbox and pref both end up with the wrong value because session store sets the wrong checked state of the checkbox, which fires an input event, which then updates the pref.
This can only happen when the pref value is changed outside of about:preferences, which is indeed the case for the Firefox Suggest pref discussed in the bug.
To fix it, I tried adding `autocomplete=off` to each input, but unfortunately that does not work because `input[type=checkbox]` is specifically not allowed to opt out of autocomplete/session store. The code path for restoring input checkboxes is:
# [CollectInputElement](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) # CollectInputElement [calls](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) [nsContentUtils::IsAutocompleteEnabled](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) # nsContentUtils::IsAutocompleteEnabled [calls](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) [HTMLInputElement::GetAutocomplete](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) # HTMLInputElement::GetAutocomplete [calls](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) [HTMLInputElement::DoesAutocompleteApply](https://searchfox.org/mozilla-central/source/dom/html/HTMLInputElement.cpp#6...) # HTMLInputElement::DoesAutocompleteApply [returns false](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) for FormControlType::InputCheckbox # Back in nsContentUtils::IsAutocompleteEnabled, the `autocomplete` string is empty because HTMLInputElement::DoesAutocompleteApply did not set it to anything, so then the method [checks](https://searchfox.org/mozilla-central/rev/3de56eb5f266f523340e739ae1b53258e0...) if the input has a form and sets `autocomplete` to the value of the form's autocomplete attribute # nsContentUtils::IsAutocompleteEnabled then returns false iff `autocomplete` is "off" # Back in CollectInputElement, if nsContentUtils::IsAutocompleteEnabled returned true, then it continues and checks a couple of other things, but at that point there's no way for the input checkbox to opt out of autocomplete/session store.
So, the only way to disable autocomplete seems to be to associate the input with a form and set `autocomplete=off` on the form, so that's what I've done.
It would be simpler to make these XUL checkboxes instead, which AFAICT don't participate in session store, but these checkboxes need the toggle-switch styling.
Finally, this is a problem for all `html:input`s used in about:preferences. I didn't update any others because I want to keep this scoped for uplift, and maybe the other input checkboxes should be XUL checkboxes anyway?
Differential Revision: https://phabricator.services.mozilla.com/D136054 --- browser/components/preferences/privacy.inc.xhtml | 7 +++++++ browser/themes/shared/preferences/privacy.css | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/browser/components/preferences/privacy.inc.xhtml b/browser/components/preferences/privacy.inc.xhtml index 0db2f3f8c1fae..edfd3c716c2f9 100644 --- a/browser/components/preferences/privacy.inc.xhtml +++ b/browser/components/preferences/privacy.inc.xhtml @@ -651,6 +651,10 @@ <checkbox id="enginesSuggestion" data-l10n-id="addressbar-locbar-engines-option" preference="browser.urlbar.suggest.engines"/> <vbox id="firefoxSuggestContainer" hidden="true"> + <!-- This form[autocomplete="off"] is necessary to prevent the inputs from + participating in session restore. autocomplete="off" is ignored on the + inputs themselves. --> + <html:form id="firefoxSuggestForm" autocomplete="off" hidden="true"/> <vbox class="firefoxSuggestOptionBox"> <hbox align="center"> <label id="firefoxSuggestNonsponsoredLabel" @@ -660,6 +664,7 @@ type="checkbox" class="toggle-button firefoxSuggestToggle" preference="browser.urlbar.suggest.quicksuggest.nonsponsored" + form="firefoxSuggestForm" aria-labelledby="firefoxSuggestNonsponsoredLabel" aria-describedby="firefoxSuggestNonsponsoredDescription"/> </hbox> @@ -676,6 +681,7 @@ type="checkbox" class="toggle-button firefoxSuggestToggle" preference="browser.urlbar.suggest.quicksuggest.sponsored" + form="firefoxSuggestForm" aria-labelledby="firefoxSuggestSponsoredLabel" aria-describedby="firefoxSuggestSponsoredDescription"/> </hbox> @@ -692,6 +698,7 @@ type="checkbox" class="toggle-button firefoxSuggestToggle" preference="browser.urlbar.quicksuggest.dataCollection.enabled" + form="firefoxSuggestForm" aria-labelledby="firefoxSuggestDataCollectionLabel" aria-describedby="firefoxSuggestDataCollectionDescription"/> </hbox> diff --git a/browser/themes/shared/preferences/privacy.css b/browser/themes/shared/preferences/privacy.css index a9e7f16cc5f14..30927d3137301 100644 --- a/browser/themes/shared/preferences/privacy.css +++ b/browser/themes/shared/preferences/privacy.css @@ -332,7 +332,7 @@ margin-block-start: 11px; }
-.firefoxSuggestOptionBox:first-child { +.firefoxSuggestOptionBox:first-of-type { /* Similar here: Make the apparent vertical space between the last checkbox and first option box the same as elsewhere. */ margin-block-start: 5px;