292a8eb611a81baafba5820de46ad46b2c8d4e09: Bug 1305325 - Part 7: Make SampleValue returning StyleAnimationValue. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439348
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 7: Make SampleValue returning StyleAnimationValue. r?birtles MozReview-Commit-ID: Izr6Cvee96Q
40f542cf7e731a391c29881f1a960b54ea46f8a3: Bug 1305325 - Part 6: Send animations even if it's paused, finished or zero playback rate. r?birtles. draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439347
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 6: Send animations even if it's paused, finished or zero playback rate. r?birtles. If all of animations on an element are paused, finished or zero playback rate, we don't send those animations to the compositor. MozReview-Commit-ID: CHjv6Buy5fa
9ae1c012e8ec11e86400e1aee459d5f485b59795: Bug 1305325 - Part 5: Make AnimationPropertyValueDetails::mValue optional. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439346
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 5: Make AnimationPropertyValueDetails::mValue optional. r?birtles MozReview-Commit-ID: AjSfaUHerqL
4cc2227bcfb545ff98adb9758ae68f6ea0e36f3e: Bug 1305325 - Part 4: Update base values for each property. r? draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439345
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 4: Update base values for each property. r? This patch adds a hashtable to store the non-animated base value of each property animated by the effects in EffectSet. The values are stored as StyleAnimationValue objects since they are intended to be used in KeyframeEffectReadOnly::ComposeStyle. 1) We have to update it even if there is no animation properties changes because changing non-animating style value does not cause any animation property changes. If we do it in Updateproperties() it's apparently less effective. 2) There is the case that we have no EffectSet in UpdateProperties(), e.g. creating an animation. What happens on Element.animate(); i) KeyframeEffect constructor i-1) UpdateProperties() ii) Animation constructor with the effect ii-1) KeyframeEffectReadOnly::SetAnimation() ii-2) The effect is added in EffectSet MozReview-Commit-ID: FPzyX9fxxx3
a7af1e013687423f7c72fc21adaf04700bd336d4: Bug 1305325 - Part 3: Add AnimValuesStyleRule::GetValue and HasValue to get the last composed style. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439344
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 3: Add AnimValuesStyleRule::GetValue and HasValue to get the last composed style. r?birtles HasValue() will be used to check that we have already composed style for a given property or not. MozReview-Commit-ID: L8lezGyON5o
0ed5e832d9dc32aefcbeb665d79516b9db78c810: Bug 1305325 - Part 2: Add AnimationProperty::HasNonReplaceComposite. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439343
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 2: Add AnimationProperty::HasNonReplaceComposite. r?birtles We will resolve base values only if HasNonReplaceComposite() returns true. MozReview-Commit-ID: LUui9BpuphW
3a6e115ed35a8fb8ead874f78f0afbb5291b3764: Bug 1305325 - Part 1: Move test cases in file_partial_keyframes.html into file_disable_animations_api_core.html. r?boris draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:30:57 +0900 - rev 439342
Push 35977 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:34:46 +0000
Bug 1305325 - Part 1: Move test cases in file_partial_keyframes.html into file_disable_animations_api_core.html. r?boris Missing keyframe will be handled only if the Web Animatins API is enabled. Test cases in file_partial_keyframes.html will not throw any exceptions once we handle missing keyframe because they are run with the Animations API pref, but still we should run the test cases with disabled the pref. So we are now going to move the test cases into file_disable_animations_api_code.html. We can't use KeyframeEffectReadOnly contructor in a new test file since the constructor is also behind the Web Animations API pref. MozReview-Commit-ID: 9ApVxJebBUj
06c2a35b19c2893d103a3d2d8aa021401cc1a4f0: Bug 1316373: add Blackboards domains to screenshare whitelist. r?mreavy draft
Nils Ohlmeier [:drno] <drno@ohlmeier.org> - Tue, 15 Nov 2016 13:28:21 -0800 - rev 439341
Push 35976 by drno@ohlmeier.org at Tue, 15 Nov 2016 21:29:57 +0000
Bug 1316373: add Blackboards domains to screenshare whitelist. r?mreavy MozReview-Commit-ID: AXRSnb4FAai
97bf3b4106480265871e7255ec47f48b6230c66a: Bug 1308061 - Part 1: Update SessionStore.jsm to notify when the list of closed objects changes draft
Bob Silverberg <bsilverberg@mozilla.com> - Wed, 09 Nov 2016 10:51:32 -0500 - rev 439340
Push 35975 by bmo:bob.silverberg@gmail.com at Tue, 15 Nov 2016 21:25:07 +0000
Bug 1308061 - Part 1: Update SessionStore.jsm to notify when the list of closed objects changes MozReview-Commit-ID: 7pBrvAhVQHP
e75b0e8229b6fc5b498a2a833f2ae46710c416c6: Bug 1305325 - Part 14: Tests composed style for missing keyframe for properties runnning on the compositor. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:47 +0900 - rev 439339
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 14: Tests composed style for missing keyframe for properties runnning on the compositor. r?birtles The error value, 0.01, used in this test is the same as we used in test_animation_omta.html. MozReview-Commit-ID: 50g3k43yAgu
bf9a4a61098a6d376fb21268d27ddfeb747c8c72: Bug 1305325 - Part 13: Compose base values on the compositor. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:47 +0900 - rev 439338
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 13: Compose base values on the compositor. r?birtles MozReview-Commit-ID: EWumBqQ82RT
5a38cbda859ad876935c43d4001a745bd8349a33: Bug 1305325 - Part 12: Tests composed style for missing keyframe for properties running on the main thread. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:47 +0900 - rev 439337
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 12: Tests composed style for missing keyframe for properties running on the main thread. r?birtles MozReview-Commit-ID: 7QuksI30bwY * * * [mq]: FIX MozReview-Commit-ID: LELkYGh35iY
ce5e890e411b6d4c17edf5c9a6cc5568dfaa62ac: Bug 1305325 - Part 11: The expected value of offset is 1.0 for input value whose offset is 1.0. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:47 +0900 - rev 439336
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 11: The expected value of offset is 1.0 for input value whose offset is 1.0. r?birtles MozReview-Commit-ID: JnsWmDBfW3p
bbf0d86e7f81ba6304a0157e27bf60e018a1e578: Bug 1305325 - Part 10: Handle missing keyframe whose offset 0 or 1 on the main thread. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:47 +0900 - rev 439335
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 10: Handle missing keyframe whose offset 0 or 1 on the main thread. r?birtles MozReview-Commit-ID: 5WMiTJQKfZd * * * [mq]: FIX MozReview-Commit-ID: 80NzO37ENrC
90f4feea80c7c8f9a179c220b47a83b2985225a9: Bug 1305325 - Part 9: Add AnimationUtils::IsCoreAPIEnable. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439334
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 9: Add AnimationUtils::IsCoreAPIEnable. r?birtles To check simulateComputeValuesFailure flag in chrome privilege, we will need to know the core Web Animations API preference is enabled in chrome privilege. MozReview-Commit-ID: GYRMc9Od4Bm
c94612b42b087ada13b3803eff0c2e0a916fcc51: Bug 1305325 - Part 8: Pass base value for opacity or transform to the compositor. r?birtles,mstange draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439333
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 8: Pass base value for opacity or transform to the compositor. r?birtles,mstange MozReview-Commit-ID: 3mxatMvsNKk
a386accadd54c1da6986c2d3384a39df895a8e9f: Bug 1305325 - Part 7: Make SampleValue returning StyleAnimationValue. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439332
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 7: Make SampleValue returning StyleAnimationValue. r?birtles MozReview-Commit-ID: Izr6Cvee96Q
87e55b04c764b970b02d948aa18aada205c2c304: Bug 1305325 - Part 6: Send animations even if it's paused, finished or zero playback rate. r?birtles. draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439331
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 6: Send animations even if it's paused, finished or zero playback rate. r?birtles. If all of animations on an element are paused, finished or zero playback rate, we don't send those animations to the compositor. MozReview-Commit-ID: CHjv6Buy5fa
4161a8bfe817cdcccb4a47fdfd4945d30cf81e0d: Bug 1305325 - Part 5: Make AnimationPropertyValueDetails::mValue optional. r?birtles draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439330
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 5: Make AnimationPropertyValueDetails::mValue optional. r?birtles MozReview-Commit-ID: 8tUv9hv1Dja
079f679904c8157300c2bb601c431517e7a39ffd: Bug 1305325 - Part 4: Update base values for each property. r? draft
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 16 Nov 2016 06:22:46 +0900 - rev 439329
Push 35974 by hiikezoe@mozilla-japan.org at Tue, 15 Nov 2016 21:23:27 +0000
Bug 1305325 - Part 4: Update base values for each property. r? This patch adds a hashtable to store the non-animated base value of each property animated by the effects in EffectSet. The values are stored as StyleAnimationValue objects since they are intended to be used in KeyframeEffectReadOnly::ComposeStyle. 1) We have to update it even if there is no animation properties changes because changing non-animating style value does not cause any animation property changes. If we do it in Updateproperties() it's apparently less effective. 2) There is the case that we have no EffectSet in UpdateProperties(), e.g. creating an animation. What happens on Element.animate(); i) KeyframeEffect constructor i-1) UpdateProperties() ii) Animation constructor with the effect ii-1) KeyframeEffectReadOnly::SetAnimation() ii-2) The effect is added in EffectSet MozReview-Commit-ID: FPzyX9fxxx3
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip