1341785 - stylo: need support for -moz-transform-* properties

Status: RESOLVED DUPLICATE of bug 1351356

(Core :: CSS Parsing and Computation, defect, P1)

Description

[Boris Zbarsky [:bzbarsky]]

And generally all the -moz properties...

Comment 1

[Boris Zbarsky [:bzbarsky]]

Note that stylo apparently supports "-moz-transform-origin" but not "-moz-transform"?

Comment 2

[Bobby Holley (:bholley)]

This is probably another good one for CJ.

Comment 3

[u459114]

I will work on this from next week.

Comment 4

[u459114]

Attachment: -moz-transform test case

Comment 5

[u459114]

hmm, generally, all -moz prop is not work.
Since webkit prefix works correctly, I will look around to see how it works.

Comment 6

[Boris Zbarsky [:bzbarsky]]

If you look in servo/components/style/properties/longhand/box.mako.rs you have lines like this:

  <%helpers:longhand name="transform-origin" animatable="True" extra_prefixes="moz webkit" boxed="True"

vs

  <%helpers:longhand name="transform" products="gecko servo" extra_prefixes="webkit"

note the extra_prefixes bits.

Looks to me like "transform-style" does the right thing already.  Anyway, the point is we should go through layout/style/nsCSSPropAliasList.h and check that all those do the right thing.

The -moz-transform case is harder because it is NOT an alias.  See the aDisallowRelativeValues argument to ParseTransform in Gecko.  It's false for "transform" but true for "-moz-transform".  I think the main impact is that percentage values are not allowed in -moz-transform?  Needs checking.

Comment 7

[Bobby Holley (:bholley)]

Nazim is going to work on this in bug 1351356.