Closed Bug 1597187 Opened 1 year ago Closed 11 months ago

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

Categories

(Testing :: Marionette, task, P3)

Version 3
task

Tracking

(firefox78 fixed)

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: whimboo, Assigned: hamzah18051, Mentored)

References

Details

(Whiteboard: [lang=py])

User Story

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

Attachments

(1 file, 1 obsolete file)

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

https://searchfox.org/mozilla-central/rev/524bed6dfbc5ae21c62632d83b7573448b29e0ac/testing/marionette/client/marionette_driver/geckoinstance.py#46

This is not necessary anymore given the those preferences have been removed with Firefox 59 (bug 1352497).

Those lines can simply be removed. Please make sure to also run the unit tests under testing/marionette/harness/marionette_harness/tests/unit to ensure nothing is broken:

./mach marionette-test %path_to_tests%

Summary: Remove setting of "datareporting.healthreport.*" preferences from Marionette → Remove setting of "datareporting.healthreport.about.reportUrl" preference from Marionette
Assignee: nobody → hamzah18051
Status: NEW → ASSIGNED
Priority: -- → P3
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3f463aabe0de
Remove setting of "datareporting.healthreport.about.reportUrl" preference from Marionette r=marionette-reviewers,whimboo
Attachment #9111921 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78

Hamzah, thanks a lot for your patch!

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