Port bug 1426362 to TB: Display status of policy engine in about:support

RESOLVED FIXED in Thunderbird 60.0

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 60.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

Last year
Bug 1426362 added the display of the policy state. We don't use it, but add it to be in sync with FX.
Assignee

Comment 1

Last year
This is only to be in sync with the m-c files. TB doesn't support the policies.

Actually about:support is broken until bug 1442866 lands.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8955805 - Flags: review?(acelists)

Comment 2

Last year
On current trunk just opening the Troubleshooting page crashes TB for me, in mozilla::EventTargetChainItem::HandleEvent(mozilla::EventChainPostVisitor&, mozilla::ELMCreationDetector&) (this=0x7f9d1d678008, aVisitor=..., aCd=...) .

Comment 3

Last year
Also https://dxr.mozilla.org/comm-central/source/mozilla/toolkit/modules/Troubleshoot.jsm#254 calls Services.policies.status without checking if Services.policies exists and it fails with a warning in the console. We should file that in Toolkit.
I tried, but fixing error that does not fix the crash.
Assignee

Comment 4

Last year
(In reply to :aceman from comment #3)
> Also
> https://dxr.mozilla.org/comm-central/source/mozilla/toolkit/modules/
> Troubleshoot.jsm#254 calls Services.policies.status without checking if
> Services.policies exists and it fails with a warning in the console. We
> should file that in Toolkit.
> I tried, but fixing error that does not fix the crash.

This one is handled by bug 1442866.
Assignee

Comment 5

Last year
(In reply to :aceman from comment #2)
> On current trunk just opening the Troubleshooting page crashes TB for me, in
> mozilla::EventTargetChainItem::HandleEvent(mozilla::EventChainPostVisitor&,
> mozilla::ELMCreationDetector&) (this=0x7f9d1d678008, aVisitor=..., aCd=...) .

No crash on current trunk on Linux Mint.

Comment 6

Last year
Comment on attachment 8955805 [details] [diff] [review]
Policies.patch

Do we really need to port this? As far as I know, Services.policies is null in TB, so you can't even test this. No need for a confusing no-op, I think.
Assignee

Comment 7

Last year
We do this to have the files in sync...and maybe they decide sometime to move the policies to toolkit.

Comment 8

Last year
Comment on attachment 8955805 [details] [diff] [review]
Policies.patch

Review of attachment 8955805 [details] [diff] [review]:
-----------------------------------------------------------------

My Troubleshooting page suddenly no longer crashes TB (may also be due to updated Mesa driver, which I do often, and the page calls checks for OpenGL properties, with some warnings shown in console) so I can proceed here.
Seems to work fine, meaning no Policies table is shown for TB.
Thanks.
Attachment #8955805 - Flags: review?(acelists) → review+
Assignee

Comment 9

Last year
Thanks.
Keywords: checkin-needed

Comment 10

Last year
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/fe09f771c2d3
Port bug 1426362 to TB: Display status of policy engine in about:support. r=aceman
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

Last year
Target Milestone: --- → Thunderbird 60.0
You need to log in before you can comment on or make changes to this bug.