Closed Bug 997625 Opened 7 years ago Closed 7 years ago
Sync panel: Checkbox for syncing passwords enabled (not disabled) while sync credentials need reauth
Latest Nightly on Windows 8.1 Regression from bug 970167. When the user has successfully set up sync locally but syncing fails on the server side (e.g. password changed on the server but not locally), the checkboxes what kind of data should be sync become disabled except the password checkbox.
Try run with 2 unrelated WebGL oranges: https://tbpl.mozilla.org/?tree=Try&showall=1&rev=3ff966028851
Attachment #8408178 - Flags: review?(ttaubert)
Comment on attachment 8408178 [details] [diff] [review] patch, v2 Review of attachment 8408178 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #8408178 - Flags: review?(ttaubert) → review+
7 years ago
Tracking for 30 and 31. Do you want to have in 29?
Yes, I think it would be really great to have this in 29. It's a very visible glitch and a very trivial fix.
Comment on attachment 8408178 [details] [diff] [review] patch, v2 [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 970167 Testing completed (on m-c, etc.): Manual smoke testing. Risk to taking this patch (and alternatives if risky): Low risk. String or IDL/UUID changes made by this patch: None. User impact if declined: Without this patch the checkbox for the passwords sync engine will always be enabled. When signed in with an unverified email address all checkboxes are disabled except the passwords engine, which looks really off and is very visible.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Verified fixed in 29, 30, and 31 on Windows 8.1 x64
Status: RESOLVED → VERIFIED
Whiteboard: [good first verify] → [good first verify] [testday-20140509]
I can also confirm that this is verified.
You need to log in before you can comment on or make changes to this bug.