Fix test test_pluginInfoURL.js fix and change blocklist preferences to thunderbird.net redirects

RESOLVED FIXED in mozilla61

Status

()

RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: sancus, Assigned: sancus)

Tracking

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

11 months ago
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)

Comment 1

11 months ago
Assignee: nobody → sancus
(Assignee)

Comment 2

11 months ago
Posted patch test_infoURL_missing fix (obsolete) — Splinter Review
This fixes the test so it depends on the pref and isn't hardcoded.
(Assignee)

Updated

11 months ago
Attachment #8972248 - Flags: review?(jorgk)
Attachment #8972248 - Flags: approval-comm-beta?
(Assignee)

Comment 3

11 months ago
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 4

11 months ago
Comment on attachment 8972248 [details] [diff] [review]
Bug_1458162.patch

Thanks for picking this up so quickly. I hope all those URLs work ;-)
Attachment #8972248 - Flags: review?(jorgk)
Attachment #8972248 - Flags: review+
Attachment #8972248 - Flags: approval-comm-beta?
Attachment #8972248 - Flags: approval-comm-beta+

Comment 5

11 months ago
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+.
(Assignee)

Comment 6

11 months ago
Ah OK, no problem. And yes I hope the URLs work too ;)

Comment 7

11 months ago
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+
(Assignee)

Comment 8

11 months ago
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
(Assignee)

Updated

11 months ago
Keywords: checkin-needed

Comment 9

11 months ago
Comment on attachment 8972258 [details] [diff] [review]
mozilla-central test_infoURL_missing fix

Carrying forward Gijs' r+
Attachment #8972258 - Flags: review+

Comment 10

11 months ago
Dear sheriff, please land the mozilla-central part for us.
Component: Preferences → Blocklisting
Flags: approval-comm-beta+
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

Comment 11

11 months ago
Pushed by gijskruitbosch@gmail.com:
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
Keywords: checkin-needed

Comment 12

11 months ago
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/479f02e12da2
Change blocklist prefs to use live.thunderbird.net redirects. r=jorgk DONTBUILD
Status: NEW → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED

Updated

11 months ago
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.