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)
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
Updated•8 years ago
|
tracking-fennec: --- → ?
Priority: -- → P1
Whiteboard: [TPE-1]
Comment 1•8 years ago
|
||
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)
Updated•8 years ago
|
tracking-fennec: ? → 51+
Should this be changed to Toolkit::Password Manager to match bug 1166947?
Comment 3•8 years ago
|
||
(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+ → ---
tracking-firefox51:
--- → ?
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]
Comment 4•8 years ago
|
||
(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
Comment 5•8 years ago
|
||
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)
Comment 7•8 years ago
|
||
Ok I simply can't reproduce on central either. What device are you using? I can repro on neither Nexus 5 nor AVD emulator.
Updated•8 years ago
|
Flags: needinfo?(teodora.vermesan)
Comment 8•8 years ago
|
||
And can you still reproduce this on the latest Nightly?
Reporter | ||
Comment 9•8 years ago
|
||
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)
Comment 10•8 years ago
|
||
Thanks!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Updated•8 years ago
|
tracking-firefox51:
+ → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•