Closed Bug 1587804 Opened 10 months ago Closed 10 months ago

Add a policy to set the private browsing search engine

Categories

(Firefox :: Enterprise Policies, task, P2)

task
Points:
3

Tracking

()

RESOLVED FIXED
Firefox 71
Iteration:
71.4 - Oct 14 - 20
Tracking Status
firefox71 --- fixed

People

(Reporter: mak, Assigned: mikedeboer)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Policies can set an engine as default, but not as Private Default.

Priority: P3 → P2

What version of Firefox is this targeted for?

Component: Search → Enterprise Policies

(In reply to Mike Kaply [:mkaply] from comment #1)

What version of Firefox is this targeted for?

  1. We're looking to be turning on the feature in nightly tomorrow, ahead of the soft freeze. We should be able to help out next week, though pointers might be useful.

Actually we don't allow search engine changes except on ESR, so we won't need this until mid next year.

so no rush.

Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Iteration: --- → 71.4 - Oct 14 - 20
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/caf57dd9ae33
Add an Enterprise Policy that allows to set the default search engine used in Private Browsing mode. r=mkaply
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

Hello,
I have several questions about this ticket:

  • I have verified and can confirm that the default search engine can be set via policy. However , it can be noticed that even though the default search engine is set for private browsing, it also applies to normal session.

*Another thing would be that even though the default search engine is set via policy, the user can still change the default search engine, and the option remains selected even after browser restart.

*Verification was done on 72.0a1 (2019-10-23), 71.0b3 BuildID:20191021164841 where the policy is indeed set(with the above mentions), however in comment 3, it states that this will also apply to ESR, but only next year. Would it be possible to let us know on which builds should we test, if this is indeed ready to be tested by QA.

Flags: needinfo?(mak77)

A quick glance at the code and it looks like we're not setting browser.search.separatePrivateDefault to true first...

The policy applying whilst not on ESR is strange...

Flags: needinfo?(mozilla)
Flags: needinfo?(mdeboer)
Flags: needinfo?(mak77)

I guess I was wrong in assuming that the defaultPrivateEngine setter would set the browser.search.separatePrivateDefault pref, but I guess I'm just confused about how that pref should be used in general and whether it's good practice to have the policy code mess with it in the first place.

Flags: needinfo?(mdeboer)

(In reply to Mike de Boer [:mikedeboer] from comment #9)

I guess I was wrong in assuming that the defaultPrivateEngine setter would set the browser.search.separatePrivateDefault pref, but I guess I'm just confused about how that pref should be used in general and whether it's good practice to have the policy code mess with it in the first place.

Yeah, I think my original ideas were wrong about that. I've filed bug 1591059 as a follow-up. We won't need it for 71, but we may want it later.

I think the only other remaining thing here is the question of the policy being enabled on non-ESR, that hopefully the other Mike can help us with.

(In reply to Vlad Lucaci, QA (:vlucaci) from comment #7)

*Verification was done on 72.0a1 (2019-10-23), 71.0b3 BuildID:20191021164841 where the policy is indeed set(with the above mentions), however in comment 3, it states that this will also apply to ESR, but only next year. Would it be possible to let us know on which builds should we test, if this is indeed ready to be tested by QA.

I just stumbled across the code:

https://searchfox.org/mozilla-central/rev/35873cfc312a6285f54aa5e4ec2d4ab911157522/toolkit/components/enterprisepolicies/EnterprisePolicies.js#397-426

The basic summary is that ESR-only policies are actually only disabled in release channels. They're still allowed in beta & nightly. So the fact it worked is expected.

Flags: needinfo?(mozilla)

I was totally going to post that and forgot. Sorry.

You need to log in before you can comment on or make changes to this bug.