From 3816143a1d419ab89c7c7ab4cceb09e573e4aaf0 Mon Sep 17 00:00:00 2001 From: Boris Zbarsky Date: Wed, 6 Jun 2018 11:34:30 -0400 Subject: [PATCH] style: Use Atomic for the staticpref version of layout.css.font-variations.enabled. Atomic is implemented in terms of AtomicBase, because that way you don't need to depend on atomic 1-byte operations. That means that the rust bindgen sees it as a u32, not a bool. It's a bit concerning that the rust code seems to be doing an unsynchronized read here, but given this is a RelaxedAtomic, that's probably ok. Bug: 1467134 Reviewed-by: emilio --- components/style/font_face.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/style/font_face.rs b/components/style/font_face.rs index 5a111925ae7..57c0f3b0b77 100644 --- a/components/style/font_face.rs +++ b/components/style/font_face.rs @@ -323,7 +323,7 @@ macro_rules! is_descriptor_enabled { ("font-variation-settings") => { unsafe { use gecko_bindings::structs::mozilla; - mozilla::StaticPrefs_sVarCache_layout_css_font_variations_enabled + mozilla::StaticPrefs_sVarCache_layout_css_font_variations_enabled != 0 } }; ($name:tt) => {