Generating StaticPrefList.h will help with some other things, like:

• splitting it into pieces to reduce recompilations (bug 1563139);
• improving Rust access to prefs (bug 1563555);
• making it harder to mess up static prefs (previously attempted in bug 1563146)

Minor things to ease the transition to StaticPrefList.yaml.

• Rename apz_touch_drag_enabled() to the correct apz_drag_touch_enabled().

• Change some trailing C-style comments to C++-style comments. This makes life
  easier for the script I wrote to convert StaticPrefList.h to
  StaticPrefList.yaml, which will be used for the next patch.

• Avoid comments on #define lines, because they aren't handled in the YAML
  input.

• Convert a multi-line #if condition to a single line, because
  Preprocessor.py doesn't handle multi-line conditions.

• Remove one unnecessary #undef PREF_VALUE directive.

• Move intl.charset.detector.iso2022jp.allowed to the correct section, so the
  YAML processing script won't complain.

• Change an int pref to int32_t, again so the YAML processing script won't
  complain.

Depends on D37526

glandium: In case it's not clear, the order of these patches is "Tweak...", Generate...", "Remove...", but Phabricator isn't showing that because I originally pushed just the "Generate..." patch.

Also note that the "Generate..." patch has a couple of njn: comments reordering things in StaticPrefList.h. Once bug 1565110 lands they will go away.

Also note that the "Generate..." patch has a couple of njn: comments reordering things in StaticPrefList.h. Once bug 1565110 lands they will go away.

Bug 1565110 landed, and I have updated the loaded patches accordingly.

https://hg.mozilla.org/mozilla-central/rev/63d7dc4650c3
https://hg.mozilla.org/mozilla-central/rev/3d0e35ab6850

https://hg.mozilla.org/mozilla-central/rev/5382cec9e306