ff1f0e87bbad443a2cc8c1f211e74f2c2e4f26e5: Bug 1015803 - Align Layers' Animation.numIterations with AnimationTiming.mIterationCount; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:50 +0900 - rev 197728
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1015803 - Align Layers' Animation.numIterations with AnimationTiming.mIterationCount; r=dholbert In the IPC Animation struct used in layers code we have a member called 'numIterations' where 'iterate forever' is represented by -1. In layout/style however we have an AnimationTiming struct with an mIterationCount member where 'iterate forever' is represented by NS_IEEEPositiveInfinity(). This patch renames 'numIterations' to 'iterationCount' and uses infinity to represent 'iterate forever'.
0fb7aeff658c85afa78d3728d924b99eaf0c2405: Bug 1004871 part 11 - Move ElementAnimation members closer together for readability and packing; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:50 +0900 - rev 197727
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 11 - Move ElementAnimation members closer together for readability and packing; r=dholbert Also shuffle the initialization of members in nsAnimationManager::BuildAnimations to roughly match the order in which they are declared (with the exception that mPlayState needs to be set before calling IsPaused() which is used to set mPauseStart).
c1e663607e15d345f8769e18e98a034c1a802ecf: Bug 1004871 part 10 - Stop passing a fill mode of "both" to GetComputedTimingAt for throttled samples; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:50 +0900 - rev 197726
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 10 - Stop passing a fill mode of "both" to GetComputedTimingAt for throttled samples; r=dholbert This was only needed when we were inspecting the returned time fraction but now that we inspect the phase it's not necessary to force the fill mode to "both".
fbea945e7b7ba7588d8a082587fac43a5d0b18ba: Bug 1004871 part 9 - Move ElementAnimations::GetPositionInIteration to ElementAnimation::GetComputedTimingAt; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:50 +0900 - rev 197725
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 9 - Move ElementAnimations::GetPositionInIteration to ElementAnimation::GetComputedTimingAt; r=dholbert This patch simply moves the code from ElementAnimations to ElementAnimation so that it can later be used in transitions code and so we can later move EnsureStyleRuleFor to ElementAnimation.
1e629c073b337aba645be374562dd887cc0631e8: Bug 1004871 part 8 - Simplify ElementAnimations::GetEventsAt; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197724
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 8 - Simplify ElementAnimations::GetEventsAt; r=dholbert This patch shuffles the code in ElementAnimations::GetEventsAt to make it easier to follow. It also removes a check for whether or not the animation is paused. Previously we would not dispatch events if the animation was paused and in its active phase (but we would if the animation had finished). There doesn't seem to be any reason for this. If the animation was paused between the last sample and the current sample and the boundary of an iteration also occurred in that time then I expect we should dispatch that event. Removing this check for the pause state does not cause any tests fail. Separating out the event logic here makes it clear that we do not dispatch start events in the situation where one sample falls before the active interval and one sample falls after it (filed as bug 1004361). This patch adds a comment to this effect.
5fbcd746fed886204db5aa30b47aaed7e25f3a80: Bug 1004871 part 7 - Remove event queueing from GetPositionInIteration and do it in GetEventsAt; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197723
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 7 - Remove event queueing from GetPositionInIteration and do it in GetEventsAt; r=dholbert This patch simply shifts the event-related code from GetPositionInIteration to GetEventsAt. Although there are simplifications that could be done to GetEventsAt, they are deferred to a subsequent patch so as not to obscure the translation of code from one function to another. As a result of moving event-related handling from GetPositionInIteration it no longer needs to support different main-thread vs compositor modes.
81079a94b199876dd7af18ae7077834713beef82: Bug 1004871 part 6 - Make GetPositionInIteration return a ComputedTiming object; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197722
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 6 - Make GetPositionInIteration return a ComputedTiming object; r=dholbert This patch makes ElementAnimations::GetPositionInIteration return a ComputedTiming object instead of just a time portion (time fraction). Since the ComputedTiming object includes phase information, we can fix those parts of EnsureStyleRule and GetEventsAt that were temporarily using the time portion to guess if the animation might have finished or not.
69aef42e06d48bb4bd9decc7218e758ba0069e69: Bug 1004871 part 5 - Add ComputedTiming data structure; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197721
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 5 - Add ComputedTiming data structure; r=dholbert This patch adds a ComputedTiming struct for storing the results of calculating the timing properties of an animation at a given sample time.
431a388b4d919bb5e20987d04e9aa4dc3842ada1: Bug 1004871 part 4 - Move FillsForwards/FillsBackwards to AnimationTiming; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197720
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 4 - Move FillsForwards/FillsBackwards to AnimationTiming; r=dholbert This patch moves the FillsForwards/FillsBackwards methods previously defined on ElementAnimations to the structure contain the fill mode: AnimationTiming. It also changes GetPositionInIteration to use these methods.
d85a37f8c8b4be9ca0715bdf075718f09ef04a5f: Bug 1004871 part 3 - Replace parameters to GetPositionInIteration with an AnimationTiming object; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197719
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 3 - Replace parameters to GetPositionInIteration with an AnimationTiming object; r=dholbert This patch makes use of the AnimationTiming struct introduced in the previous patch to simplify calls to ElementAnimations::GetPositionInIteration.
1ee21cbff6175618f3578e87e424c70eb51a577b: Bug 1004871 part 2 - Add AnimationTiming struct, to encapsulate animation timing parameters; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:49 +0900 - rev 197718
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 2 - Add AnimationTiming struct, to encapsulate animation timing parameters; r=dholbert Introduces a struct to store timing parameters for passing to GetPositionInIteration. In future this struct is expected to be expanded to include other timing parameters as well (based roughly on Web Animations' "Timing" interface, hence the name AnimationTiming).
698920ab7165ced0ac8381d5af1183e1cd805dbf: Bug 1004871 part 1 - Factor event queuing out of EnsureStyleRuleFor; r=dholbert
Brian Birtles <birtles@gmail.com> - Wed, 28 May 2014 16:51:48 +0900 - rev 197717
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1004871 part 1 - Factor event queuing out of EnsureStyleRuleFor; r=dholbert This patch moves event queuing out of EnsureStyleRuleFor into a separate method. This is a preparatory step towards making GetPositionInIteration into a more generic method for calculating the current time fraction. In order to achieve this, GetPositionInIteration needs to be able to calculate the correct time portion for times outside the range [0, 1] even when it is not passed a ElementAnimation object. Specifically, it needs the fill mode of the animation to be passed in. (Rather than using FillForwards/FillBackwards this patch just compares the NS_STYLE_ANIMATION_FILL_MODE_* values directly but FillForwards/FillBackwards are restored in a subsequent patch when they are added to the struct used to lump the timing parameters together.) There are a number of places where positionInIteration is used to determine if the current sample occurs in the active phase or after. This is sub-optimal but is fixed in a subsequent patch in this series. The actual work of removing event queuing from GetPositionInIteration is deferred to a subsequent patch in order to keep the changes as small as possible. This patch simply makes separate calls to GetPositionInIteration for interpolating and for event queuing.
d943461254536df14e6bc1a6160de92347221c33: Bug 743703: allow mirroring of trace logs to NSPR; fix backwards lazy allocation defines r=pkerr
Randell Jesup <rjesup@jesup.org> - Wed, 28 May 2014 03:18:33 -0400 - rev 197716
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 743703: allow mirroring of trace logs to NSPR; fix backwards lazy allocation defines r=pkerr
f35d007a22a76b6a7693eafd381f967ed638d86c: Bug 990812 - Restrict sessionstore, tabview, and browser frame scripts to tabbrowser browsers r=adw
Tim Taubert <ttaubert@mozilla.com> - Fri, 23 May 2014 14:22:36 +0200 - rev 197715
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 990812 - Restrict sessionstore, tabview, and browser frame scripts to tabbrowser browsers r=adw
c17e7e3cd7294871cc33a107a3973f26ec7c02b5: Bug 990812 - Fix nsIMessageManager documentation r=smaug
Tim Taubert <ttaubert@mozilla.com> - Tue, 27 May 2014 21:46:06 +0200 - rev 197714
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 990812 - Fix nsIMessageManager documentation r=smaug
86de442c14a4adf25f8fc730d76d9ea41793d3eb: Bug 1013478 - Use GCC 4.8 on Android if available. r=glandium
Jim Chen <nchen@mozilla.com> - Thu, 22 May 2014 10:44:00 +0200 - rev 197713
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1013478 - Use GCC 4.8 on Android if available. r=glandium
8dff41f0ca9c545e98085ed909ff13d5c627979d: Bug 1016073 - Update stopApplication to work on Android 2.3. r=bc
William Lachance <wlachance@mozilla.com> - Tue, 27 May 2014 17:18:35 -0400 - rev 197712
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1016073 - Update stopApplication to work on Android 2.3. r=bc
99d60f78b54366a62d149d0ab62184d11a1a6e9b: Bug 940924 - Marionette should list the number of unexpected/expected fails at the end of the test run. r=jgriffin
Juien Pagès <j.parkouss@gmail.com> - Tue, 27 May 2014 10:01:00 +0200 - rev 197711
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 940924 - Marionette should list the number of unexpected/expected fails at the end of the test run. r=jgriffin
98644e8a0e128dde6f80f71bc7fac91d240d6a09: Bug 1013341 - Fix 'make package' for GCC ASan builds. r=glandium
Georg Koppen <gk@torproject.org> - Mon, 26 May 2014 11:29:56 +0200 - rev 197710
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1013341 - Fix 'make package' for GCC ASan builds. r=glandium
fd6d8a361cee14240eebb7bc56149526cd879892: Bug 1015550 - Line boundary is incorrect for multi-char bullet. r=surkov
Xidorn Quan <quanxunzhen@gmail.com> - Tue, 27 May 2014 18:07:00 +0200 - rev 197709
Push 5990 by asasaki@mozilla.com at Mon, 09 Jun 2014 21:40:24 +0000
Bug 1015550 - Line boundary is incorrect for multi-char bullet. r=surkov
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip