Closed Bug 1458162 Opened 2 years ago Closed 2 years ago
Fix test test
_plugin Info URL .js fix and change blocklist preferences to thunderbird .net redirects
In Bug 1457745 we removed the Thunderbird-specific blocklist prefs. In the future there's a chance we'll be hosting the blocklist ourselves, and pointing directly at the Mozilla cdn makes this more difficult to change. I've set up some redirects in https://github.com/thundernest/live-redirects/commit/857f0cb650f4b5adfde248dec9f33071cddb5f9c and if we use these instead, changing where Thunderbird looks for this data is much faster and simpler.
Assignee: nobody → sancus
This fixes the test so it depends on the pref and isn't hardcoded.
Comment on attachment 8972251 [details] [diff] [review] test_infoURL_missing fix I'm not sure if you're the right person to r? for this, Gijs, but if not could you redirect it? And of course let me know if there's any issue with altering the test in this way.
Attachment #8972251 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8972248 [details] [diff] [review] Bug_1458162.patch Thanks for picking this up so quickly. I hope all those URLs work ;-)
Oh, you need to put the bug number into the commit message: Bug xxx - such and such. r=yyy No worries for the C-C patch, I'll fix that, but for M-C you need to upload a new patch. You can do it after the review and add the reviewer. You then set r+ yourself and write: Carrying over yyy's r+.
Ah OK, no problem. And yes I hope the URLs work too ;)
Comment on attachment 8972251 [details] [diff] [review] test_infoURL_missing fix Review of attachment 8972251 [details] [diff] [review]: ----------------------------------------------------------------- LGTM with the commit message fixed to include a bug number etc.
Attachment #8972251 - Flags: review?(gijskruitbosch+bugs) → review+
Thanks. I've updated the commit message as requested to include Bug number and r=gijs. Will need someone to check it in for me, of course.
Attachment #8972251 - Attachment is obsolete: true
Comment on attachment 8972258 [details] [diff] [review] mozilla-central test_infoURL_missing fix Carrying forward Gijs' r+
Attachment #8972258 - Flags: review+
Dear sheriff, please land the mozilla-central part for us.
Component: Preferences → Blocklisting
Product: Thunderbird → Toolkit
Summary: Change blocklist preferences to thunderbird.net redirects → Fix test test_pluginInfoURL.js fix and change blocklist preferences to thunderbird.net redirects
Version: unspecified → Trunk
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/6876bbcd3c3f Change xpcshell test toolkit/mozapps/extensions/test/xpcshell/test_pluginInfoURL.js to use blocklist pref for test_infoURL_missing. r=Gijs
Pushed by email@example.com: https://hg.mozilla.org/comm-central/rev/479f02e12da2 Change blocklist prefs to use live.thunderbird.net redirects. r=jorgk DONTBUILD
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Component: Blocklist Policy Requests → Blocklist Implementation
You need to log in before you can comment on or make changes to this bug.