59 bytes, text/x-review-board-request
Part 1 of bug 1200494 (https://hg.mozilla.org/mozilla-central/rev/e07ec12de6c0) needs to be backed out again before bug 1343744 hits Beta/Release, otherwise we'll break the downloads list (and xpcshell tests as well).
Assignee: nobody → jh+bugzilla
See Also: → 1344543
I've tried disabling the Intl API on a try build to check that the downloadutils xpcshell tests still work as expected (since they previously weren't enabled), but I'm only getting a broken build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2526287a0ae6f91bf736c726d9f79d56c649e150
Download xpcshell tests still pass: https://treeherder.mozilla.org/#/jobs?repo=try&revision=68413037ea33b83c9cd2ecec8ebe8212fa28786f
Comment on attachment 8847241 [details] Bug 1344143 - Backing out changeset e07ec12de6c0 (Bug 1200494). https://reviewboard.mozilla.org/r/120242/#review122240
Attachment #8847241 - Flags: review?(mak77) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/b57650ab9be8 Backing out changeset e07ec12de6c0 (Bug 1200494). r=mak
https://hg.mozilla.org/mozilla-central/rev/b57650ab9be8 backed also from m-c
Could you request uplift to aurora? Fennec 54 will still turn off ICU...
Comment on attachment 8847241 [details] Bug 1344143 - Backing out changeset e07ec12de6c0 (Bug 1200494). Approval Request Comment [Feature/Bug causing the regression]: Bug 1343725 [User impact if declined]: Intl API is not coming to Beta/Release after all for the time being - without this patch, the download manager will be broken. [Is this code covered by automated tests?]: Yes. [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]: none [Is the change risky?]: No. [Why is the change risky/not risky?]: Straight backout of bug 1200494 except for adjusted bug numbers for followup. [String changes made/needed]: none
Attachment #8847241 - Flags: approval-mozilla-aurora?
Maybe this needs marking as fixed for the uplift request to show up wherever it needs to show up?
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Comment on attachment 8847241 [details] Bug 1344143 - Backing out changeset e07ec12de6c0 (Bug 1200494). Backout changes to avoid breaking download manager. Aurora54+.
Attachment #8847241 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.