searching for reviewer(hiro)
58b3e727210440e6a7c26cf664ddc03f8d616822: Bug 1484901 - Add tolerance for time comparison in playing-an-animation.html; r=hiro
Brian Birtles <birtles@gmail.com> - Tue, 21 Aug 2018 21:39:02 +0000 - rev 830682
Push
118847 by bmo:hskupin@gmail.com at Wed, 22 Aug 2018 09:09:03 +0000
Bug 1484901 - Add tolerance for time comparison in playing-an-animation.html; r=hiro
This test appears to occasionally fail. The suspected cause is a combination of:
* timer clamping (due to vsync) resulting in the current timeline time after
resuming playback and the "ready time" being equal
* difference in floating-point precision used in calculating the two measures --
one calculation done in JS and one done using the UA's internal representation
of times.
Therefore this patch adds the standard tolerance for comparing time values to
this comparison.
Differential Revision:
https://phabricator.services.mozilla.com/D3857
dfac768978de71ff96351842189f4f242f0d99fc: Bug 1484148 - Make CSSTransition startTime test timeout when waiting for transition events after a fixed number of frames; r=hiro
Brian Birtles <birtles@gmail.com> - Mon, 20 Aug 2018 13:09:41 +0900 - rev 830416
Push
118832 by bmo:ntim.bugs@gmail.com at Tue, 21 Aug 2018 13:33:17 +0000
Bug 1484148 - Make CSSTransition startTime test timeout when waiting for transition events after a fixed number of frames; r=hiro
See
bug 1467344 comment 34.
Differential Revision:
https://phabricator.services.mozilla.com/D3753
ba0eda3c4965d94f9537aace472d65118ea1d11c: Bug 1484148 - Add a test for resuming from paused; r=hiro
Brian Birtles <birtles@gmail.com> - Mon, 20 Aug 2018 13:09:34 +0900 - rev 830415
Push
118832 by bmo:ntim.bugs@gmail.com at Tue, 21 Aug 2018 13:33:17 +0000
Bug 1484148 - Add a test for resuming from paused; r=hiro
As
bug 1467344 comment 40, it appears we don't currently have any tests that
cover resuming a paused animation.
Differential Revision:
https://phabricator.services.mozilla.com/D3752
877cdf25b483aaccf75c0ef6821194523dcce259: Bug 1467344 - Update corresponding tests for CSS animations to make them consistent; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:06 +0900 - rev 830139
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update corresponding tests for CSS animations to make them consistent; r=hiro
This is a collection of fix-ups to the corresponding tests for CSS animations to
apply the same sort of naming etc. that we have now applied to CSS transitions
tests.
0394a44b3955e4230a79cd8226a5cd779cf068ee: Bug 1467344 - Update event-dispatch test now that we can use the updated EventWatcher; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:05 +0900 - rev 830138
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update event-dispatch test now that we can use the updated EventWatcher; r=hiro
37038c5ebcb4450b1a008743be8580c140e3102b: Bug 1467344 - Add spec links to test files; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:05 +0900 - rev 830137
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Add spec links to test files; r=hiro
d0c3ab5c291b0743911fc3b62938e3a2c130f20c: Bug 1467344 - Move tests from dom/animations/tests/css-transitions to WPT; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:04 +0900 - rev 830136
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Move tests from dom/animations/tests/css-transitions to WPT; r=hiro
5b8c402641ac0a302770b6661b5f01c77226c70e: Bug 1467344 - Drop use of flushComputedStyle in css-transitions tests; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:04 +0900 - rev 830135
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Drop use of flushComputedStyle in css-transitions tests; r=hiro
flushComputedStyle is as follows:
var cs = getComputedStyle(elem);
cs.marginLeft;
That _probably_ flushes style in most engines, but it's possible some engine in
the future will optimize things in a way that it can flush marginLeft without
flushing, say, backgroundColor. Before moving these tests to wpt, it would be
better to explicitly flush the property (or at least one of the properties) we
are transitioning.
This patch also updates the "Setting zero combined duration" test from
test_animation-cancel.html since, in making this change, I realized the test was
wrong.
Specifically, it was testing that when we set a combined duration of zero that
a transition is canceled. However, nothing in the spec requires this. The spec
only requires a non-zero combined duration to _start_ a transition and only
cancels an existing transition if the combined duration becomes _zero_ if the
"the end value of the running transition is not equal to the value of the
property in the after-change style".
This test passed, however, because it changed the transition property to
margin-top, hence it was actually testing the same condition as the previous
test.
64abf6978f00f919aa926d56b6ce908392549529: Bug 1467344 - Update tests in test_setting-effect.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830134
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_setting-effect.html; r=hiro
This patch splits the first test into four separate tests since it really seems
to be testing four different things.
Likewise, the later patch for replacing the effect is split into two parts:
- One to test the playState behavior
- One to test the value reported by transitionProperty
The test, "After setting a transition's effect to null, transitionend is still
dispatched", is dropped since it is covered by the last test in
test_event-dispatch.html.
0160e2c379e3a1874efc9ffd75691c1da208ad7f: Bug 1467344 - Update tests in test_keyframeeffect-getkeyframes.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830133
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_keyframeeffect-getkeyframes.html; r=hiro
This patch merely tidies up some whitespace and uses of quotes / template
strings. There are no substantive changes.
45a715e4cf0d31ef2ffa6fcd843112180ba38378: Bug 1467344 - Update tests in test_animation-element-get-animations.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830132
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_animation-element-get-animations.html; r=hiro
This mostly just tidies up these tests to give them sensible titles.
The only test with major substantive changes is the first test. This test tests
three things:
1) That getAnimations() returns one transition per transitioning property
2) That getAnimations() returns transitions in the order they were generated
3) That CSS transitions have their start time set based on when they were
generated.
(2) is covered later in this file by the test: 'getAnimations sorts
transitions by when they were generated'
(3) is really a test for startTime, if anything, not getAnimations().
I'm not sure how necessary it is, but I've added it to
test_animation-starttime.html for now.
As a result, this patch updates this first test to only cover (1).
ffce5e4f4d45e90f116bb0bf7a4c9953009c6c67: Bug 1467344 - Tidy up test in test_csstransition-transitionproperty.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830131
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Tidy up test in test_csstransition-transitionproperty.html; r=hiro
This is mostly whitespace tidy-ups. The only substantive change is the fix to
the test description which previously seemed to be missing some words.
2fcae020385795acd81f858538438708aa3763d0: Bug 1467344 - Update tests in test_animation-starttime.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830130
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_animation-starttime.html; r=hiro
Apart from obvious tidy-ups the substantive changes to tests in this file are as
follows:
* Skipping forward through animation
-> This is really testing two things:
(a) That you can seek a transition using the start time.
(b) That seeking a transition using the start time triggers
dispatching events.
This patch splits the above into two separate tests.
* Skipping backwards through animation,
-> All these tests are really just exercising event dispatch which is
already covered by test_event-dispatch.html.
As a result this patch drops these tests.
* Setting startTime to null
-> Covered by 'Setting an unresolved start time sets the hold time'
in wpt/web-animations/timing-model/animations/setting-the-start-time-of-an-animation.html
* Animation.startTime after pausing
-> Covered by 'Pausing clears the start time'
in wpt/web-animations/timing-model/animations/pausing-an-animation.html
a4a1c2383121e53bb913333eb04ac6193fa9133c: Bug 1467344 - Update tests in test_animation-ready.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830129
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_animation-ready.html; r=hiro
The test: 'A new ready promise is created each time play() is called the
animation property', is covered in:
web-animations/interfaces/Animation/ready.html
by the 'A new ready promise is created when play()/pause() is called' test.
As a result, this patch removes that test.
a611b59253c5c9865119e88e376e5dd8631fe461: Bug 1467344 - Drop test_animation-pausing.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830128
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Drop test_animation-pausing.html; r=hiro
This seems to be adequately covered by:
web-animations/timing-model/animations/pausing-an-animation.html
(specifically the last test: 'The animation's current time remains fixed
after pausing')
web-animations/timing-model/animation-effects/simple-iteration-progress.html
(which checks that the iteration progress is correctly calculated from the
current time)
b023fa315b32aaaea6d59184262f37665b1560cc: Bug 1467344 - Update tests in test_animation-finished.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830127
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_animation-finished.html; r=hiro
For the first test, we really want to check that it is possible to restart
transitions (since they are otherwise disassociated once they finish) hence this
patch updates the test to the actual output in style rather than timing.
The second test is covered in
web-animations/timing-model/animations/reversing-an-animation.html
by the 'Playing a finished and reversed animation seeks to end' test so this
patch removes it from this file.
c5722de174bc37cfd768e62366da5acc22298969: Bug 1467344 - Update tests in test_animation-currenttime.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:03 +0900 - rev 830126
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update tests in test_animation-currenttime.html; r=hiro
This patch basically completely rewrites this file to make it simpler and remove
unnecessary tests.
In particular:
* It sets the duration and delay to 100s to match what we use in most tests
* All the helper methods at the start are inlined into the relevant tests
because doing so makes the tests much easier to read and debug.
* The first two tests are combined and extended
- Testing the initial current time is zero is not generally useful but it is
when we're testing setting it
- What the second test really wants to test is that the style updates (i.e.
you really can seek a transition, not just that the currentTime value
updates)
- The trick where we set the start time just to get out of the pending state
seems unnecessary, we may as well just wait on ready. In fact, we don't
_need_ to do that, but this test is about checking you can seek a transition
in motion.
* In the fourth test, 'Skipping backwards through transition', we don't check
the event is dispatched since that is covered in test_event-dispatch.html
in the 'Active -> Before' test.
* The final test, 'Animation.currentTime after pausing', is covered by
web-animations/timing-model/animations/pausing-an-animation.html
242d779840719472bea5adcd114d2a1bdb08eb96: Bug 1467344 - Fix some minor formatting in test_animation-computed-timing.html; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830125
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Fix some minor formatting in test_animation-computed-timing.html; r=hiro
14200486590f8f2e121c74ded7986dccd12ff5ef: Bug 1467344 - Drop some unnecessary <body> elements from test files; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830124
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Drop some unnecessary <body> elements from test files; r=hiro
We're a bit inconsistent about this, but generally we try to keep this files as
minimal as possible.
There is still a lot of other cruft in the starttime and currenttime test files
but we will remove that when we tidy up those files in later patches in this
series.
fbdd599db4490ea7e651ef7f2a05c5f6584f6c0d: Bug 1467344 - Update the titles of each test file; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830123
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Update the titles of each test file; r=hiro
9427f6cff88ff691b812846f819c6e93e1ef1aea: Bug 1467344 - Fix the spelling of canceled/canceling in dom/animation/test/css-transitions/; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830122
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Fix the spelling of canceled/canceling in dom/animation/test/css-transitions/; r=hiro
5097e03c335b588d7d3a3e369dd4f3785ea908a5: Bug 1467344 - Replace var with const/let in dom/animation/test/css-transitions/; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830121
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Replace var with const/let in dom/animation/test/css-transitions/; r=hiro
8d053037352a55cb0ff3f6900e43679901c23d6d: Bug 1467344 - Use async/await in dom/animation/test/css-transitions/; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:02 +0900 - rev 830120
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Use async/await in dom/animation/test/css-transitions/; r=hiro
As part of this change, the "Restart transition after cancelling transition
immediately" test in test_event-dispatch.html needed significant changes.
The reason is that this test was calling:
watcher.wait_for([ 'transitioncancel',
'transitionrun',
'transitionstart' ]);
However, it was not waiting for the result of that call and hence was not
actually checking if the events were being dispatched. And, they are not.
There are two problems. Firstly, this test cancels the transition by setting
display:none. However, transitions don't run on display:none elements so
attempting to restart the transition will not generate transition events.
This might be a bug, but it is not a recent regression in any case (I tested
back to Firefox 54).
However, this test does not require using display:none to cancel. There are
_many_ tests that check that display:none generates a transitioncancel event.
This test only needs to make the transition idle. As a result, this patch makes
that test call transition.cancel() instead.
However, even with that change this test will not pass because it sets
a transition-delay of 100s (presumably so that it does not need to check for
a transitionstart event). As a result this test should not wait on _both_
transitionrun and transitionstart but just transitionrun.
a8ae57ad9f1595a88c75ef0b3c41760c94e1fc27: Bug 1467344 - Use arrow functions in dom/animation/test/css-transitions/; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 16 Aug 2018 14:41:01 +0900 - rev 830119
Push
118817 by bmo:ntim.bugs@gmail.com at Mon, 20 Aug 2018 09:00:50 +0000
Bug 1467344 - Use arrow functions in dom/animation/test/css-transitions/; r=hiro
This patch also drops the checkStateOnSettingCurrentTimeToZero function since it
is no longer used.
78be4bbf4b050f6614bb9f4115f57fb61f4890df: Bug 1479681: Fix the loop in nsTransitionManager dealing with stopping `all` transitions. r=hiro
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 31 Jul 2018 11:32:53 +0200 - rev 824736
Push
117996 by bmo:emalysz@mozilla.com at Tue, 31 Jul 2018 19:30:35 +0000
Bug 1479681: Fix the loop in nsTransitionManager dealing with stopping `all` transitions. r=hiro
The loop was mutating the nsCSSPropertyID used to guard the exit, which is
obviously wrong.
This branch is pretty rarely taken, since people don't usually specify `all` as
a transition property other than the first, for which case we take the fast path
with `checkProperties = false`. Our test-suite failed to catch this.
Added a crashtest that hangs without this patch.
The reason
bug 1478990 regressed this is because it changed the order of
nsCSSPropertyID so that `p` actually went backwards causing the infinite loop,
but the bug was introduced (by me, whoops) in
bug 1309752.
Differential Revision:
https://phabricator.services.mozilla.com/D2552
MozReview-Commit-ID: Ii3D1FaZ31R
0d72c7996d60a7c07e35c5f90d78b02a47d17460: Bug 1479681: Fix the loop in nsTransitionManager dealing with stopping `all` transitions. r=hiro a=aryx
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 31 Jul 2018 11:32:53 +0200 - rev 824607
Push
117945 by bmo:mikokm@gmail.com at Tue, 31 Jul 2018 10:52:49 +0000
Bug 1479681: Fix the loop in nsTransitionManager dealing with stopping `all` transitions. r=hiro a=aryx
The loop was mutating the nsCSSPropertyID used to guard the exit, which is
obviously wrong.
This branch is pretty rarely taken, since people don't usually specify `all` as
a transition property other than the first, for which case we take the fast path
with `checkProperties = false`. Our test-suite failed to catch this.
Added a crashtest that hangs without this patch.
The reason
bug 1478990 regressed this is because it changed the order of
nsCSSPropertyID so that `p` actually went backwards causing the infinite loop,
but the bug was introduced (by me, whoops) in
bug 1309752.
Differential Revision:
https://phabricator.services.mozilla.com/D2552
MozReview-Commit-ID: Ii3D1FaZ31R
34b671e3dd5d29ee2c4429b816bbb37a8b764f63: Bug 1479234 - Include missing headers in D3D11YCbCrImage.cpp and VRLayerChild.cpp. r=hiro
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Tue, 31 Jul 2018 08:13:19 +0900 - rev 824598
Push
117945 by bmo:mikokm@gmail.com at Tue, 31 Jul 2018 10:52:49 +0000
Bug 1479234 - Include missing headers in D3D11YCbCrImage.cpp and VRLayerChild.cpp. r=hiro
MozReview-Commit-ID: C8VQO46Lm1
dd1e164c5cecc77308d75f2e710af10f1998e5a3: Bug 1479234 - Include missing headers in D3D11YCbCrImage.cpp and VRLayerChild.cpp. r?hiro
draft
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Tue, 31 Jul 2018 08:13:19 +0900 - rev 824477
Push
117927 by hikezoe@mozilla.com at Tue, 31 Jul 2018 00:42:56 +0000
Bug 1479234 - Include missing headers in D3D11YCbCrImage.cpp and VRLayerChild.cpp. r?hiro
MozReview-Commit-ID: C8VQO46Lm1
b1e7eb636989823eb14df671c2362efd95f49cb4: Bug 1479239: prefers-reduced-motion should be sensible re. unsupported platforms / resistFingerprinting. r=hiro
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 29 Jul 2018 03:18:19 +0200 - rev 823871
Push
117809 by mozilla@buttercookie.de at Sun, 29 Jul 2018 18:19:15 +0000
Bug 1479239: prefers-reduced-motion should be sensible re. unsupported platforms / resistFingerprinting. r=hiro
With the current early-returning behavior, the expression will unconditionally
evaluate to false, which is a bit of a footgun.
Make sure to always return no-preference in unsupported platforms or when
resisting fingerprinting.
Differential Revision:
https://phabricator.services.mozilla.com/D2491
MozReview-Commit-ID: 41uUudut7b4
dc5ea0cbf005b65654b6ed8f44f447011295f0e3: Bug 1479230: Remove useless condition in GetPrefersReducedMotion. r=hiro
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 29 Jul 2018 02:11:53 +0200 - rev 823867
Push
117809 by mozilla@buttercookie.de at Sun, 29 Jul 2018 18:19:15 +0000
Bug 1479230: Remove useless condition in GetPrefersReducedMotion. r=hiro
This query is always enabled in content, so this check is always true.
Differential Revision:
https://phabricator.services.mozilla.com/D2490
MozReview-Commit-ID: 5etk5TM0agz
e9fd7073981480be9893eda1590d1bdb9c764e24: Bug 1478213 - Don't ignore a redundant change to setting the animation start time if we are pending; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 26 Jul 2018 15:07:52 +0900 - rev 823063
Push
117560 by rwood@mozilla.com at Thu, 26 Jul 2018 14:25:44 +0000
Bug 1478213 - Don't ignore a redundant change to setting the animation start time if we are pending; r=hiro
As discussed here:
https://github.com/w3c/csswg-drafts/issues/2691
We have a similar check in SetCurrentTime (with the exception that, according to
the spec, this behavior applies to either play OR pause pending, instead of just
pause-pending) so this patch tries to match the comment and format of that
check.
Differential Revision:
https://phabricator.services.mozilla.com/D2410
420ef12d797168aa361cbb75ee2cc54fc24e80d2: Bug 1478213 - Add test that fails if we ignore a seemingly redundant setting of startTime to null; r=hiro
Brian Birtles <birtles@gmail.com> - Thu, 26 Jul 2018 15:07:35 +0900 - rev 823062
Push
117560 by rwood@mozilla.com at Thu, 26 Jul 2018 14:25:44 +0000
Bug 1478213 - Add test that fails if we ignore a seemingly redundant setting of startTime to null; r=hiro
Differential Revision:
https://phabricator.services.mozilla.com/D2409
14958295caced9e6a87796c2e3b2b694d00a4d86: Bug 1477486 - Add dom.animations-api.getAnimations.enabled pref to css-logical test suite; r=hiro
Brian Birtles <birtles@gmail.com> - Mon, 23 Jul 2018 09:21:23 +0900 - rev 821368
Push
117078 by bmo:gl@mozilla.com at Mon, 23 Jul 2018 05:29:28 +0000
Bug 1477486 - Add dom.animations-api.getAnimations.enabled pref to css-logical test suite; r=hiro
In particular the animation-003 test needs it but we may as well turn it on for
all tests since we'll possibly add more tests like this in the future.
28c58c3b91fcf9bf061e0ad23faf5f87ea0f3202: Bug 1477486 - Fix typo in animation-003.tenative.html filename; r=hiro
Brian Birtles <birtles@gmail.com> - Mon, 23 Jul 2018 09:21:08 +0900 - rev 821367
Push
117078 by bmo:gl@mozilla.com at Mon, 23 Jul 2018 05:29:28 +0000
Bug 1477486 - Fix typo in animation-003.tenative.html filename; r=hiro
fe00b76f0e9f6095ecc58be302b8d9c5e008311e: Bug 1477486 - Add dom.animations-api.getAnimations.enabled pref to css-logical test suite; r?hiro
draft
Brian Birtles <birtles@gmail.com> - Mon, 23 Jul 2018 09:21:23 +0900 - rev 821333
Push
117063 by bmo:bbirtles@mozilla.com at Mon, 23 Jul 2018 00:21:50 +0000
Bug 1477486 - Add dom.animations-api.getAnimations.enabled pref to css-logical test suite; r?hiro
In particular the animation-003 test needs it but we may as well turn it on for
all tests since we'll possibly add more tests like this in the future.
MozReview-Commit-ID: FaRgJkV2jUs
f8c34fb8697f69a7be8c555e7cf97c7dfd3ce6e7: Bug 1477486 - Fix typo in animation-003.tenative.html filename; r?hiro
draft
Brian Birtles <birtles@gmail.com> - Mon, 23 Jul 2018 09:21:08 +0900 - rev 821332
Push
117063 by bmo:bbirtles@mozilla.com at Mon, 23 Jul 2018 00:21:50 +0000
Bug 1477486 - Fix typo in animation-003.tenative.html filename; r?hiro
MozReview-Commit-ID: 5LgZTUqVtKH
86eaab5999fb94f7c494263f76b897902b59e7a8: Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r=bz,hiro
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819462
Push
116562 by bmo:gsquelart@mozilla.com at Tue, 17 Jul 2018 22:54:34 +0000
Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r=bz,hiro
This is probably the last thing we will ship since it needs the most spec work.
MozReview-Commit-ID: LLmDBLCsCBJ
bcc8cc984ddf661c63b390becfbb952c1d4ee638: Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r=hiro
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819461
Push
116562 by bmo:gsquelart@mozilla.com at Tue, 17 Jul 2018 22:54:34 +0000
Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r=hiro
MozReview-Commit-ID: 9Xn32jLlpq1
f302f2d706fba26847018e9f961548e5b5f38f5d: Bug 1471814 - Add a preference for animation composite modes; r=bz,hiro
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819460
Push
116562 by bmo:gsquelart@mozilla.com at Tue, 17 Jul 2018 22:54:34 +0000
Bug 1471814 - Add a preference for animation composite modes; r=bz,hiro
This feature should not be shipped until the various definitions of addition for
each additive property are properly specified.
Unlike other patches in this series, compositing is not frequently used
internally (e.g. by DevTools etc.) so there is no need to enable this by default
for system code.
Also, it turns out we have inadvertently been shipping part of this feature for
some time now. The next patch in this series will add tests for that case and
disable that part of the feature (a suitable intent to unship will follow). This
patch merely adapts and extends the existing tests without affecting the surface
area covered by the combination of the newly-added pref and the existing
dom.animations-api.core.enabled pref.
MozReview-Commit-ID: Htr6mlyCBav
bb1c7e6f5d7a96d37baf2b2177e7ea2631cc8dfb: Bug 1471814 - Add a preference for implicit keyframes; r=bz,hiro
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819459
Push
116562 by bmo:gsquelart@mozilla.com at Tue, 17 Jul 2018 22:54:34 +0000
Bug 1471814 - Add a preference for implicit keyframes; r=bz,hiro
This preference controls whether authors are allowed to specify animations
without a 0% or 100% keyframe.
We intend to ship this soon but this preference acts as a safeguard in case we
discover we need to disable it.
This feature is very convenient and commonly used so this patch ensures it is
always enabled for system content.
MozReview-Commit-ID: BHTsuS2xO61
7b9959f1bc8037ffda94f98fa6e5668120fc1711: Bug 1471814 - Add a preference for Web Animations timelines; r=bz,hiro
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819458
Push
116562 by bmo:gsquelart@mozilla.com at Tue, 17 Jul 2018 22:54:34 +0000
Bug 1471814 - Add a preference for Web Animations timelines; r=bz,hiro
We don't intend to ship this in the near future until the integration with
AnimationWorklet is clear (although we might ship a read-only version).
That said, we use this feature extensively internally (e.g. in DevTools etc.) so
we enable this feature for system callers.
MozReview-Commit-ID: AhB7ZmU1Xzw
8a4fd9ccbfd7b09e31630e341e4be7ffdc65e728: Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819086
Push
116436 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 05:43:02 +0000
Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
This is probably the last thing we will ship since it needs the most spec work.
MozReview-Commit-ID: LLmDBLCsCBJ
72950710a8b34ba1817bbc49cae8aa91fb70ff3f: Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r?hiro
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819085
Push
116436 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 05:43:02 +0000
Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r?hiro
MozReview-Commit-ID: 9Xn32jLlpq1
e5312332dc0c87414e27cbcab4e23c6c5348d9cf: Bug 1471814 - Add a preference for animation composite modes; r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819084
Push
116436 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 05:43:02 +0000
Bug 1471814 - Add a preference for animation composite modes; r?hiro, r?bz
This feature should not be shipped until the various definitions of addition for
each additive property are properly specified.
Unlike other patches in this series, compositing is not frequently used
internally (e.g. by DevTools etc.) so there is no need to enable this by default
for system code.
Also, it turns out we have inadvertently been shipping part of this feature for
some time now. The next patch in this series will add tests for that case and
disable that part of the feature (a suitable intent to unship will follow). This
patch merely adapts and extends the existing tests without affecting the surface
area covered by the combination of the newly-added pref and the existing
dom.animations-api.core.enabled pref.
MozReview-Commit-ID: Htr6mlyCBav
f71b4a1eea91f53a69712d904d1bb641fa297366: Bug 1471814 - Add a preference for implicit keyframes; r?hiro r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819083
Push
116436 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 05:43:02 +0000
Bug 1471814 - Add a preference for implicit keyframes; r?hiro r?bz
This preference controls whether authors are allowed to specify animations
without a 0% or 100% keyframe.
We intend to ship this soon but this preference acts as a safeguard in case we
discover we need to disable it.
This feature is very convenient and commonly used so this patch ensures it is
always enabled for system content.
MozReview-Commit-ID: BHTsuS2xO61
0406bb33a0f7efd82a6f931ee75d1087e45dc71e: Bug 1471814 - Add a preference for Web Animations timelines; r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819082
Push
116436 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 05:43:02 +0000
Bug 1471814 - Add a preference for Web Animations timelines; r?hiro, r?bz
We don't intend to ship this in the near future until the integration with
AnimationWorklet is clear (although we might ship a read-only version).
That said, we use this feature extensively internally (e.g. in DevTools etc.) so
we enable this feature for system callers.
MozReview-Commit-ID: AhB7ZmU1Xzw
4b98671ca3500205d80249233aebb9751e36978e: Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819065
Push
116426 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 03:40:56 +0000
Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
This is probably the last thing we will ship since it needs the most spec work.
MozReview-Commit-ID: LLmDBLCsCBJ
d5b8e239408da33125e2058f2133a6ec6ff9f493: Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r?hiro
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819064
Push
116426 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 03:40:56 +0000
Bug 1471814 - Ignore the composite member on Keyframe objects when the compositing pref is not set; r?hiro
MozReview-Commit-ID: 9Xn32jLlpq1
123fe49e4b1500c6d9b1a212a087c0c14411b412: Bug 1471814 - Add a preference for animation composite modes; r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819063
Push
116426 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 03:40:56 +0000
Bug 1471814 - Add a preference for animation composite modes; r?hiro, r?bz
This feature should not be shipped until the various definitions of addition for
each additive property are properly specified.
Unlike other patches in this series, compositing is not frequently used
internally (e.g. by DevTools etc.) so there is no need to enable this by default
for system code.
Also, it turns out we have inadvertently been shipping part of this feature for
some time now. The next patch in this series will add tests for that case and
disable that part of the feature (a suitable intent to unship will follow). This
patch merely adapts and extends the existing tests without affecting the surface
area covered by the combination of the newly-added pref and the existing
dom.animations-api.core.enabled pref.
MozReview-Commit-ID: Htr6mlyCBav
add551af284260ecf337510de9872520b88f6e8e: Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
draft
Brian Birtles <birtles@gmail.com> - Sat, 14 Jul 2018 09:23:03 +0900 - rev 819041
Push
116420 by bmo:bbirtles@mozilla.com at Tue, 17 Jul 2018 01:10:47 +0000
Bug 1471814 - Add a preference for {Document,Element}.getAnimations(); r?hiro, r?bz
This is probably the last thing we will ship since it needs the most spec work.
MozReview-Commit-ID: LLmDBLCsCBJ