Use StaticPrefs in ScrollFrameHelper::IsSmoothScrollingEnabled.
Categories
(Core :: Layout, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
Details
Attachments
(1 file)
Assignee | ||
Comment 1•5 years ago
|
||
Comment 2•5 years ago
|
||
Kris: here's an example where a pref value is specified in firefox.js
but it's the same value as what is specified in StaticPrefList.yaml
. Detecting such cases (including redundant overrides in mobile.js
, as well as prefs defined in all.js
) would be a nice extension of the prefs linter, if you are in the mood.
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0dfb6920936b Use StaticPrefs in ScrollFrameHelper::IsSmoothScrollingEnabled. r=njn
Comment 4•5 years ago
|
||
(In reply to Nicholas Nethercote [:njn] from comment #2)
Kris: here's an example where a pref value is specified in
firefox.js
but it's the same value as what is specified inStaticPrefList.yaml
. Detecting such cases (including redundant overrides inmobile.js
, as well as prefs defined inall.js
) would be a nice extension of the prefs linter, if you are in the mood.
I've been thinking about that lately, so I guess it's time to file a bug for it. Finding the names alone is easy enough (I can add a use case for the linter to use the optional |path| input, then pass that pref file to be loaded instead of all.js
) but in its current state the linter doesn't look at value
because it has no understanding of preprocessing. First we would have to revisit the idea of preprocessing the yaml file so it understands concepts like @IS_VALUE@
, which means declaring some sort of dummy buildconfig. Or we can just note the limitations of the linter (e.g. 'Lintpref doesn't understand #defines so it's always best to double-check') and let it handle what values it can understand.
Comment 5•5 years ago
|
||
bugherder |
Description
•