-
Notifications
You must be signed in to change notification settings - Fork 315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean up preferences set in configure_firefox.py #1060
Comments
Is there a list of the prefs that have been removed? I was curious about this and looked for some in |
I started checking the preferences in Firefox's code. The slowStartup ones have simply been removed: https://bugzilla.mozilla.org/show_bug.cgi?id=1690854 |
Hey @palmeida, |
Hi @vringar , I could do the PR, especially for the simple ones that were just removed and don't provide any useful functionality. If there are some that need to be replaced with new preferences I don't know if I'll be able to do it (I can try). In any case, I can add a comment here with the results of my investigation and go from there. I have been using searchfox now. One doubt I have is whether the string existing in the code is enough to keep it or should functionality be tested. For instance, if "browser.rights.3.shown" were to be removed, would that affect OpenWPM audits? Of course the easiest path is to start by removing/adapting the preferences that don't exist anymore. |
Thank you for putting time into this! Don't stress too much about reimplementing the functionality that has been lost over time. As for ambiguous prefs, I'd just keep them for now and focus on clear cut cases. Once that is done, we can create new issues for the ambiguous prefs and split up that work. |
@palmeida Hey, do you have any WIP branch that I could pick up? |
Sorry, I haven't had any time to look into it since November. I'm still working on a project that uses OpenWPM, so I might go back to this, but I've been busy with other issues. |
A number of prefs we set have been removed for multiple years.
We should investigate how to maintain the functionality we originally used the prefs for.
The text was updated successfully, but these errors were encountered: