1573268 - Convert the varcache prefs in nsLayoutUtils.cpp to static prefs

Status: RESOLVED FIXED

(Core :: Preferences: Backend, task)

Description

[Kris Wright :KrisWright]

For the most part, the prefs in nsLayoutUtils follow this pattern:

  static bool sShouldLog;
  static bool sShouldLogPrefCached;

  if (!sShouldLogPrefCached) {
    sShouldLogPrefCached = true;
    Preferences::AddBoolVarCache(
        &sShouldLog, "layers.offmainthreadcomposition.log-animations");
  }

  return sShouldLog;

Or, they're declared in nsLayoutUtils::Initialize(). They look mostly straightforward and could be converted to use static prefs, with these add-pref-and-return-value functions removed.

Comment 1

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert two layers.offmainthreadcomposition.* prefs to static prefs. r=njn

Converts layers.offmainthreadcomposition.async-animations and layers.offmainthreadcomposition.log-animations to their respective static prefs. Since all IsAnimationLoggingEnabled() did was create a pref and return the variable sShouldLog, this function is removed and replaced with the static pref.

Comment 2

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert layout.animated-image-layers.enabled to static pref. r=njn

Converts layout.animated-image-layers.enabled to a static pref and removes the nsLayoutUtils::AnimatedImageLayersEnabled() function, replacing it with the static pref.

Comment 3

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert layout.css.ruby.intercharacter.enabled to static pref. r=njn

Converts layout.css.ruby.intercharacter.enabled to a static pref and removes the associated function nsLayoutUtils::IsInterCharacterRubyEnabled(). Also removes the macro INTERCHARACTER_RUBY_ENABLED_PREF_NAME, since it was only being used to add the varcache pref.

Comment 4

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert font.size.inflation.maxRatio to static pref. r=njn

Creates the new 'font' category in StaticPrefList.yaml and adds font.size.inflation.maxRatio to the category. Removes the old static uint in nsLayoutUtils and the function that returns it.

Comment 5

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert 3 font.size.inflation.* prefs to static prefs. r=njn

Converts font.size.inflation.minTwips, font.size.inflation.emPerLine, and font.size.inflation.mappingIntercept to static prefs and removes their associated functions from nsLayoutUtils. There are associated member variables in PressShell, but since documentation specified that these variables are set specifically to prevent changes to the cache from being read until page reload, I made the decision to leave these and set them to the static prefs.

Comment 6

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert font.size.inflation.lineThreshold to a static pref. r=njn

Converts font.size.inflation.lineThreshold varcache pref to a static pref. Like previous revisions, this retains the member variable in PresShell.

Comment 7

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert two font.size.inflation.* prefs to static prefs. r=njn

Converts font.size.inflation.forceEnabled and font.size.inflation.disabledInMasterProcess to static prefs. Like previous revisions, I retained the member variables in PresShell and set them to the static prefs.

Comment 8

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert font.size.systemFontScale to static pref. r=njn

Converts font.size.systemFontScale to a static pref. Removes the function in nsLayoutUtils and does the float division directly in PresShell.

Comment 9

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert zoom.maxPercent and zoom.minPercent to static prefs. r=njn

Converts zoom.maxPercent and zoom.minPercent to static prefs, which creates a new "zoom" category on StaticPrefList.yaml.

Comment 10

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert nglayout.debug.invalidation to static pref. r=njn

Converts nglayout.debug.invalidation to a static pref. Retains the old getter function from the old static bool, because it makes an additional comparison.

Comment 11

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert layout.interruptible-reflow.enabled to static pref. r=njn

Converts layout.interruptible-reflow.enabled to a static pref and updates its usage.

Comment 12

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - remove the varcache definition for svg.transform-box.enabled. r=njn

svg.transform-box.enabled is already a static pref, so I removed the varcache definition of it in nsLayoutUtils.

Comment 13

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert two layout.idle-period.* prefs to static prefs. r=njn

Converts layout.idle_period.required_quiescent_frames and layout.idle_period.time_limit to static prefs. These are the last prefs in nsLayoutUtils::initialize(), but since the function still calls nsComputedDOMStyle::RegisterPrefChangeCallbacks() the commit retains it.

Comment 14

[Kris Wright :KrisWright]

Attachment: Bug 1573268 - Convert layout.css.outline-style-auto.enabled to static pref. r=njn

Converts layout.css.outline-style-auto.enabled to a static pref and removes nsLayoutUtils::isOutlineStyleAutoEnabled().

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0b74cfbb22fd
Convert two layers.offmainthreadcomposition.* prefs to static prefs. r=njn
https://hg.mozilla.org/integration/autoland/rev/aa9e64f35d08
Convert layout.animated-image-layers.enabled to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/43cbfaf3e97e
Convert layout.css.ruby.intercharacter.enabled to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/5028fca248cc
Convert font.size.inflation.maxRatio to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/cd573e89e3a8
Convert 3 font.size.inflation.* prefs to static prefs. r=njn
https://hg.mozilla.org/integration/autoland/rev/266c9ff0255f
Convert font.size.inflation.lineThreshold to a static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/c84c8016d09e
Convert two font.size.inflation.* prefs to static prefs. r=njn
https://hg.mozilla.org/integration/autoland/rev/2bfed8cd65e8
Convert font.size.systemFontScale to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/eb90a1fef7e9
Convert zoom.maxPercent and zoom.minPercent to static prefs. r=njn
https://hg.mozilla.org/integration/autoland/rev/b7bef0d94937
Convert nglayout.debug.invalidation to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/33dd94200b51
Convert layout.interruptible-reflow.enabled to static pref. r=njn
https://hg.mozilla.org/integration/autoland/rev/c657ac21f56a
remove the varcache definition for svg.transform-box.enabled. r=njn
https://hg.mozilla.org/integration/autoland/rev/def3d7b5c636
Convert two layout.idle-period.* prefs to static prefs. r=njn
https://hg.mozilla.org/integration/autoland/rev/45e0edd03be0
Convert layout.css.outline-style-auto.enabled to static pref. r=njn

Comment 16

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/0b74cfbb22fd
https://hg.mozilla.org/mozilla-central/rev/aa9e64f35d08
https://hg.mozilla.org/mozilla-central/rev/43cbfaf3e97e
https://hg.mozilla.org/mozilla-central/rev/5028fca248cc
https://hg.mozilla.org/mozilla-central/rev/cd573e89e3a8
https://hg.mozilla.org/mozilla-central/rev/266c9ff0255f
https://hg.mozilla.org/mozilla-central/rev/c84c8016d09e
https://hg.mozilla.org/mozilla-central/rev/2bfed8cd65e8
https://hg.mozilla.org/mozilla-central/rev/eb90a1fef7e9
https://hg.mozilla.org/mozilla-central/rev/b7bef0d94937
https://hg.mozilla.org/mozilla-central/rev/33dd94200b51
https://hg.mozilla.org/mozilla-central/rev/c657ac21f56a
https://hg.mozilla.org/mozilla-central/rev/def3d7b5c636
https://hg.mozilla.org/mozilla-central/rev/45e0edd03be0