c53adee3f1139cbc6537d36b25f7b3c02c06e4ac: Bug 1365068 - Cleanup getListSuggestions. r=MattN
Jonathan Guillotte-Blouin <jguillotteblouin@mozilla.com> - Mon, 15 May 2017 14:15:36 -0700 - rev 359770
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1365068 - Cleanup getListSuggestions. r=MattN MozReview-Commit-ID: 2FVelDjB4pV
52caa4cf161857c494e91a5287e3117705ace625: Bug 1364871: Update expectations. r=me
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 20 May 2017 19:38:56 +0200 - rev 359769
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1364871: Update expectations. r=me MozReview-Commit-ID: 7vq5rb0Li2P
d4c254e98c7f514afc5e75811eb07f846aa6ac9c: merge mozilla-central to autoland. r=merge a=merge
Sebastian Hengst <archaeopteryx@coole-files.de> - Sat, 20 May 2017 18:53:36 +0200 - rev 359768
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
merge mozilla-central to autoland. r=merge a=merge
2e2958841a086642abfd8aa574866cf6c09ee7b9: servo: Merge #16961 - style: Move some bloom filter code outside of matching.rs (from emilio:bloom-simplify); r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 20 May 2017 09:15:27 -0500 - rev 359767
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
servo: Merge #16961 - style: Move some bloom filter code outside of matching.rs (from emilio:bloom-simplify); r=heycam Also simplify it, while we're at it. Source-Repo: https://github.com/servo/servo Source-Revision: 3ca7f4fc9296c2dd53e644ac25ddaa6411147d24
3ca6e6b80f4754b8f9182248a7a2188654561915: Bug 1334036 - Part 12: Enable off-main thread animations. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Mon, 08 May 2017 11:22:13 +0800 - rev 359766
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 12: Enable off-main thread animations. r=birtles Remove the unconditional return from FindAnimationsForCompositor(), so we can check if there is any property running on compositor. Also, enable a reftest and fix the mochitest expectation: 1. The expectation numbers of test_animations_omta.html in e10s and non-e10s are different, so skip non-e10s. 2. We pass all tests in test_animations_omta_start.html with e10s; however, got 3 test failures with non-e10s, so skip this file with non-e10s. MozReview-Commit-ID: IuOyAsUYguU
096add92a79f793e2a9ed6388be1576d1ae62d57: Bug 1334036 - Part 11: Trigger animation-only restyle when we handle an event with coordinates. r=birtles,heycam
Boris Chiou <boris.chiou@gmail.com> - Fri, 19 May 2017 16:16:41 +0800 - rev 359765
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 11: Trigger animation-only restyle when we handle an event with coordinates. r=birtles,heycam We need to request an animation-only restyle to force flush all throttled animations on main thread when we handle an event with coordinates (e.g. mouse event). MozReview-Commit-ID: KkjeQVsLgTl
db95d6d01b0bf2e752b22c858041e4f0908ea61c: Bug 1334036 - Part 10: Return AnimationValue for BaseStyle. r=hiro
Boris Chiou <boris.chiou@gmail.com> - Wed, 10 May 2017 11:06:19 +0800 - rev 359764
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 10: Return AnimationValue for BaseStyle. r=hiro We need to retrieve the correct base style for Servo backend, so change the return value to AnimationValue and update KeyframeEffectReadOnly::BaseStyle(). MozReview-Commit-ID: 9FL3h1DLoJt
3fbe048239146d2658f51147ae6b9414e83bebf0: Bug 1334036 - Part 9: Add one FFI which return None transform. r=emilio
Boris Chiou <boris.chiou@gmail.com> - Fri, 19 May 2017 16:09:45 +0800 - rev 359763
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 9: Add one FFI which return None transform. r=emilio This FFI is used by Servo_AnimationValue_GetTransform(), which needs to handle and return none transform properly. MozReview-Commit-ID: 49cFXE2BIbm
3d4a563e33bed51faaf752e19d2d3df761edc8bd: Bug 1334036 - Part 8: Add AddLayerChangesForAnimation in ServoRestyleManager. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Tue, 02 May 2017 14:03:16 +0800 - rev 359762
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 8: Add AddLayerChangesForAnimation in ServoRestyleManager. r=birtles Some changes to animations don't affect the computed style and yet still require the layer to be updated. Therefore, we also need to call AddLayerChangesForAnimation in ServoRestyleManager. In this patch, we factor out this function from GeckoRestyleManager, so we can reuse it. MozReview-Commit-ID: LL7D1oGS65l
663f9e058cd385ba83d176dbe679184dcf10a81d: Bug 1334036 - Part 7: Merge two similiar MaybeUpdateCascadeResults functions. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Thu, 11 May 2017 13:26:07 +0800 - rev 359761
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 7: Merge two similiar MaybeUpdateCascadeResults functions. r=birtles MozReview-Commit-ID: 2NDdQRF3DQ8
933bece8ce165549c5bc0c1a5964cffd94404c5b: Bug 1334036 - Part 6: Trigger restyle if important rules are changed. r=birtles,emilio
Boris Chiou <boris.chiou@gmail.com> - Fri, 19 May 2017 16:04:40 +0800 - rev 359760
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 6: Trigger restyle if important rules are changed. r=birtles,emilio If we add/remove important rules, we should call MaybeUpdateCascadeResults() to make sure EffectSet::mPropertiesWithImportantRules is correct, and so we can avoid that these important rules are overridden by animations running on compositor. Currently, we call MaybeUpdateCascadeResults only while iterating elements which needs to be restyled, so we should request a restyle on this element whose important rules are changed. MozReview-Commit-ID: 87MBQrirVto
750c87d6188415b5ba3d10b1ed95bc6cfb7266d0: Bug 1334036 - Part 5: Implement FFI for finding properties overriding animations. r=birtles,emilio
Boris Chiou <boris.chiou@gmail.com> - Fri, 19 May 2017 15:58:47 +0800 - rev 359759
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 5: Implement FFI for finding properties overriding animations. r=birtles,emilio We need to traverse rule tree to get the important rules, so we will not override them if they have animations running on compositor. MozReview-Commit-ID: 67NO2nIcUfq
e417ced91e375ffe66fb598942b0fa6b5a2d2a1b: Bug 1334036 - Part 4: Remove unused UpdateCascadeResults function. r=birtles,hiro
Boris Chiou <boris.chiou@gmail.com> - Wed, 10 May 2017 13:46:36 +0800 - rev 359758
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 4: Remove unused UpdateCascadeResults function. r=birtles,hiro We don't use the public UpdateCascadeResults method, so remove it. MozReview-Commit-ID: A2lWZaHWHTZ
babbbe4db7072be2672271e415987ed6d7c0cfd2: Bug 1334036 - Part 3: Add a flag to represent we are in pre-traversal. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Thu, 11 May 2017 13:08:35 +0800 - rev 359757
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 3: Add a flag to represent we are in pre-traversal. r=birtles We need this flag to avoid assertion in PostRestyleForAnimation(), which may be called from MaybeUpdateCascadeResults() in pre-traversal. MozReview-Commit-ID: 46AfoIUb9o3
ce3fdd4c0d51c6dfb30600eaca6194e80996f28e: Bug 1334036 - Part 2: Restyle all elements with animations if there are non-animation restyles. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Wed, 17 May 2017 11:41:43 +0800 - rev 359756
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 2: Restyle all elements with animations if there are non-animation restyles. r=birtles We restyle elements with non-animation restyles even if the animations are throttled. MozReview-Commit-ID: Exhd4qVx7su
bb7ba1de3ff3f911d9e03ac95842934fb34a5f4d: Bug 1334036 - Part 1: Avoid mutating mElementsToRestyle during pre-traversal. r=birtles
Boris Chiou <boris.chiou@gmail.com> - Fri, 28 Apr 2017 14:00:13 +0800 - rev 359755
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1334036 - Part 1: Avoid mutating mElementsToRestyle during pre-traversal. r=birtles During pre-traversal of EffectCompositor, we call MaybeUpdateCascadeResult(), which may add new element into mElementsToRestyle, as a result, we may iterate a mutated mElementsToRestyle. In this patch, we copy the element which needs update cascade results into another set and traverse this new set to call MaybeUpdateCascadeResult(). After that, do normal pre-traversal on mElementsToRestyle. MozReview-Commit-ID: 3uo6Ec5JNjp
187e507bbb17ff9892fb908d9dce93242434e1eb: servo: Merge #16963 - stylo: Bug 1334036 - Enable animations running on compositor (from BorisChiou:stylo/animation/omta); r=emilio
Boris Chiou <boris.chiou@gmail.com> - Sat, 20 May 2017 07:24:21 -0500 - rev 359754
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
servo: Merge #16963 - stylo: Bug 1334036 - Enable animations running on compositor (from BorisChiou:stylo/animation/omta); r=emilio These are interdependent patches of Bug 1334036, which enables off-main thread animations. We add one FFI to get the property id set which overriding animations, so we can make sure the cascade result is correct for off-main thread animations. --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix [Bug 1334036](https://bugzilla.mozilla.org/show_bug.cgi?id=1334036) - [X] These changes do not require tests because we support off-main thread animation only on Gecko, and there are enough test cases there. Source-Repo: https://github.com/servo/servo Source-Revision: 5a012cc9b15890fe8ad132e941d8f896b405472c
d6af6beaf98d09dfaf784851d6f8a57e4072b369: servo: Merge #16942 - Make font-stretch animatable (from mantaroh:font-stretch); r=hiro
Mantaroh Yoshinaga <mantaroh@gmail.com> - Sat, 20 May 2017 01:52:52 -0500 - rev 359753
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
servo: Merge #16942 - Make font-stretch animatable (from mantaroh:font-stretch); r=hiro <!-- Please describe your changes on the following line: --> This is a PR for https://bugzilla.mozilla.org/show_bug.cgi?id=1357663 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors <!-- Either: --> - [X] There are tests for these changes, a test case will be landed in web-platform-tests in https://bugzilla.mozilla.org/show_bug.cgi?id=1357663 <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> Source-Repo: https://github.com/servo/servo Source-Revision: 2c3a03b254c696d7be7a3d85ae5957628db63206
9885206e1a4822b609acd886157958721409a391: Bug 1364871: Revert workaround added in bug 1328319. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 20 May 2017 03:26:24 +0200 - rev 359752
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1364871: Revert workaround added in bug 1328319. r=heycam MozReview-Commit-ID: 804TV5qW3Xh
ca5ce34ad1cf84d8744ab12ef35948d0f4baaac2: Bug 1364871: Restyle ::backdrop too. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 19 May 2017 23:59:23 +0200 - rev 359751
Push 31857 by philringnalda@gmail.com at Sun, 21 May 2017 20:00:29 +0000
Bug 1364871: Restyle ::backdrop too. r=heycam MozReview-Commit-ID: BGWRiaeZiD8
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip