Closed Bug 1573968 Opened Last month Closed Last month

WheelTransaction::Prefs should use static prefs

Categories

(Core :: Preferences: Backend, task)

task
Not set

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: KrisWright, Assigned: KrisWright)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

It looks like the WheelTransaction::Prefs class exists solely to create and access five prefs for WheelTransaction, initialized here: https://searchfox.org/mozilla-central/rev/3366c3d24f1c3818df37ec0818833bf085e41a53/dom/events/WheelHandlingHelper.cpp#532-546

The prefs are already generated as part of static prefs:
https://searchfox.org/mozilla-central/rev/c7e8bc4996f979e5876b33afae3de3b1ab4f3ae1/modules/libpref/init/StaticPrefList.yaml#5445-5454,5471-5479,6150-6153

So these varcache prefs should really be removed and their usages updated. Other instances of these prefs are already using their StaticPrefs values.

All of these prefs are already static prefs, so this removes the varcache pref definitions from WheelTransaction and replaces them with the existing static prefs.

Pushed by kwright@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/77b5eec7b9e5
Remove WheelTransaction::Prefs. r=njn
Status: NEW → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.