Closed Bug 1445028 Opened 2 years ago Closed 2 years ago
about:preferences#general is broken if identity
.fxaccounts .enabled=false on Firefox Developer Edition
59 bytes, text/x-review-board-request
From bug 1444564 comment 0, > 'Firefox Developer Edition Updates' section doesn't work. 'Check for updates' > button causes error > > TypeError: gAppUpdater is undefined preferences:1:1 > It is because of bug 1434706. > https://hg.mozilla.org/mozilla-central/diff/109cd0a34ffe/browser/components/preferences/in-content/main.js#l1.14
Sorry about that, patch incoming.
Assignee: nobody → eoger
Status: NEW → ASSIGNED
Comment on attachment 8958468 [details] Bug 1445028 - Hide sync-dev-edition-root properly if sync is disabled. https://reviewboard.mozilla.org/r/227416/#review233206
Attachment #8958468 - Flags: review?(jaws) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/4d6480b1c320 Hide sync-dev-edition-root properly if sync is disabled. r=jaws
Comment on attachment 8958468 [details] Bug 1445028 - Hide sync-dev-edition-root properly if sync is disabled. Approval Request Comment [Feature/Bug causing the regression]: bug 1434706 [User impact if declined]: about:preferences will be broken for users with Sync enabled even if not logged (aka 99% of the users) on Dev Edition. [Is this code covered by automated tests?]: Don't think so. [Has the fix been verified in Nightly?]: Yes. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: bug 1443593 [Is the change risky?]: No. [Why is the change risky/not risky?]: These are fairly small changes. [String changes made/needed]: -
Attachment #8958468 - Flags: approval-mozilla-beta?
Comment on attachment 8958468 [details] Bug 1445028 - Hide sync-dev-edition-root properly if sync is disabled. regression fix for about:preferences on dev edition, beta60+, should be in 60.0b4
Attachment #8958468 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
I've reproduced this bug on DevEdition 60.0b2 (20180307142707) by setting the "identity.fxaccounts.enabled" pref to false and following the steps from comment 0. I can no longer see this error: "TypeError: gAppUpdater is undefined preferences:1:1", in the browser console after checking the updates via Check for updates button. Tested on DevEdition 60.0b5 (20180319175655) across platforms: Windows 10 x64, MacOS X 10.11 and Ubuntu 16.04 x64.
This bug is also verified as fixed, on DevEdition 61.0b10 (20180530184300) running Win 10 x64, macOS 10.13 and Ubuntu 16.04 x64.
You need to log in before you can comment on or make changes to this bug.