Open
Bug 1289947
Opened 8 years ago
Updated 2 years ago
Logins not saved on www.icloud.com
Categories
(Toolkit :: Password Manager: Site Compatibility, defect, P3)
Toolkit
Password Manager: Site Compatibility
Tracking
()
NEW
People
(Reporter: MattN, Unassigned)
References
(Depends on 1 open bug, )
Details
https://www.icloud.com/ uses an <iframe> for the login form and doesn't use a <form> inside of that but it also doesn't trigger capture from navigation (bug 1166947).
Comment 1•6 years ago
|
||
Matt, any thoughts on if this is fixable. I see at least one user trying to use FF57 running into this: https://twitter.com/lorriemeyer/status/931623182691270656
Flags: needinfo?(MattN+bmo)
Reporter | ||
Comment 2•6 years ago
|
||
It probably is fixable but unfortunately password manager isn't being actively worked on now or H1 next year so unless that changes this is unlikely to get fixed any time soon.
Flags: needinfo?(MattN+bmo)
Comment 3•6 years ago
|
||
The user actually reported to me its working on icloud now. Maybe the service changed something. I don't have an account so I can't easily test myself.
Comment hidden (off-topic) |
Reporter | ||
Comment 6•5 years ago
|
||
It looks like iCloud isn't using pushState/replaceState but is instead manipulating window.location.hash
upon login and therefore aFlags & Ci.nsIWebProgressListener.LOCATION_CHANGE_SAME_DOCUMENT
is 1 while aWebProgress.loadType & Ci.nsIDocShell.LOAD_CMD_PUSHSTATE
is 0.
onLocationChange (LoginManagerContent.jsm#63)
hashDidChange (javascript-packed.js#903) => window.location.hash!=="#"+e&&(window.location.hash=e)
_notifyPropertyObservers (javascript-packed.js#73)
endPropertyChanges (javascript-packed.js#69)
clearHash (javascript-packed.js#902)
enterState (javascript-packed.js#958)
enterState (javascript-packed.js#440)
_enterState (javascript-packed.js#440)
_executeGotoStateActions (javascript-packed.js#439)
gotoState (javascript-packed.js#439)
_flushPendingStateTransition (javascript-packed.js#443)
_cleanupStateTransition (javascript-packed.js#440)
_executeGotoStateActions (javascript-packed.js#440)
gotoState (javascript-packed.js#439)
redirectToState (javascript-packed.js#909)
checkIfDefaultAppIsCreated (javascript-packed.js#908)
languageDidLoad (javascript-packed.js#908)
_invokeCallback (javascript-packed.js#272)
run (javascript-packed.js#122)
_invokeCallback (javascript-packed.js#272)
loadModule (javascript-packed.js#269)
userDidAuthenticate (javascript-packed.js#910)
authDidValidate (javascript-packed.js#816)
…
Priority: P3 → P2
Comment hidden (off-topic) |
Reporter | ||
Updated•5 years ago
|
Flags: qe-verify+
Comment 8•2 years ago
|
||
Moving to P3 as this is not happening in the next release cycle.
Priority: P2 → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•