Commit Graph

1120 Commits

Author SHA1 Message Date
Michael Layzell
befc7b0924 Bug 1331434 - Part 9: Return after ErrorResult::Throw in /dom, r=ehsan
MozReview-Commit-ID: CWjx4L8LTr9
2017-03-07 19:06:48 -05:00
Hiroyuki Ikezoe
84aa371271 Bug 1344598 - Use nsIDocument::IsStyledByServo to check we are using servo or not. r=boris
MozReview-Commit-ID: 1yQUAwHy51

--HG--
extra : rebase_source : 2cacc29fe060fae75ccf7df638b1b060ec774388
2017-03-06 11:42:57 +09:00
Boris Chiou
bb298790fe Bug 1342957 - Fix incorrect MOZ_ASSERT in AnimationEffectReadOnly. r=hiro
MozReview-Commit-ID: 6amRZ9AGudQ

--HG--
extra : rebase_source : 350c0d64f371aea0568fc25e345e5de733142abc
2017-02-28 01:38:52 +08:00
Hiroyuki Ikezoe
f268b13da5 Bug 1341987 - Part 4: Use nsIFrame's nsPresContext instead of getting from element. r=birtles
MozReview-Commit-ID: 3pWt7d2MKsZ

--HG--
extra : rebase_source : cb34e1aa141d3a6aaa13f3482c8abf1cf83c14f9
2017-02-27 11:34:48 +09:00
Hiroyuki Ikezoe
feb893ec07 Bug 1341987 - Part 3: Update the last refresh time for transform only when we send transform animations to the compositor. r=birtles
We don't need to update the time during composing style, we just need to
update when transform animations are sent to the compositor. The most
recent refresh time of nsRefreshDriver should be the same in either side.
If the time is skewed, that means we skip composing style, if it happened
that's another bug.

MozReview-Commit-ID: Dxtuocf1ml1

--HG--
extra : rebase_source : 93e8fad24948b1c0b89fa7946639154a82f4a3ae
2017-02-27 11:34:46 +09:00
Hiroyuki Ikezoe
821747a2d1 Bug 1341987 - Part 2: Drop cascade level from last refresh time for transform. r=birtles
We no longer need separate time for cascade levels respectively since we do
send all type of animations on the same element to the compositor regardless
of cascade level.

MozReview-Commit-ID: 7cGQGE3yHvm

--HG--
extra : rebase_source : 009e0cc510faea2b83760c141da545e87de8aeaf
2017-02-27 11:30:56 +09:00
Hiroyuki Ikezoe
0911fca845 Bug 1341987 - Part 1: Rename AnimationRuleRefreshTime to LastTransformSyncTime. r=birtles
It's just for transform animations on the compositor.

MozReview-Commit-ID: 288CcG4kko4

--HG--
extra : rebase_source : 943f67beeecd3c05b7ae2c6608b7a725c47f2b23
2017-02-27 11:29:27 +09:00
Boris Zbarsky
5f864e31c5 Reenable several crashtests that no longer crash or leak or time out with stylo. No bug, r=bzbarsky
MozReview-Commit-ID: 88XjZyn4mnv
2017-02-24 15:22:37 -05:00
Hiroyuki Ikezoe
f78e12aaf6 Bug 1338927 - Part 8: Drop Servo_AnimationValues_Populate. r=birtles
MozReview-Commit-ID: 83WCbKL35h

--HG--
extra : rebase_source : e56a070443a0502b1a69a7ec8bcab2dccbf09557
2017-02-23 09:52:44 +09:00
Hiroyuki Ikezoe
bcc1bbe00b Bug 1338927 - Part 7: In case of stylo, use servo computed values instead of nsStyleContext to update properties. r=birtles
We convert nsStyleContext* to servo's computed values in UpdateProperties()
since it is called from various places (e.g. SetTarget()) with nsStyleContext*.

MozReview-Commit-ID: KxPjm82Crl

--HG--
extra : rebase_source : 04d3385dad97dad905079e21b7305c70afb23b8a
2017-02-23 09:52:44 +09:00
Hiroyuki Ikezoe
df8772b2fa Bug 1338927 - Part 6: Templatize SetKeyframes(). r=birtles
We have SetKeyframes(nsStyleContext*) for the same reason
of UpdateProperties(nsStyleContext*).

MozReview-Commit-ID: 7WjMU66MzpU

--HG--
extra : rebase_source : b5d60e9153e94cba6bb28184eb961c67ef85398b
2017-02-23 09:52:44 +09:00
Hiroyuki Ikezoe
deb07d6f55 Bug 1338927 - Part 5: Templatize UpdateProperties(). r=birtles
We had to implement UpdateProperties(nsStyleContext*) explicitly
since there are some calls of UpdateProperties() with RefPtr<nsStyleContext>.
Without this compiler tries to use template function instead.

MozReview-Commit-ID: 72NgwmJ4kcx

--HG--
extra : rebase_source : 30ac40c8ebc6d5e6bdf4326e6436daa95645ffc7
2017-02-23 09:52:44 +09:00
Hiroyuki Ikezoe
c7df70030a Bug 1338927 - Part 4: Templatize BuildProperties(). r=birtles
We'd like to use template to manage functions that have almost the same functionality
both on stylo and gecko instead of branching in each functions because it will be
easier to maintain stylo and gecko for a while.

MozReview-Commit-ID: 25ukMpOeqLj

--HG--
extra : rebase_source : 145b66695f9772f5cb527bbcf15b5a3fc2e34510
2017-02-23 09:52:44 +09:00
Hiroyuki Ikezoe
b5a2377fb5 Bug 1338927 - Part 3: Introduce ServoComputedStyleValues. r=birtles
The operator bool() will be used in the case where we call
MOZ_ASSERT() for const ref of this struct.

MozReview-Commit-ID: tCtpXmPJvV

--HG--
extra : rebase_source : f878c49ab93d8173f9de99b500efc280db886ea3
2017-02-23 09:52:43 +09:00
Hiroyuki Ikezoe
64f3fa3651 Bug 1338927 - Part 2: Generate ComputedKeyframeValues array from servo's computed values. r=birtles,manishearth
Servo_GetComputedKeyframeValues is almost the same as what we do in
KeyframeUtils::GetComputedKeyframeValues() for Gecko. Unlike the function
for Gecko, this function does not allocate each ComputedKeyframeValues element,
the allocation has been done before calling Servo_GetComputedKeyframeValues.

MozReview-Commit-ID: LRbriWoal2l

--HG--
extra : rebase_source : 785bb1bc6a62a2f9764fff09f5047942855e3c25
2017-02-23 09:52:43 +09:00
Hiroyuki Ikezoe
5de465377c Bug 1338927 - Part 1: Drop nsStyleContext argument from KeyframeUtils::GetAnimationPropertiesFromKeyframes. r=birtles
It's not used at all.

MozReview-Commit-ID: 2vAmlDhp12G

--HG--
extra : rebase_source : e1f9b226ff4d3fdd6f6bb6b03d6b157c5d153598
2017-02-23 09:52:43 +09:00
Shing Lyu
18c071c4c5 Bug 1337229 - Implement the deep operator== for RawServoAnimationValue r=hiro,manishearth
MozReview-Commit-ID: BiNNrIFXqtf

--HG--
extra : rebase_source : 78586bdf49b9d96db81af3e04b4a7522c1efce02
2017-02-24 15:05:23 +08:00
Jamie Nicol
f06e779e2d Bug 1339578 - Remove min active layer size for animations; r=mattwoodrow
Even for very small layers we want to avoid doing work on the main
thread.

At the same time, however, increase the minimum active layer size for
animations which come from restyles. These involve the main thread
anyway, so there is less to be gained from using an active
layer. Since switching items between active and inactive can have
large knock-on effects, we want to make sure it really is worth making
the layer active.

MozReview-Commit-ID: 8N6xlVW4Dp3

--HG--
extra : rebase_source : bd9a97899faaf187e5e148126711bb0ff5a29ee6
2017-02-09 18:00:32 +00:00
Iris Hsiao
7a962a70f2 Backed out changeset 2d21596af124 (bug 1337229) for developer's request 2017-02-22 17:40:13 +08:00
Shing Lyu
6ebbfba25f Bug 1337229 - Implement deep operator== for RawServoAnimationValue r=hiro,manishearth
MozReview-Commit-ID: Euvv9zarFe3

--HG--
extra : rebase_source : b67bb89ed35dca04d6dc81afdfbea701fdede449
2017-02-17 14:39:24 +08:00
Xidorn Quan
444ba0fc97 Bug 1340033 - Remove dependency from StyleAnimationValue.h to ServoBindings.h. r=hiro
MozReview-Commit-ID: 79czvLmftmv

--HG--
extra : rebase_source : 57d7a55ed48a0e9475076fa930226b6ea99a4e22
2017-02-16 15:14:06 +11:00
Boris Chiou
a6378fdab6 Bug 1338087 - Part 3: Remove asserts-if on crashtest 1244595-1.html. r=heycam
MozReview-Commit-ID: 7L2p7WvroSP

--HG--
extra : rebase_source : e6cd578d56921b34b5bd2bd38006443418cce69a
2017-02-13 13:55:29 +08:00
Boris Chiou
921bec7548 Bug 1338087 - Part 2: Drop the computation of StyleAnimationValue on stylo. r=hiro
MozReview-Commit-ID: 4oAzC6m2vie

--HG--
extra : rebase_source : 139674c8059a918b6850b74e4bdeb8a49b4a2571
2017-02-10 15:51:00 +08:00
Boris Chiou
62bf2f991d Bug 1338087 - Part 1: Don't apply paced spacing on stylo. r=hiro
To avoid using StyleAnimationValue on stylo, we should skip
ApplySpacing, which means We always fall back to distribute spacing on stylo.

MozReview-Commit-ID: 9DNwhOI8saf

--HG--
extra : rebase_source : c535139a8171bcdc874fef6bc1a12045957de9e1
2017-02-10 16:37:45 +08:00
Cameron McCormack
f63cdecde2 Bug 1297899 - Part 6: Move RestyleManagerHandle functionality into RestyleManager. r=bholley
MozReview-Commit-ID: 7lsti0bGzNr

--HG--
extra : rebase_source : 13c64026190afe5de25f540adb6deea9f518149f
2017-02-13 11:21:33 +08:00
Xidorn Quan
95c1b23bd9 Bug 1339341- Downgrade several stylo errors to warnings. r=heycam
MozReview-Commit-ID: 2Sa8dS8uSo5

--HG--
extra : rebase_source : c6809309bbef5faf458b56018b964619ff386238
2017-02-14 18:26:10 +11:00
Mantaroh Yoshinaga
b9fcaa1663 Bug 1302648 part 9 - Add test of setting target effect of CSS Animations. r=birtles
MozReview-Commit-ID: 6MVVFY4uJC4

--HG--
extra : rebase_source : 3b176090e54ef4cc51eaf4ea3f31815b03e9f1bc
2017-02-10 12:32:45 +09:00
Mantaroh Yoshinaga
e43bd142f6 Bug 1302648 part 8 - Add animationcancel tests. r=birtles
MozReview-Commit-ID: 44nT8BBNgzT

--HG--
extra : rebase_source : d52d56c08e0cee6c29a4926429fbad227ae19c66
2017-02-10 12:32:45 +09:00
Mantaroh Yoshinaga
073bbe8e59 Bug 1302648 part 7 - More simple animation event dispatch tests. r=birtles
MozReview-Commit-ID: 1fJD75QSkxf

--HG--
extra : rebase_source : ac459b6a664fb97332f2a061602169744e4b057c
2017-02-10 12:32:45 +09:00
Mantaroh Yoshinaga
fa7dee3362 Bug 1302648 part 4 - Call UpdateTiming() after removing the animation from the timeline. r=birtles
We will need to remove animation from timeline before calling Animation::UpdateTiming() in order to fire the cancel event.
In bug 1264125, we request one more tick after calling Animation::Cancel(), however we won't need to call this request if we apply this changeset.

MozReview-Commit-ID: h0dxUdtgkl

--HG--
extra : rebase_source : ac96209b28fdc781bf3787d348f2ae989e695640
2017-02-10 12:32:44 +09:00
Phil Ringnalda
df4305e633 Merge m-c to autoland 2017-02-12 15:31:37 -08:00
Hiroyuki Ikezoe
ca14d9287e Bug 1338769 - Don't move const reference into AnimValuesStyleRule. r=birtles
MozReview-Commit-ID: 8koosLn9VKn

--HG--
extra : rebase_source : 01147a281b6d823766c25470d02efe1aa89ce1f8
2017-02-12 08:30:22 +09:00
Hiroyuki Ikezoe
392ed0305b Bug 1331704 - Part 3: Drop all of unused stuff. r=birtles a=abillings
MozReview-Commit-ID: BWAC0iZLw6t
2017-02-11 19:11:45 +09:00
Hiroyuki Ikezoe
06cde8eda0 Bug 1331704 - Part 2: Resolve base styles during UpdateProprties. r=birtles a=abillings
EnsureBaseStyle() requires an already resolved nsStyleContext
and resolves the base style by ResolveStyleByRemovingAnimation().

MozReview-Commit-ID: BHqJiBJspQY
2017-02-11 19:11:45 +09:00
Hiroyuki Ikezoe
8ec82eeec6 Bug 1331704 - Part 1: Store base styles in KeyframeEffectReadOnly instead of EffectSet. r=birtles a=abillings
Now we dropped mNeedsBaseStyleSet bit set.

MozReview-Commit-ID: Gv2rU0jpyk1
2017-02-11 19:11:45 +09:00
Brian Birtles
492acb1b5c Bug 1338404 - Fix keyframe handling for single-valued lists r=hiro
MozReview-Commit-ID: 5MNSDm3Qr8b

--HG--
extra : rebase_source : 97f89e50d0e28fae206c29e6dee575493289e2eb
2017-02-10 15:21:15 +09:00
Carsten "Tomcat" Book
3ed98bee2a merge mozilla-inbound to mozilla-central a=merge 2017-02-10 13:49:56 +01:00
Boris Chiou
606e52ba0f Bug 1337313 - Part 2: Add AnimationValue::SerializeSpecifiedValue. r=hiro
Move the implementation of uncomputing to string into AnimationValue,
and add one more FFI for RawServoAnimationValue.

MozReview-Commit-ID: JFnXnGO3mlp

--HG--
extra : rebase_source : 83c0e084ef4202a5c98c668d68bfe9066196d735
2017-02-09 11:33:17 +08:00
Boris Chiou
ac7c271b7f Bug 1337313 - Part 1: Use AnimationValue in CreatePropertyValue. r=hiro
Replace the argument type with AnimationValue.

MozReview-Commit-ID: IeydVOLKTB9

--HG--
extra : rebase_source : 9537e40c9ffa16919a08765daa5d12e020e5d17c
2017-02-08 16:15:18 +08:00
Hiroyuki Ikezoe
a0e80666fe Bug 1333846 - Part 2: Evaluate scale values for additive or accumulative animations. r=birtles a=abillings
MozReview-Commit-ID: LSKJNH5OJbs
2017-02-09 11:28:47 +09:00
Hiroyuki Ikezoe
4047570520 Bug 1333846 - Part 1: Move ContainsAnimatedScale() codes into dom/animation/. r=birtles a=abillings
MozReview-Commit-ID: 6bWeTfCmjtd
2017-02-09 11:28:47 +09:00
Cameron McCormack
71d4bbc431 Bug 1334735 - Part 2: Add separate flag to track need to flush throttled animations. r=bz,birtles
MozReview-Commit-ID: 6Vi3laKcbmG
2017-02-10 10:42:28 +08:00
Cameron McCormack
b2ee81223c Bug 1334735 - Part 1: Move need style/flush flags from document to pres shell. r=bz
MozReview-Commit-ID: 2Amf9yGRiJA
2017-02-10 10:42:27 +08:00
Boris Chiou
155c2acd50 Bug 1335942 - Part 2: Use mozilla::AnimationValue in AnimationPropertySegment. r=hiro,manishearth
MozReview-Commit-ID: L6U1A223jsa

--HG--
extra : rebase_source : 26d8ec4386443d6526fed2f4926c6e4519e6bbd0
2017-02-04 13:57:08 +08:00
Boris Chiou
13d39b9757 Bug 1335942 - Part 1: Introduce mozilla::AnimationValue. r=hiro,manishearth
Currently, we have StyleAnimationValue for Gecko and
RawServoAnimationValue for Servo, and use this struct to wrap them to
avoid the duplicated code.

MozReview-Commit-ID: ATDBSCOX2C5

--HG--
extra : rebase_source : 85991ef5e23a445ae6302264508d1c06a46c0e29
2017-02-08 11:00:36 +08:00
Brian Birtles
5ba8812302 Bug 1332206 - Don't clamp steps timing functions outside [0, 1] range; r=hiro
MozReview-Commit-ID: AiSdxQvU2mj

--HG--
extra : rebase_source : 77fb266b62a3a93ac134c1d65d1107c0bb8a110d
2017-02-08 09:25:31 +09:00
Sebastian Hengst
0264cdf6be Bug 1336311 - Change code comments with http://hg.mozilla.org to https://. r=gps
For components also referencing it in code, see the blockers of bug 1336311.

MozReview-Commit-ID: 4tUZ24HKBWy

--HG--
extra : rebase_source : ec16149f525b9b7eaca7f96f1369929d21497121
2017-02-07 17:52:56 +01:00
Mantaroh Yoshinaga
1ba8149d5d Bug 1336899 - Should not fire the transitioncancel after ending CSS Transition. r=birtles
MozReview-Commit-ID: IrqNNjVg0xg

--HG--
extra : rebase_source : 769e4646375d41b5621ee71c8f0ce5eb44bae8d1
2017-02-06 16:21:01 +09:00
Hiroyuki Ikezoe
af7c9e29b7 Bug 1326306 - Skip test_restyles.html on Android x86. r=gbrown
MozReview-Commit-ID: AHp28FajVoT

--HG--
extra : rebase_source : 045de247f860c249f6d9f982fa507d6070460bf5
2017-02-02 12:55:59 +09:00
Boris Chiou
015e8b8198 Bug 1324554 - Part 3: More tests for test_animation_properties.html. r=hiro
MozReview-Commit-ID: 6MkkP8yNGvg

--HG--
extra : rebase_source : 091a1d72747b445a80c255bcc09e33fe23760446
2017-02-02 11:05:35 +08:00