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