Brian Birtles (:birtles, Feb 26~ parental leave) Reporter
	Description • 6 years ago

The CSS Timing spec has finally been updated to include the replacement to the previously specced frames() timing function:

  https://drafts.csswg.org/css-timing-1/#step-timing-functions

Basically it consists of four new step position keywords:

  jump-start
  jump-end
  jump-none
  jump-both

The first two are mostly aliases but at least 'jump-start' will need to be preserved as a separate keyword to 'start' so that they serialize differently. Both 'jump-end' and 'end' are not serialized so we don't need to preserve that distinction.

We should drop our implementation of frames() and add these keywords instead.

	Boris Chiou [:boris] Assignee
	Comment 1 • 6 years ago

Maybe I have time to work on this, given my other bugs have some spec issues.

	Brian Birtles (:birtles, Feb 26~ parental leave) Reporter
	Comment 2 • 6 years ago

Note that the CSS Timing Functions spec has just been renamed to CSS Easing Functions so if there are links to css-timing in tests and so on, it might be worthwhile to update them to css-easing while we're touching those files.

  https://drafts.csswg.org/css-easing/

	Boris Chiou [:boris] Assignee
	Comment 3 • 6 years ago

(In reply to Brian Birtles (:birtles) from comment #2)
> Note that the CSS Timing Functions spec has just been renamed to CSS Easing
> Functions so if there are links to css-timing in tests and so on, it might
> be worthwhile to update them to css-easing while we're touching those files.
> 
>   https://drafts.csswg.org/css-easing/

One of my patch will rename this. Besides, I'm start to think is it possible to drop/simplify the conversion between computed::TimingFunction and nsTimingFunction (i.e. gecko_bindings/sugar/ns_timing_function.rs)? Maybe we could do that after [1] is merged into cbindgen.

[1] https://github.com/eqrion/cbindgen/pull/219

	Boris Chiou [:boris] Assignee
	Comment 4 • 6 years ago

frames() timing function was removed from the spec, so we drop it.
Besides, some devtool tests are removed because they use frame(). I will
add them back by using new step function later.

	Boris Chiou [:boris] Assignee
	Comment 5 • 6 years ago

Rename css-timing to css-easing according to the spec update.

Depends on D9309

	Boris Chiou [:boris] Assignee
	Comment 6 • 6 years ago

TimingFunction is defined in a separate spec (i.e. css-easing), instead
of transform, so we move it into a different file.

Depends on D9310

	Boris Chiou [:boris] Assignee
	Comment 7 • 6 years ago

So we can generate generic enum by cbindgen (for TimingFunction).

Depends on D9311

	Boris Chiou [:boris] Assignee
	Comment 8 • 6 years ago

First, we generate StyleComputedTimingFunction by cbindgen from Rust, and use
it in nsTimingFunction, so we could copy it directly without handling
the different memory layout. However, we have to rewrite the
nsTimingFunction and mozilla::ComputedTimingFunction for this.

Second, the rust-bindgen seems cannot generate the correct generic members
from complex C++ templates, especially for the nested template struct:
e.g.

template<A, B>
union Foo {
  struct Bar1 {
    A mA;
  };
  struct Bar2 {
    B mB;
  };

  Bar1 bar1;
  Bar2 bar2;
};

The generated rust code may look like:

pub struct Foo<A, B> {
  // Foo_Bar1_Body should have one template parameter.
  pub bar1: root::__BindgenUnionField<root::Foo_Bar1_Body>,
  // Foo_Bar2_Body should have one template parameter.
  pub bar2: root::__BindgenUnionField<root::Foo_Bar2_Body>,
}
pub struct Foo_Bar1_Body<A> { ... }
pub struct Foo_Bar2_Body<B> { ... }

So we have to hide StyleTimingFunction to avoid the compilation errors.

Depends on D9312

	Boris Chiou [:boris] Assignee
	Comment 9 • 6 years ago

1. Add a new preference, layout.css.step-position-jump.enabled, for
   step(_, jump-*) timing functions.
2. We still keep Start and End tags, even though there is no difference
   between them. Therefore, we could disable the preference if needed.
3. Update the calculation of StepTiming to match the algorithm in the spec.

Depends on D9313

	Boris Chiou [:boris] Assignee
	Comment 10 • 6 years ago

Add new mochitests, and web-platform tests, and devtool tests.

Depends on D9314

	Boris Chiou [:boris] Assignee
	Comment 11 • 6 years ago

We make sure the step number is always positive, so using
computed::Integer is safe and can derive ToComputedValue.

Depends on D9311

	Boris Chiou [:boris] Assignee
	Comment 13 • 6 years ago

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b2d7e0457825aa9843a5cb85948daa66a490984c

	Pulsebot
	Comment 14 • 6 years ago

Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b46aa2883e42
Part 1: Drop frames() timing function r=birtles
https://hg.mozilla.org/integration/autoland/rev/e9bf605e24a2
Part 2: Rename css-timing to css-easing r=birtles
https://hg.mozilla.org/integration/autoland/rev/dca7e62232c1
Part 3: Split TimingFunction into a separate file to match spec r=emilio
https://hg.mozilla.org/integration/autoland/rev/d6e5970c62be
Part 4: Replace u32 with computed::Integer for computed::TimingFunction r=emilio
https://hg.mozilla.org/integration/autoland/rev/5b744b960fe0
Part 5: Bump cbindgen to 0.6.6 r=emilio
https://hg.mozilla.org/integration/autoland/rev/79675a8112d6
part 6: Generate StyleTimingFunction and drop ns_timing_function.rs r=emilio,birtles
https://hg.mozilla.org/integration/autoland/rev/77d9e79ed3b9
part 7: Implement steps(jump-*) functions r=birtles
https://hg.mozilla.org/integration/autoland/rev/24936aee7543
part 8: Tests r=birtles

	Natalia Csoregi [:nataliaCs]
	Comment 15 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/b46aa2883e42
https://hg.mozilla.org/mozilla-central/rev/e9bf605e24a2
https://hg.mozilla.org/mozilla-central/rev/dca7e62232c1
https://hg.mozilla.org/mozilla-central/rev/d6e5970c62be
https://hg.mozilla.org/mozilla-central/rev/5b744b960fe0
https://hg.mozilla.org/mozilla-central/rev/79675a8112d6
https://hg.mozilla.org/mozilla-central/rev/77d9e79ed3b9
https://hg.mozilla.org/mozilla-central/rev/24936aee7543

	Pulsebot
	Comment 16 • 6 years ago

Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cd47bdbd4efa
Update min cbindgen version in osx.py.

	Pulsebot
	Comment 17 • 6 years ago

Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/6771e96bb376
Update min cbindgen version in osx.py. a=emilio

	Boris Chiou [:boris] Assignee
	Comment 18 • 6 years ago

(In reply to Pulsebot from comment #16)
> Pushed by emilio@crisal.io:
> https://hg.mozilla.org/integration/mozilla-inbound/rev/cd47bdbd4efa
> Update min cbindgen version in osx.py

Thanks for this!

	Dorel Luca [:dluca]
	Comment 19 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/cd47bdbd4efa

	Brian Birtles (:birtles, Feb 26~ parental leave) Reporter
	Comment 20 • 6 years ago

Looks like the test changes didn't get upstreamed. Filed bug 1502788 for that.

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 21 • 6 years ago

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13868 for changes under testing/web-platform/tests

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 22 • 6 years ago

Upstream PR merged

	Chris Mills [:cmills]
	Comment 23 • 5 years ago

Note to MDN writers:

I have added a note to the Fx65 rel notes to cover this (two actually):
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/65#CSS

In terms of the documentation, there are a number of places this is mentioned that will need to be updated to remove mention of frames(), and add in description of the new keywords and how they are used:
https://developer.mozilla.org/en-US/docs/Web/CSS/transition-timing-function
https://developer.mozilla.org/en-US/docs/Web/CSS/animation-timing-function
https://developer.mozilla.org/en-US/docs/Web/CSS/single-transition-timing-function
https://developer.mozilla.org/en-US/docs/Web/CSS/animation
https://developer.mozilla.org/en-US/docs/Web/CSS/transition

Syntax in the MDN data repo and browser compat data will also need updating.

For the spec definitions, see:
https://drafts.csswg.org/css-easing-1/#step-timing-functions