Closed Bug 1597189 Opened 6 years ago Closed 5 years ago

Remove setting of "datareporting.healthreport.about.reportUrl" preference from geckodriver

Categories

(Testing :: geckodriver, task)

Version 3
task
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1662544

People

(Reporter: whimboo, Unassigned, Mentored)

References

Details

(Whiteboard: [lang=rust])

User Story

Please check the following URL in how to get started:
https://firefox-source-docs.mozilla.org/testing/geckodriver/index.html

In prefs.rs we still set the preferences for datareporting.healthreport.about.reportUrl to a custom value:

https://searchfox.org/mozilla-central/rev/524bed6dfbc5ae21c62632d83b7573448b29e0ac/testing/geckodriver/src/prefs.rs#73

This is not necessary anymore given the those preferences have been removed with Firefox 59 (bug 1352497), and we no longer support that release of Firefox

Those lines can simply be removed.

These lines lines have already been removed from pref.rs we can close this guy out.

Note that the pref still appears to be in a typescript file:

mozilla-unified rg -F 'datareporting.healthreport.about.reportUrl'    
remote/test/puppeteer/src/node/Launcher.ts
564:      'datareporting.healthreport.about.reportUrl': `http://${server}/dummy/abouthealthreport/`,

Is it worth removing that guy too?

Right, that was done on bug 1662544, but I forget to remove it via bug 1662549. Mind filing a bug for that under the remote protocol product?

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(me)
Resolution: --- → DUPLICATE

I actually filed bug 1665283. Thanks for pointing that out.

Flags: needinfo?(me)
You need to log in before you can comment on or make changes to this bug.