Closed Bug 1719667 Opened 3 years ago Closed 3 years ago

Do not set recommended preferences when not wanted

Categories

(Remote Protocol :: Agent, defect, P2)

defect
Points:
2

Tracking

(firefox91 fixed)

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: whimboo, Assigned: jdescottes)

References

Details

(Whiteboard: [bidi-m1-mvp])

Attachments

(1 file)

This bug came up for my work on bug 1693805 where CDP tries to set recommended preferences and by default all common recommended preferences are set. But this should not be done given that in case of running Marionette tests in CI we turn off this feature via marionette.prefs.recommended = false:

https://searchfox.org/mozilla-central/rev/da25888c4495585c532640f0e5efad07b1037621/remote/components/marionette.js#486-494

As such we have to do follow-up work from bug 1718052, which shared the usage of recommended prefs with the Remote Agent. Means we could use remote.prefs.recommended in RecommendedPrefs.jsm.

Note that this pref is only turned off in our CI, so any automation that is using geckodriver isn't affected.

No longer blocks: 1718052
Depends on: 1718052
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Whiteboard: [bidi-m1-mvp] → [webdriver:triage]
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a5c04de0bab3 [remote] Do not set recommended preferences when not wanted r=webdriver-reviewers,whimboo
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Points: --- → 2
Priority: -- → P2
Whiteboard: [webdriver:triage] → [bidi-m1-mvp]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: