1913711 - [ES][pccomponentes] Autofill is not working properly on several of the address form fields

Status: VERIFIED FIXED

(Toolkit :: Form Autofill, task, P3)

Description

[Anca Soncutean, Desktop QA]

Found in

• 131.0a1

Affected versions

• 131.0a1

Tested platforms

• Windows 11

Preconditions

• Install Firefox es-ES build
• Have set in about:config:
  * browser.search.region - ES
  * extensions.formautofill.addresses.experiments.enabled - True
  * extensions.formautofill.addresses.supportedCountries - ES
• Use ES VPN

Steps to reproduce

1. Go to https://www.pccomponentes.com
2. Add an item to Cart and navigate to the checkout page
3. Autofill the address form and observed the populated fields

Expected result

• Form autofill/preview/clear are correctly applied to all eligible fields based on the saved data.

Actual result

• Preview is not working for the Province field
• Preview and autofill work independently: Telephone field by itself, Postal code and City separately and the the rest of the fields separately as well
• Preview and Clear functions don't work for the Street number, but it will be autofilled if accessed directly from this field, but note that it will retain the previous info submitted, that was never saved through Address Capture doorhanger ( see bug 1913697), no info from about:preferences will be displayed.
• Additional street info seams to have the main role for the street info here, the preview/autofill/clear works but the information displayed is not for additional info, and for the main street info.

Regression range

• N/A

Additional notes

• The only issue on Chrome is for the Province field, where previews doesn't work as well.
• Fathom sample.
• Clear will delete all fields that are populated regardless of the field and if the preview/autofill works independently.

Comment 1

[Anca Soncutean, Desktop QA]

Attachment: pccomponantes - screencast issue.gif

Comment 2

[Neil Deakin]

Attachment: Reduced Testcase

Comment 3

[Neil Deakin]

Attachment: Bug 1913711, remove 'provincia' from address-level2, and null-check isoids, r=dimi,#credential-management-reviewers

Remove the word 'provincia' from address-level2 since it should match address-level1, and already exists there, also null-check isoids when comparing province codes since some countries such as Spain do not have them defined in the metadata.

This doesn't affect the actual site for this bug since it uses a custom dropdown for province with a real hidden <select> behind the scenes, but it does fix filling in that hidden <select> as shown in the reduced testcase.

Comment 4

[Neil Deakin]

Attachment: Bug 1913711, only check the alternative name for house number if the field hasn't already been changed by a previous heuristic, as in this case _parseHouseNumberFields already updated the field, r=dimi,#credential-management-reviewers

Depends on D255122

Comment 5

[Neil Deakin]

Attachment: Bug 1913711, add a test for address-line1 and province being incorrectly identified, r=dimi,#credential-management-reviewers

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/8af6cfeb10f3
https://hg.mozilla.org/integration/autoland/rev/2685b405310f
add a test for address-line1 and province being incorrectly identified, r=dimi,credential-management-reviewers

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/985ebfa111c6
https://hg.mozilla.org/integration/autoland/rev/5caf8d2ce2e1
Revert "Bug 1913711, add a test for address-line1 and province being incorrectly identified, r=dimi,credential-management-reviewers" for causing bc failures @ browser_autofill_address_housenumber.js

Comment 8

[amarc]

Backed out for causing bc failures @ browser_autofill_address_housenumber.js

• Backout link
• Push with failures
• Failure Log

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/ecfa5739b6aa
https://hg.mozilla.org/integration/autoland/rev/70c9ba781adb
remove 'provincia' from address-level2, and null-check isoids, r=dimi,credential-management-reviewers
https://github.com/mozilla-firefox/firefox/commit/37da62121998
https://hg.mozilla.org/integration/autoland/rev/56bda26c98ae
only check the alternative name for house number if the field hasn't already been changed by a previous heuristic, as in this case _parseHouseNumberFields already updated the field, r=dimi,credential-management-reviewers
https://github.com/mozilla-firefox/firefox/commit/10d28523252a
https://hg.mozilla.org/integration/autoland/rev/e2282b93feac
add a test for address-line1 and province being incorrectly identified, r=dimi,credential-management-reviewers

Comment 10

[agoloman]

https://hg.mozilla.org/mozilla-central/rev/70c9ba781adb
https://hg.mozilla.org/mozilla-central/rev/56bda26c98ae
https://hg.mozilla.org/mozilla-central/rev/e2282b93feac

Comment 11

[Alin Ilea, Desktop Test Engineering]

Attachment: 2025-06-30_11h11_32.mp4

Hi Neil,
I just verified this in the latest Nightly build 142.0a1 (2025-06-29), and all the eligible fields are now autofilled, except the Province one(drop-down field), but the autofill preview(which worked before) is no longer displayed in any field (see the screen recording attached in this comment).
Just let me know if we can consider this bug as fixed and mark it as verified accordingly and file a new bug for the preview issue, or we should re-open this one?
Thanks!

Comment 12

[Neil Deakin]

The preview seems to only be an issue for the first address you select on the dropdown menu. It is probably related to the manner in which the site changes the placeholder when the field is focused. It is very minor and unlikely that we will try to fix this edge case for this one site so I don't think a bug is worth filing for this.

Comment 13

[Alin Ilea, Desktop Test Engineering]

Thanks for the answer. I will mark this as verified.