Make nscssproperty a method of longhand/shorthand/alias class.

components/style/properties/data.py

@@ -299,6 +299,12 @@ class Longhand(object):
             return computed
         return "<{} as ToAnimatedValue>::AnimatedValue".format(computed)
 
+    def nscsspropertyid(self):
+        ident = self.ident
+        if ident == "float":
+            ident = "float_"
+        return "nsCSSPropertyID::eCSSProperty_%s" % ident
+
 
 class Shorthand(object):
     def __init__(self, name, sub_properties, spec=None, servo_pref=None, gecko_pref=None,
@@ -362,6 +368,9 @@ class Shorthand(object):
     def enabled_in_content(self):
         return self.enabled_in == "content"
 
+    def nscsspropertyid(self):
+        return "nsCSSPropertyID::eCSSProperty_%s" % self.ident
+
 
 class Alias(object):
     def __init__(self, name, original):
@@ -388,6 +397,9 @@ class Alias(object):
     def enabled_in_content(self):
         return self.enabled_in == "content"
 
+    def nscsspropertyid(self):
+        return "nsCSSPropertyID::eCSSPropertyAlias_%s" % self.camel_case
+
 
 class Method(object):
     def __init__(self, name, return_type=None, arg_types=None, is_mut=False):

components/style/properties/helpers.mako.rs

@@ -872,17 +872,3 @@
     </%self:logical_setter_helper>
     }
 </%def>
-
-<%def name="alias_to_nscsspropertyid(alias)">
-    <%
-        return "nsCSSPropertyID::eCSSPropertyAlias_%s" % to_camel_case(alias)
-    %>
-</%def>
-
-<%def name="to_nscsspropertyid(ident)">
-    <%
-        if ident == "float":
-            ident = "float_"
-        return "nsCSSPropertyID::eCSSProperty_%s" % ident
-    %>
-</%def>

components/style/properties/helpers/animated_properties.mako.rs

@@ -68,7 +68,7 @@ pub fn nscsspropertyid_is_animatable(property: nsCSSPropertyID) -> bool {
     match property {
         % for prop in data.longhands + data.shorthands_except_all():
             % if prop.animatable:
-                ${helpers.to_nscsspropertyid(prop.ident)} => true,
+                ${prop.nscsspropertyid()} => true,
             % endif
         % endfor
         _ => false
@@ -154,12 +154,12 @@ impl From<nsCSSPropertyID> for TransitionProperty {
     fn from(property: nsCSSPropertyID) -> TransitionProperty {
         match property {
             % for prop in data.longhands:
-            ${helpers.to_nscsspropertyid(prop.ident)} => {
+            ${prop.nscsspropertyid()} => {
                 TransitionProperty::Longhand(LonghandId::${prop.camel_case})
             }
             % endfor
             % for prop in data.shorthands_except_all():
-            ${helpers.to_nscsspropertyid(prop.ident)} => {
+            ${prop.nscsspropertyid()} => {
                 TransitionProperty::Shorthand(ShorthandId::${prop.camel_case})
             }
             % endfor
@@ -179,7 +179,7 @@ pub fn nscsspropertyid_is_transitionable(property: nsCSSPropertyID) -> bool {
     match property {
         % for prop in data.longhands + data.shorthands_except_all():
             % if prop.transitionable:
-                ${helpers.to_nscsspropertyid(prop.ident)} => true,
+                ${prop.nscsspropertyid()} => true,
             % endif
         % endfor
         _ => false

components/style/properties/properties.mako.rs

@@ -424,14 +424,8 @@ impl NonCustomPropertyId {
     #[cfg(feature = "gecko")]
     fn to_nscsspropertyid(self) -> nsCSSPropertyID {
         static MAP: [nsCSSPropertyID; ${len(data.longhands) + len(data.shorthands) + len(data.all_aliases())}] = [
-            % for property in data.longhands:
+            % for property in data.longhands + data.shorthands + data.all_aliases():
-                ${helpers.to_nscsspropertyid(property.ident)},
+                ${property.nscsspropertyid()},
-            % endfor
-            % for property in data.shorthands:
-                ${helpers.to_nscsspropertyid(property.ident)},
-            % endfor
-            % for property in data.all_aliases():
-                ${helpers.alias_to_nscsspropertyid(property.ident)},
             % endfor
         ];
 
@@ -1617,11 +1611,11 @@ impl PropertyId {
         use gecko_bindings::structs::*;
         match id {
             % for property in data.longhands:
-                ${helpers.to_nscsspropertyid(property.ident)} => {
+                ${property.nscsspropertyid()} => {
                     Ok(PropertyId::Longhand(LonghandId::${property.camel_case}))
                 }
                 % for alias in property.alias:
-                    ${helpers.alias_to_nscsspropertyid(alias.name)} => {
+                    ${alias.nscsspropertyid()} => {
                         Ok(PropertyId::LonghandAlias(
                             LonghandId::${property.camel_case},
                             AliasId::${alias.camel_case}
@@ -1630,11 +1624,11 @@ impl PropertyId {
                 % endfor
             % endfor
             % for property in data.shorthands:
-                ${helpers.to_nscsspropertyid(property.ident)} => {
+                ${property.nscsspropertyid()} => {
                     Ok(PropertyId::Shorthand(ShorthandId::${property.camel_case}))
                 }
                 % for alias in property.alias:
-                    ${helpers.alias_to_nscsspropertyid(alias.name)} => {
+                    ${alias.nscsspropertyid()} => {
                         Ok(PropertyId::ShorthandAlias(
                             ShorthandId::${property.camel_case},
                             AliasId::${alias.camel_case}