Convert Varcache prefs in TimeoutManager to StaticPrefs
Categories
(Core :: Preferences: Backend, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: KrisWright, Assigned: KrisWright)
References
Details
Attachments
(12 files)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
The 13 VarCache prefs in TimeoutManager.cpp
could all be converted to StaticPrefs and added to StaticPrefList.yaml. These prefs are found in TimeoutManager::Initialize(), here: https://searchfox.org/mozilla-central/source/dom/base/TimeoutManager.cpp#481
Assignee | ||
Comment 1•6 years ago
|
||
Converts dom.min_timeout_value varcache pref from TimeoutManager::Initialize() to a static pref, and removes its entry and associated constants from TimeoutManager.cpp and all.js
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 2•6 years ago
|
||
Converts dom.disable_open_click_delay from a VarCache pref in TimeoutManager.cpp to StaticPref, removes all.js entry, and corrects a typo where the value was mentioned in nsGlobalWindowInner.cpp.
Assignee | ||
Comment 3•6 years ago
|
||
Converts dom.timeout.max_consecutive_callbacks_ms varcache pref in TimeoutManager.cpp to static pref, removes associated variables and all.js entry, and adds entry to StaticPrefList.yaml.
Assignee | ||
Comment 4•6 years ago
|
||
Converts dom.timeout.throttling_delay in TimeoutManager.cpp from varcache pref to static pref, removes entry in all.js, and adds entry in StaticPrefList.yaml. Favors the all.js value over the declared value in TimeoutManager.cpp.
Assignee | ||
Comment 5•6 years ago
|
||
Removes dom.min_tracking_timeout_value and dom.min_tracking_background_timeout_value, because they are never used.
Assignee | ||
Comment 6•6 years ago
|
||
Converts dom.min_background_timeout_value in TimeoutManager.cpp to static pref, removes entry in all.js, adds entry to StaticPrefList.yaml.
Assignee | ||
Comment 7•6 years ago
|
||
Converts dom.timeout.background_budget_regeneration_rate varcache pref in TimeoutManager.cpp to static pref, removes entry in all.js, adds entry to StaticPrefList.yaml.
Assignee | ||
Comment 8•6 years ago
|
||
Converts dom.timeout.foreground_budget_regeneration_rate varcache pref in TimeoutManager.cpp to static pref, removes entry in all.js, adds entry in StaticPrefList.yaml.
Assignee | ||
Comment 9•6 years ago
|
||
Converts dom.timeout.background_throttling_max_budget varcache pref to static pref, removes all.js entry, adds entry to StaticPrefList.yaml.
Assignee | ||
Comment 10•6 years ago
|
||
Converts dom.timeout.foreground_throttling_max_budget to static pref, removes all.js entry, updates usage, adds entry to StaticPrefList.yaml.
Assignee | ||
Comment 11•6 years ago
|
||
Converts dom.timeout.budget_throttling_max_delay varcache pref to static pref, removes entry from all.js, and adds entry to StaticPrefList.yaml
Assignee | ||
Comment 12•6 years ago
|
||
Converts dom.timeout.enable_budget_timer_throttling from varcache pref to static pref, removes entry from all.js, adds entry to StaticPrefList.yaml. Uses the all.js value and not the value declared in TimeoutManager.cpp. Since this removes the last varcache pref from TimeoutManager::Initialize(), I also removed the Initialize() function and its call in nsGlobalWindowInner.
Comment 13•6 years ago
|
||
All done! Nice work. I recommend doing a try push once you've done your fix-ups. I often use this invocation:
./mach try fuzzy -q "^build" -q "^test-linux64/" -q "^test-linux64-asan/"
defined with the alias try-build-all-test-linux64-some
. It builds on all platforms and runs all tests on some of the Linux64 platforms.
Comment 14•5 years ago
|
||
Comment 15•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2655857e489d
https://hg.mozilla.org/mozilla-central/rev/a5d0fdf98930
https://hg.mozilla.org/mozilla-central/rev/9f8ada36cf9b
https://hg.mozilla.org/mozilla-central/rev/5986fb6ed82f
https://hg.mozilla.org/mozilla-central/rev/a8d9b6cde064
https://hg.mozilla.org/mozilla-central/rev/0708d16f56de
https://hg.mozilla.org/mozilla-central/rev/68cb4f3a31fd
https://hg.mozilla.org/mozilla-central/rev/89aacbb2f52f
https://hg.mozilla.org/mozilla-central/rev/3d7f9cba78a7
https://hg.mozilla.org/mozilla-central/rev/ba7879d87773
https://hg.mozilla.org/mozilla-central/rev/760fd1f56980
https://hg.mozilla.org/mozilla-central/rev/0a9a0b8b35fa
Description
•