Closed Bug 1297019 Opened 8 years ago Closed 8 years ago

The Password Doorhanger is not displayed on Android

Categories

(Toolkit :: Password Manager, defect, P1)

51 Branch
ARM
Android
defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox50 --- unaffected
firefox51 --- affected

People

(Reporter: TeoVermesan, Unassigned)

Details

(Keywords: regression)

Steps to reproduce:
1. Go to goo.gl/vi7z6
2. Fill in username and password
3. Login

Expected results:
- A doorhanger asking to remember or not the password is displayed. 

Actual results:
- The doorhanger is not displayed

Note:
- regression window:
good build: 17-08
bad build: 18-08
pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=fe895421dfbe1f1f8f1fc6a39bb20774423a6d74&tochange=97a52326b06a07930216ebefa5af333271578904
tracking-fennec: --- → ?
Priority: -- → P1
Whiteboard: [TPE-1]
I wonder if this is the same as bug 1279924, which is depending on bug 1287202 for a fix? This page also uses a <form>, but I don't have enough knowledge to know if that's relevant.

Matt: does this seem correct?
Flags: needinfo?(MattN+bmo)
tracking-fennec: ? → 51+
Should this be changed to Toolkit::Password Manager to match bug 1166947?
(In reply to Andrzej Hunt :ahunt from comment #1)
> I wonder if this is the same as bug 1279924, which is depending on bug
> 1287202 for a fix? This page also uses a <form>, but I don't have enough
> knowledge to know if that's relevant.

No, bug 1279924 is about a page that we never handled properly whereas this bug is about a regression and uses a regular form submit event.

I'm confused about this being broken as Johann is working on this code in bug 1266836 and it seems to be working on Android (though perhaps he fixed this issue in that bug).

Johann, can you confirm this is broken for you?
tracking-fennec: 51+ → ---
Component: Logins, Passwords and Form Fill → Password Manager
Flags: needinfo?(MattN+bmo) → needinfo?(jhofmann)
Product: Firefox for Android → Toolkit
Summary: The Password Doorhanger is not displayed → The Password Doorhanger is not displayed on Android
Whiteboard: [TPE-1]
(It's possible this is still a front-end Java/Android issue but I'm not sure).

GeckoConsole and pwmgr Debug Logging[1] output would help

[1] https://wiki.mozilla.org/Firefox:Password_Manager_Debugging
Works fine with my patch. I'll dig into it to find out what the error is when I revert to central, to check if/how I fixed this.
Flags: needinfo?(jhofmann)
Tracking 51+ for this regression.
Ok I simply can't reproduce on central either. What device are you using? I can repro on neither Nexus 5 nor AVD emulator.
Flags: needinfo?(teodora.vermesan)
And can you still reproduce this on the latest Nightly?
I cannot reproduce it anymore using xiaomi mi i4 - Android 5.0.2 and Nexus 7 - Android 6.0.
The password doorhanger is displayed after trying to login.
Flags: needinfo?(teodora.vermesan)
Thanks!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.