mirror of
https://github.com/servo/servo.git
synced 2025-08-03 20:50:07 +01:00
style: Specify has_no_effect_on_gecko_scrollbars on inherited internal properties
Otherwise we assert due to the styles being different from the cached ones, but it doesn't matter as these don't influence Gecko scrollbars. Differential Revision: https://phabricator.services.mozilla.com/D117147
This commit is contained in:
parent
f0a178b56a
commit
6d9580fef0
2 changed files with 8 additions and 2 deletions
|
@ -340,13 +340,14 @@ class Longhand(Property):
|
||||||
assert (
|
assert (
|
||||||
has_effect_on_gecko_scrollbars in [None, False, True]
|
has_effect_on_gecko_scrollbars in [None, False, True]
|
||||||
and not style_struct.inherited
|
and not style_struct.inherited
|
||||||
or (gecko_pref is None) == (has_effect_on_gecko_scrollbars is None)
|
or (gecko_pref is None and enabled_in != "")
|
||||||
|
== (has_effect_on_gecko_scrollbars is None)
|
||||||
), (
|
), (
|
||||||
"Property "
|
"Property "
|
||||||
+ name
|
+ name
|
||||||
+ ": has_effect_on_gecko_scrollbars must be "
|
+ ": has_effect_on_gecko_scrollbars must be "
|
||||||
+ "specified, and must have a value of True or False, iff a "
|
+ "specified, and must have a value of True or False, iff a "
|
||||||
+ "property is inherited and is behind a Gecko pref"
|
+ "property is inherited and is behind a Gecko pref or internal"
|
||||||
)
|
)
|
||||||
self.need_index = need_index
|
self.need_index = need_index
|
||||||
self.gecko_ffi_name = gecko_ffi_name or "m" + self.camel_case
|
self.gecko_ffi_name = gecko_ffi_name or "m" + self.camel_case
|
||||||
|
|
|
@ -213,6 +213,7 @@ ${helpers.predefined_type(
|
||||||
initial_value="computed::XLang::get_initial_value()",
|
initial_value="computed::XLang::get_initial_value()",
|
||||||
animation_value_type="none",
|
animation_value_type="none",
|
||||||
enabled_in="",
|
enabled_in="",
|
||||||
|
has_effect_on_gecko_scrollbars=False,
|
||||||
spec="Internal (not web-exposed)",
|
spec="Internal (not web-exposed)",
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
@ -224,6 +225,7 @@ ${helpers.predefined_type(
|
||||||
animation_value_type="none",
|
animation_value_type="none",
|
||||||
gecko_ffi_name="mScriptSizeMultiplier",
|
gecko_ffi_name="mScriptSizeMultiplier",
|
||||||
enabled_in="",
|
enabled_in="",
|
||||||
|
has_effect_on_gecko_scrollbars=False,
|
||||||
spec="Internal (not web-exposed)",
|
spec="Internal (not web-exposed)",
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
@ -263,6 +265,7 @@ ${helpers.single_keyword(
|
||||||
spec="Internal (not web-exposed)",
|
spec="Internal (not web-exposed)",
|
||||||
animation_value_type="none",
|
animation_value_type="none",
|
||||||
enabled_in="",
|
enabled_in="",
|
||||||
|
has_effect_on_gecko_scrollbars=False,
|
||||||
needs_conversion=True,
|
needs_conversion=True,
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
@ -273,6 +276,7 @@ ${helpers.predefined_type(
|
||||||
engines="gecko",
|
engines="gecko",
|
||||||
animation_value_type="none",
|
animation_value_type="none",
|
||||||
enabled_in="",
|
enabled_in="",
|
||||||
|
has_effect_on_gecko_scrollbars=False,
|
||||||
gecko_ffi_name="mScriptMinSize",
|
gecko_ffi_name="mScriptMinSize",
|
||||||
spec="Internal (not web-exposed)",
|
spec="Internal (not web-exposed)",
|
||||||
)}
|
)}
|
||||||
|
@ -284,6 +288,7 @@ ${helpers.predefined_type(
|
||||||
engines="gecko",
|
engines="gecko",
|
||||||
animation_value_type="none",
|
animation_value_type="none",
|
||||||
enabled_in="",
|
enabled_in="",
|
||||||
|
has_effect_on_gecko_scrollbars=False,
|
||||||
spec="Internal (not web-exposed)",
|
spec="Internal (not web-exposed)",
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue