1574907 - Saved entry does not display password at field level when it's being set from multiple entries

Status: RESOLVED WORKSFORME

(Toolkit :: Password Manager, defect, P3)

Description

[Diana Rus]

Environment
Device: Sony Xperia Z5 (Android 7.0)
Build: ESR Firefox Nightly 68.1a1(2019-08-17) Firefox Beta 69.0b15, Firefox Nightly 70.0a1(2019-08-19)

Steps to reproduce:

1. Go to "goo.gl/vi7z6" and insert a username and password.
2. Tap on "Login" button.
3. Choose from the triggered doorhanger the "Remember" option.
4. Insert a different username and password.
5. Tap on "Login" button.
6. From the triggered doorhanger choose "Remember".
7. Open page "goo.gl/vi7z6" in a different tab.
8. Tap on the username field and choose an entry.

Expected Result: The password field contains the saved password for the login entry.

Actual Result: The password field remains empty after choosing the entry.

Note

1. The issue reproduces on tested builds: Firefox Beta 69.0b15, Firefox Nightly 70.0a1(2019-08-19)
2. The issue does not reproduce on ESR Firefox Nightly 68.1a1(2019-08-17)

Comment 1

[Matthew N. [:MattN]]

[Tracking Requested - why for this release]: Sounds like a bad regression for login autocomplete on Android.

(In reply to Diana Rus from comment #0)

Thank you for filing this.

…
8. Tap on the username field and choose an entry.

Expected Result: The password field contains the saved password for the login entry.

Actual Result: The password field remains empty after choosing the entry.

So an autocomplete popup does show up for the username field but when you tap on the username row it doesn't fill the password?

Note

1. The issue reproduces on tested builds: Firefox Beta 69.0b15, Firefox Nightly 70.0a1(2019-08-19)
2. The issue does not reproduce on ESR Firefox Nightly 68.1a1(2019-08-17)

Is it possible to get a regression range from mozilla-central builds?

Would it be possible to get testing with the preference signon.debug enabled. I guess it would log to adb logcat?

Comment 2

[Diana Rus]

Attachment: LogCat-Android.txt

Comment 3

[Diana Rus]

Hi,
I have also checked the issue on Firefox Release 68.0.2 with Sony Xperia Z5 (Android 7.0) and does not reproduces so I will update the status for firefox-68 as unaffected.
My last info about moz-regression is that the tool stops at a build dated on 2019-05-16.
What I can provide you is a logcat. I wil try and be as minimal as I can with file size and the actions that need to be present.
Attached you have the logcat "LogCat-Android.txt", please check and see whether this helps in your investigations.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Do we have any reason to believe that this affects all mobile browsers or just Fennec?

Comment 5

[Matthew N. [:MattN]]

Fennec and desktop are the only apps that uses the whole Toolkit::Password Manager at the moment. Fenix only uses it for autofill heuristics, not autocomplete, so it shouldn't be affected by this specific bug though.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Given that Fennec ships off ESR68 now, it doesn't sound like we need to track this then.

Comment 7

[Matthew N. [:MattN]]

Does that mean that Fennec will never get the broken code of Fx69? i.e. Can we WONTFIX this bug?

Comment 8

[Ryan VanderMeulen [:RyanVM]]

That's correct, assuming there's no chance of anything GeckoView-based ever hitting this.

Comment 9

[Matthew N. [:MattN]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #8)

That's correct, assuming there's no chance of anything GeckoView-based ever hitting this.

As of now, GV isn't using autocomplete but IMO they should eventually though those discussions haven't really happened much yet…

I will move this to P2 for now then so it's still on the radar in case GV ends up using this.

Comment 10

[Sergey Galich [:serg]]

Moving to P3 as this is not happening in the next release cycle.