Remove Option<>s from TransformOperation.

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P3
normal
RESOLVED FIXED
5 months ago
4 months ago

People

(Reporter: emilio, Assigned: emilio)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(5 attachments)

They're not usable via FFI, and they should not be needed. This patches also change the serialization of transform operations to follow the same principle we use for single transform properties.

I think that slight change is fine, if we can make it work, and consistent with other properties. Alternative is adding more TransformOperation variants, which I rather not do.

I called it contextual_skip_if, though better names welcome.

Otherwise, deriving ToComputedValue and ToAnimatedValue in structs or enums with
other where clauses just doesn't work.

Depends on D21858

Trivial drive-by cleanup.

Depends on D21859

Also drive-by cleanup.

Depends on D21860

This may or may not be part of the plan to get rid of nsCSSValue ;)

Option is not usable via FFI, and they should not be needed (we should be
following the shortest serialization principle instead). These patches also do
that, which matches the other transform properties. I think that slight change
is fine, if we can make it work, and consistent with other properties.

Alternative is adding more TransformOperation variants or such, which I rather
not do.

Depends on D21861

Summary: Remove Option<>s from TransformOperatior. → Remove Option<>s from TransformOperation.
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b5dec9e96f42
Implement a version of #[css(skip_if)] that takes more context. r=xidorn
https://hg.mozilla.org/integration/autoland/rev/7aa72a6aa1b5
When deriving something with an output type, map preconditions as well. r=xidorn
https://hg.mozilla.org/integration/autoland/rev/5a4a2f2465b0
Use skip_if for translate serialization. r=xidorn
https://hg.mozilla.org/integration/autoland/rev/2b811ffbed9b
Use contextual_skip_if for background-size. r=xidorn
https://hg.mozilla.org/integration/autoland/rev/950681627741
Remove Options from TransformOperation. r=xidorn
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/15715 for changes under testing/web-platform/tests
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.