Allow address autofill to be enabled per country independently of credit card au...
source link: https://bugzilla.mozilla.org/show_bug.cgi?id=1745248
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
Allow address autofill to be enabled per country independently of credit card autofill
Categories
(Toolkit :: Form Autofill, enhancement, P1)
Tracking
(bug has been fixed and VERIFIED)
99 Branch
People
(Reporter: tgiles, Assigned: tgiles)
References
Details
Currently adding a new supported country to extensions.formautofill.supportedCountries
will enable both credit card autofill and address autofill for that country. However credit card autofill and address autofill should be able to be enabled independently from each other as we enable Form Autofill for more countries.
The current idea is to create a new pref extensions.formautofill.addresses.supportedCountries
so that we can ensure non-tested countries do not have address autofill enabled by default. The address autofill feature will still be available in about:preferences
but will not be supported until we have an opportunity to test and fix issues of the feature in these future countries.
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/b1808ec2fe97 Allow each autofill feature to detect if it should be enabled depending on search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/8aeb5a348950 Allow each autofill feature to hide its prefs if the feature is not supported in the search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/ef2454116a21 Fix tests from removing the form autofill module pref and allowing each feature to determine if it should be available or not. r=dimi,sgalich
Backed out for causing Mochitests with software webrender enabled failures in browser/extensions/formautofill/test/browser/browser_first_time_use_doorhanger.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/4ea700301581f02192a46f6fa2b4e568b2c34bd9
Alright, fixed the failing tests. Let's try this again.
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/03dd6cedf996 Allow each autofill feature to detect if it should be enabled depending on search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/b09a96820c2c Allow each autofill feature to hide its prefs if the feature is not supported in the search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/b39b99edef97 Fix tests from removing the form autofill module pref and allowing each feature to determine if it should be available or not. r=dimi,sgalich
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/06d57e874522 Allow each autofill feature to detect if it should be enabled depending on search region. r=dimi,sgalich,preferences-reviewers,skhamis,LougeniaBailey https://hg.mozilla.org/integration/autoland/rev/c7212ce535e4 Allow each autofill feature to hide its prefs if the feature is not supported in the search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/f54ce6b7dd26 Fix tests from removing the form autofill module pref and allowing each feature to determine if it should be available or not. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/324a6acb8612 Add form autofill pref migration and tests. r=dimi,sgalich
Backed out for causing issues with credit card sync.
(In reply to Iulian Moraru from comment #19)
Backed out for causing issues with credit card sync.
Thanks for that, but I think we also will want another patch to "reserve" UIVersion 123 - IIUC, any profiles where this has run will have set the "browser.migration.version" pref to 123 - so when someone else comes along and adds the next migration, they will use 123 again, but those profiles will not run it. A comment noting that 123 should not be reused will probably suffice.
Steps to verify this bug:
- Ensure region detection works as expected for address and credit card autofill. This can be done by the following:
- Set
browser.search.region
to "US" - Set
extensions.formautofill.addresses.supported
to "detect" - Set
extensions.formautofill.creditCards.supported
to "detect" - Restart browser
- Navigate to
about:preferences
and search for "Saved Credit Cards" and "Saved Addresses" - The two buttons for management dialogs should appear and be available for use
- Set
browser.search.region
to "DE" - Set
extensions.formautofill.creditCards.supportedCountries
to "US,CA,DE" - Restart browser
- Navigate to
about:preferences
and search for "Saved Credit Cards" and "Saved Addresses" - Only "Saved Credit Cards" should be available to use, the "Saved Addresses" button shouldn't exist at this point.
- Set
extensions.formautofill.creditCards.supportedCountries
back to "US,CA" and setextensions.formautofill.addresses.supportedCountries
to "US,CA,DE" - Restart the browser
- Navigate to
about:preferences
and search for "Saved Credit Cards" and "Saved Addresses" - Only "Saved Addresses" should be available to use, the "Saved Credit Cards" button shouldn't exist at this point.
- Set
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/229e5309bc92 Allow each autofill feature to detect if it should be enabled depending on search region. r=dimi,sgalich,preferences-reviewers,skhamis,LougeniaBailey https://hg.mozilla.org/integration/autoland/rev/7ceca0dff062 Allow each autofill feature to hide its prefs if the feature is not supported in the search region. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/76ddec1f1d48 Fix tests from removing the form autofill module pref and allowing each feature to determine if it should be available or not. r=dimi,sgalich https://hg.mozilla.org/integration/autoland/rev/104a9dfbecb6 Add form autofill pref migration and tests. r=dimi,sgalich
Verified - Fixed in latest Nightly 99.0a1 (build id: 20220210213101), everything is working as expected. We verified the bug following the steps provided in comment 23 and also we performed exploratory testing around those prefs using different locale builds.
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK