Closed Bug 1597187 Opened 5 years ago Closed 5 years ago

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

Categories

(Remote Protocol :: 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: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78

Hamzah, thanks a lot for your patch!

Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: