1442866 - TB fails on about:support with ReferenceError: reference to undefined property "policies" - TEST-UNEXPECTED-FAIL | [snip]/mozmill/content-tabs/test-about-support.js

Status: RESOLVED FIXED

(Firefox :: Enterprise Policies, defect)

Description

[Richard Marti (:Paenglab)]

Since bug 1426362 TB fails in about:config with:
ReferenceError: reference to undefined property "policies"[Learn More]  Troubleshoot.jsm:254:5
Troubleshoot data provider failed: application
TypeError: Services.policies is undefined  Troubleshoot.jsm:154

It's no wonder as this is a browser only feature. Please could you make to not fail when it's undefined or only check when it's run from FX?

And maybe think about moving the policies to toolkit? Then TB could use this too in the future

Comment 1

[Jorg K (CEST = GMT+2)]

Could we do something like:
try {
  data.policiesStatus = Services.policies.status;
} catch (e) {
  // Other applications like Thunderbird don't have policies.
  data.policiesStatus = null;
}

Richard, could you try that and submit a patch for review.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: policiesStatus.patch

With the try/catch I still get the ReferenceError, so I decided to use the AppConstants.

Comment 3

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8955767 [details] [diff] [review]
policiesStatus.patch

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

Sorry about that. I tried to protect against this problem in aboutSupport.js (see https://hg.mozilla.org/mozilla-central/rev/476e0758eb1c#l2.12), but I forgot about Troubleshoot.jsm.

This patch is fine, although I suggest that you try the following version:

if (Services.policies) {
  data.policiesStatus = Services.policies.status;
}

No need for the else part. And then on aboutSupport.js, we could change the line that I linked above to check for data.policiesStatus instead of Services.policies.

if it doesn't throw any errors, it's probably the best approach. Otherwise, I'll take this patch

Comment 4

[Richard Marti (:Paenglab)]

Attachment: policiesStatus.patch

(In reply to :Felipe Gomes (needinfo me!) from comment #3)
> This patch is fine, although I suggest that you try the following version:
> 
> if (Services.policies) {
>   data.policiesStatus = Services.policies.status;
> }

Ah, yes this is a lot simpler...and works without error.

Comment 5

[Richard Marti (:Paenglab)]

Thanks!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8577732074db
Make the policiesStatus work on TB too. r=Felipe

Comment 7

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/8577732074db