1308785 - Support calc() in transform: scale() and rotate()

Status: RESOLVED FIXED

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

Description

[davidkpiano]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:49.0) Gecko/20100101 Firefox/49.0
Build ID: 20160922113459

Steps to reproduce:

This CodePen demonstrates the issue: http://codepen.io/davidkpiano/pen/85100956149faf1d6069c42644b0a659

Steps to reproduce:
1. Add the `transform: scale(calc(1 + 1));` rule to any element (using the dev tools or otherwise)
2. Assert that the element is scaled as if it had the rule `transform: scale(2);`.


Actual results:

The element is not scaled, as the rule is considered invalid by Firefox.


Expected results:

The element should be scaled to twice its size, as if it had the rule `transform: scale(2);`.

Comment 1

[davidkpiano]

A similar issue exists with rotate()

rotateY(calc(1 * 30deg))

does not work. It should behave as if it were:

rotateY(30deg)

Comment 2

[Brian Birtles (:birtles, Feb 26~ parental leave)]

Comment 1 appears to be a dupe of bug 956573. Not sure about the original bug, however.

Comment 3

[Kuba Niewiarowski]

Resolved with Stylo, right?

Comment 4

[Brian Birtles (:birtles, Feb 26~ parental leave)]

Confirmed this is fixed in Stylo builds (i.e. version 57+ on desktop). Marking this as blocked on supporting Stylo in Android although perhaps we should just dupe it to that bug since this will be fixed as soon as that bug is.

Comment 6

[Brian Birtles (:birtles, Feb 26~ parental leave)]

We never came to a very clear agreement about how to handle bugs fixed by Stylo (Daniel, Astley, and Xidorn had slightly different ideas). However, I think we agree to make them block stylo-behavior-change and mark them fixed either:

1. as soon as they were fixed on m-c, or
2. as soon as Stylo was enabled on Android, or
3. as soon as we remove the old style system

Now that we have enabled Stylo on Android (bug 1366049), i.e. 2 from above, I think it's probably reasonable to mark this fixed.

I'm a bit lost as to how to update the status flags though for now I'll just mark this fixed from 57 onwards.