Disable enablePrivilege in automated tests so that no new enablePrivilege calls sneaking in

ASSIGNED
Assigned to

Status

Testing
General
ASSIGNED
7 months ago
7 months ago

People

(Reporter: emk, Assigned: emk)

Tracking

(Blocks: 1 bug)

Version 3
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
We can't kill enablePrivilege from Talos yet, but it should not be used in any other test suites.

security.turn_off_all_security_so_that_viruses_can_take_over_this_computer was introduced to enable enablePrivilege in automation, but it is overloaded for various other purposes now. So we will have to add a new pref to control only enablePrivilege.
(Assignee)

Updated

7 months ago
Depends on: 1365466
No longer depends on: 1365466
Depends on: 1365466
(Assignee)

Updated

7 months ago
Blocks: 546848
(Assignee)

Updated

7 months ago
Blocks: 462483
No longer blocks: 546848
Comment hidden (mozreview-request)
(Assignee)

Updated

7 months ago
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
don't we have many other cases of enablePrivilege in tree outside of Talos:
https://dxr.mozilla.org/mozilla-central/search?q=enablePrivilege

Is there a reason why we cannot kill this from Talos?
(Assignee)

Comment 3

7 months ago
(In reply to Joel Maher ( :jmaher) from comment #2)
> don't we have many other cases of enablePrivilege in tree outside of Talos:
> https://dxr.mozilla.org/mozilla-central/search?q=enablePrivilege

Those tests are dead (not running in automation). I'm not bothered to remove or restore them.

> Is there a reason why we cannot kill this from Talos?

Bug 715588 was WONTFIXed. It will take time to write a custom extension, sign the extension, make sure it does not introduce instability, seek some alternatives if it does, etc. But I would not like to leave enablePrivilege enabled in other test suites just because I can't remove enablePrivilege from Talos at the moment.
(Assignee)

Comment 4

7 months ago
Try runs for the record:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c4be8608f83be911b61ed46e07e65d99cbc280a1
https://treeherder.mozilla.org/#/jobs?repo=try&revision=18d04ff658c5e804fb9bb7b4c541ca25e1ab89dc
For talos there is talos-powers which is used by all tests.  The bug referenced was closed 3 years ago, a lot has changed since then.

I notice this patch deletes a test case test_bug504877.html, is that case not providing any value?  How did we determine it was ok to delete the test?

I need to understand more before giving this an r+.

In talos a lot of the instanced of enablePrivilege are related to the profiler, I suspect with talos-powers we could resolve anything we need, or maybe with the new profile built in we don't need this:
https://dxr.mozilla.org/mozilla-central/search?q=path%3Atesting%2Ftalos+enablePrivilege&redirect=true
Comment on attachment 8869400 [details]
Bug 1365199 - Add an enablePrivilege specific pref and use it in Talos.

waiting on more data before doing formal review.
Attachment #8869400 - Flags: review?(jmaher)
You need to log in before you can comment on or make changes to this bug.