In bug 1107378 comment 2, dbaron asked that I add a nsCSSParser startup method that hooks up a boolean to an about:config pref, rather than doing the "pref / is-pref-cached" dance the first time the code is run. We've got some existing code in nsCSSParser that already does this dance (for "sOpentypeSVGEnabled"), so I'm splitting off this helper-bug to create the startup function to hook up that existing bool to a pref, and then in bug 1107378 I'll address dbaron's review comment by adding a new line to this newly-created startup function. (This way we'll be internally consistent within nsCSSParser, and we can keep this pref-hookup-refactoring logically separate from bug 1107378.) (Note: I'm marking this as "depends-on" bug 1135181 simply because they share some contextual code (in nsLayoutStatics::Initialize), and I'm layering this patch on top of bug 1135181's patch.)

(Er, of course we need to declare the static bool up at the top of the .cpp file, rather than at the bottom, in order to reference it from code in the middle of the file. Fixed that here.)

(Gentle review-ping -- this is all that's blocking bug 1107378 from landing, & I'd love to get this & that in before too long, for maximal nightly testing/baking!)

Comment on attachment 8567314 [details] [diff] [review] fix v2 Review of attachment 8567314 [details] [diff] [review]: ----------------------------------------------------------------- Sorry, slipped past my gaze; looks good.

Thanks! Landed: https://hg.mozilla.org/integration/mozilla-inbound/rev/6eccf5302c26