1220287 - Get rid of some non-used prefs

Status: NEW

(Core :: DOM: Core & HTML, defect)

Description

[Andrea Marchesini [:baku]]

Attachment: pref.patch

I wrote a script to check which prefs we use. Some of then are not used at all:

Looking for accessibility.usebrailledisplay... NO
Looking for accessibility.typeaheadfind.enabletimeout... NO
Looking for toolkit.asyncshutdown.timeout.crash... NO
Looking for media.mp3.enabled... NO
Looking for media.apple.mp4.enabled... NO
Looking for application.use_ns_plugin_finder... NO
Looking for browser.chrome.toolbar_style... NO
Looking for gfx.prefer-mesa-llvmpipe... NO
Looking for capability.policy.default.SOAPCall.invokeVerifySourceHeader... NO
Looking for browser.popups.showPopupBlocker... NO
Looking for viewmanager.do_doublebuffering... NO
Looking for browser.addon-watch.percentage-limit... NO
Looking for media.gmp-manager.certs.1.issuerName... NO
Looking for media.gmp-manager.certs.1.commonName... NO
Looking for media.gmp-manager.certs.2.issuerName... NO
Looking for media.gmp-manager.certs.2.commonName... NO
Looking for profile.confirm_automigration... NO
Looking for profile.migration_directory... NO
Looking for profile.seconds_until_defunct... NO
Looking for profile.manage_only_at_launch... NO
Looking for prefs.converted-to-utf8... NO
Looking for print.print_extra_margin... NO
Looking for plugin.allow.asyncdrawing... NO
Looking for autocomplete.ungrab_during_mode_switch... NO
Looking for autocomplete.grab_during_popup... NO

This is a very low priority bug.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Does the tool check whether prefs are used as part of a pref branch?

Comment 2

[Andrea Marchesini [:baku]]

Basic stuff, yes. The rest has been checked manually. Specially for some 'easy-to-grep' prefs like: grab_during_popup, confirm_automigration, etc.

Comment 3

[:Gijs (he/him)]

(In reply to Andrea Marchesini (:baku) from comment #0)
> Looking for accessibility.typeaheadfind.enabletimeout... NO

This seems to be in use in comm-central, fwiw. (I vaguely recall trying to get rid of it before now...)

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

at least also profile.manage_only_at_launch is used by c-c.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8681466 [details] [diff] [review]
pref.patch

so, would be safer to remove only prefs which aren't used in m-c nor in c-c