Port bug 1363421 |UserAgentOverrides.jsm is visible on startup profiles| to C-C

RESOLVED FIXED in Thunderbird 55.0

Status

RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jorgk, Assigned: Paenglab)

Tracking

Trunk
Thunderbird 55.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

a year ago
+++ This bug was initially created as a clone of Bug #1363421 +++

This looks like rehashing what was done for bug 1352381:
https://hg.mozilla.org/comm-central/rev/0e469cfeb978ddc6a9e6114b4da3ecedd870ff14

M-C did this:
https://hg.mozilla.org/integration/autoland/rev/bc372bd6dff3

So it looks like we rip out what was done in bug 1352381 and add this instead:
+@RESPATH@/components/UAOverridesBootstrapper.js
+@RESPATH@/components/UAOverridesBootstrapper.manifest
(Assignee)

Comment 1

a year ago
Created attachment 8869747 [details] [diff] [review]
Bug1366499.patch

This should work for TB. SM has still a:

    if (Services.prefs.getBoolPref("general.useragent.complexOverride.moodle"))
      UserAgentOverrides.addComplexOverride(addMoodleOverride);

I think this needs to be removed too. But then the pref is also no more needed.
Frank-Rainer, what should I do?

IB has no UserAgentOverrides, I also don't add the lines to package-manifest.
Attachment #8869747 - Flags: review?(jorgk)
Attachment #8869747 - Flags: feedback?(frgrahl)
(Reporter)

Comment 2

a year ago
Comment on attachment 8869747 [details] [diff] [review]
Bug1366499.patch

Yes, this is what I imagined. I'll leave the review to FRG, he knows more about the issue than I do.
Attachment #8869747 - Flags: review?(jorgk)
Attachment #8869747 - Flags: review?(frgrahl)
Attachment #8869747 - Flags: feedback?(frgrahl)
Attachment #8869747 - Flags: feedback+
Comment on attachment 8869747 [details] [diff] [review]
Bug1366499.patch

LGTM and r+ but please take out the whole _setUpUserAgentOverrides from nsSuiteGlue.js and the moodle pref from suite/browser/browser-prefs.js.

The moodle pref was added in bug 801318 for SeaMonkey and has been backed out in Fx later. I think it is ancient history. I accessed the site as demo student and it seems to work ok. There is also an indication that it has long ago been fixed in moodle:

https://bugzilla.mozilla.org/show_bug.cgi?id=799502#c24

IanN if you don't concur I will put it back in in a followup bug.
Flags: needinfo?(iann_bugzilla)
Attachment #8869747 - Flags: review?(frgrahl) → review+
(Assignee)

Comment 4

a year ago
Created attachment 8869780 [details] [diff] [review]
Bug1366499.patch v2

Okay, removed the whole _setUpUserAgentOverrides.
Assignee: nobody → richard.marti
Attachment #8869747 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8869780 - Flags: review?(frgrahl)
Comment on attachment 8869780 [details] [diff] [review]
Bug1366499.patch v2

Looks good. Thanks
Attachment #8869780 - Flags: review?(frgrahl) → review+

Comment 6

a year ago
Fine by me
Flags: needinfo?(iann_bugzilla)
(Assignee)

Comment 7

a year ago
https://hg.mozilla.org/comm-central/rev/9aeec5533d4cb36ce50bad7e1e5148dea19f4945
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 55.0
You need to log in before you can comment on or make changes to this bug.