d1f22265c37e24b00641259768cf24f0e1d499d2: b=1023697 use MediaStream to convert stream time to ticks in media engine sources r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:57 +1200 - rev 188298
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 use MediaStream to convert stream time to ticks in media engine sources r=roc
b46a382f57fb7e4905a8534a483fd302044c996b: b=1023697 use MediaStream to convert stream time to seconds in HTMLMediaElement r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:57 +1200 - rev 188297
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 use MediaStream to convert stream time to seconds in HTMLMediaElement r=roc
027bb6e81be268bb70eb3baf5255c9e40373d87b: b=1023697 use MediaStream to convert ticks to time in MediaDecoderStateMachine r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:56 +1200 - rev 188296
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 use MediaStream to convert ticks to time in MediaDecoderStateMachine r=roc
d9ac53c6e7465ccfd68f96d3a1dd8e46b6c0d263: b=1023697 change MediaDecoderStateMachine::mSyncPointInMediaStream from StreamTime to µs r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:56 +1200 - rev 188295
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 change MediaDecoderStateMachine::mSyncPointInMediaStream from StreamTime to µs r=roc This allows the conversion from StreamTime to be performed while a stream is available.
225c72840e52f83169d792a3c13690e3bb13fdaa: b=1023697 add methods to MediaStream for tick and time conversions r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:56 +1200 - rev 188294
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 add methods to MediaStream for tick and time conversions r=roc When MediaTime depends on the audio rate of the graph, the stream will know the graph rate and so can perform the conversion. Existing methods in StreamBuffer.h and MediaSegment.h will be removed once clients have been changed to the new methods.
8b11b0e372f30ebda26e67777916351acb04a3bb: b=1023697 remove unused TicksToTimeRound() r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:44:55 +1200 - rev 188293
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=1023697 remove unused TicksToTimeRound() r=roc
698fae4afa729fb5bbe5cdd30f342c125153eb2a: b=995690 don't advance audio output stream position 1 tick beyond what has been produced r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:41:29 +1200 - rev 188292
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=995690 don't advance audio output stream position 1 tick beyond what has been produced r=roc This may result in repeating a sample sometimes due to rounding effects, but that should only happen once per unblocked sequence of track frames.
2f5473458f0a53f990061b28d958da07b724f634: b=995690 correct mLastTickWritten to only advance when not blocked r=roc
Karl Tomlinson <karlt+@karlt.net> - Thu, 12 Jun 2014 16:40:51 +1200 - rev 188291
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
b=995690 correct mLastTickWritten to only advance when not blocked r=roc mLastTickWritten and offset are stream positions, which do not advance when the stream is blocked.
3b9ed7396f9ad3b7874d6f27558510ab984ef64f: Bug 1004377 - Dispatch events for CSS Animations with empty keyframes rules; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:18:14 +0900 - rev 188290
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1004377 - Dispatch events for CSS Animations with empty keyframes rules; r=dholbert This patch removes the check that skipped queueing events for animations without keyframes since the spec indicates such animations should dispatch events. There is a further correctness fix here for the case where a keyframes rule is modified using the CSSOM so that it becomes empty. Previously when we came to create the new animation rules we would end up setting ElementAnimations::mNeedRefreshes to false since we check if the keyframes rule is empty and if it is we would skip all further processing (including setting mNeedsRefreshes). That means that: (a) We may end up unregistering from the refresh observer so we would never dispatch the end event for such an animation. (b) If the animation was running on the compositor we may never remove it from the compositor or may not do it in a timely fashion. To fix both these problems, this patch removes the check for an empty keyframes rule so that mNeedsRefreshes is set in this case.
441a1c280cd86d05f71e3bd5f00f4b62e315ac8b: Bug 1018862 part 9 - Remove nsIDOMWindowUtils::GetOMTAOrComputedStyle; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:48 +0900 - rev 188289
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 9 - Remove nsIDOMWindowUtils::GetOMTAOrComputedStyle; r=dholbert Now that getOMTAOrComputedStyle is no longer used by any tests it should be removed. It is a footgun because it can lead the test author to think they are testing compositor animations (OMTA) when in fact they are just testing main thread animations. This has bitten us before because while we were getting results from the compositor on some platforms when run locally, on Tinderbox the only build configurations that had OMTA turned on were falling back to returning the computed style.
cea6a1920ce7925591662ef3910090da8b0053f7: Bug 1018862 part 8 - Add special handling for non-invertible transforms; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188288
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 8 - Add special handling for non-invertible transforms; r=dholbert This patch adjusts the tests for transform transitions that run on the compositor to handle transitions that begin with a non-invertible transform. In this case the first sample at the start of the animation won't create a layer because in nsDisplayTransform::BuildLayer / FrameLayerBuilder::BuildContainerLayerFor we'll skip creating the layer once we notice the equivalent matrix is singular. In this patch we detect that case and force an extra sample betwee 0 and 200s at 100s. This means the layer will be created at t=100s and be available for querying at the next sample. Similar issues could occur, for example, if the transforms at both t=0s and t=100s are not invertible but currently that doesn't occur. We can add handling for that if and when it becomes necessary.
1d7898e89dab2f4d90421a43c1a82fd12fae03b8: Bug 1018862 part 7 - Convert test_transitions_per_property.html to use OMTA test methods; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188287
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 7 - Convert test_transitions_per_property.html to use OMTA test methods; r=dholbert This patch takes the existing tests for transitions running on the compositor and makes them re-use the same test utility methods as used for testing CSS Animations that run on the compositor. This means these tests now also check that the transition is in fact running on the compositor when it is expected to. It seems the big_omta_round_error is no longer needed so I've removed that. The test that begins with "skew(45deg, 45deg)" currently fails so it is skipped here. This is addressed in the next patch in the series.
99f24b59c78260ab928ab5d074197e481aba225a: Bug 1018862 part 6 - Fix handling of 3d matrices in omta_is and co.; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188286
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 6 - Fix handling of 3d matrices in omta_is and co.; r=dholbert This patch fixes a bug in the handling of 3d matrices represented as an array of numbers.
0f6fd048a317f6e45e5ebcc4b23c5b0d9718e3b4: Bug 1018862 part 5 - Move paint listener promise wrappers to animation_utils.js; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188285
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 5 - Move paint listener promise wrappers to animation_utils.js; r=dholbert This patch moves some utility methods from test_animations_omta.html to animation_utils.js so that they can be used for testing transitions as well.
49775bc6825e8b0508f13a7d2f9202dc51c53557: Bug 1018862 part 4 - Move omta_is and friends to animation_utils.js; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188284
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 4 - Move omta_is and friends to animation_utils.js; r=dholbert This patch simply moves code from test_animations_omta.html to animation_utils.js so that we can use it for testing transitions as well.
1c0a44e761ec4ca144801c6ac377d14c4173ab2e: Bug 1018862 part 3 - Fix the order of arguments to omta_is_approx; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188283
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 3 - Fix the order of arguments to omta_is_approx; r=dholbert This patch simply re-arranges the order of arguments to omta_is_approx so that the delta sits along side the values being compared. This, I think, makes more sense and also is more consistent when converting tests from test_animations.html to test_animations_omta.html since the "RunningOn" parameter is consistently inserted in the second-to-last position just before the description for both omta_is and omta_is_approx.
4900384bce9f1d8cf37f03aeaca715776b07863d: Bug 1018862 part 2 - Make new_div no longer secretly flush styles; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:47 +0900 - rev 188282
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 2 - Make new_div no longer secretly flush styles; r=dholbert This patch removes the line from new_div that forced a style flush. This was very confusing because: * It behaved differently to new_div in test_animations.html so copying tests over was more complex (particularly when registering for events is involved). * It meant after setting up initial style using new_div you could just call waitForPaints but if you updated style using elem.style you'd need to call waitForPaintsFlushed. In adjusting test_animations_omta.html we are able to simplify the tests somewhat. This patch also adds a few additional checks that waiting to update the compositor does not produce different results.
1f3384e94822869c2f2c13d4beaa28b59f410f3b: Bug 1018862 part 1 - Factor out common async animation test methods; r=dholbert
Brian Birtles <birtles@gmail.com> - Thu, 12 Jun 2014 13:17:46 +0900 - rev 188281
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1018862 part 1 - Factor out common async animation test methods; r=dholbert This patch moves some test utility methods from test_animations_omta.html to animation_utils.js. It also renames addAsyncTest to addAsyncAnimTest and likewise for a few other methods.
7ee90f5b0aac6cc15f49fc56dc8ca12a4b0b2f6b: Bug 1006485 - Part b: hg rename LockedFile.h/cpp/webidl in a separate patch for better readability; r=bent
Jan Varga <jan.varga@gmail.com> - Thu, 12 Jun 2014 05:35:47 +0200 - rev 188280
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1006485 - Part b: hg rename LockedFile.h/cpp/webidl in a separate patch for better readability; r=bent
00d924bed2d1be6252ba8aa4c0c66a4d7baf2a71: Bug 1006485 - Part a: Rename FileHandle to MutableFile and LockedFile to FileHandle; r=bent
Jan Varga <jan.varga@gmail.com> - Thu, 12 Jun 2014 05:35:29 +0200 - rev 188279
Push 26950 by emorley@mozilla.com at Thu, 12 Jun 2014 13:50:58 +0000
Bug 1006485 - Part a: Rename FileHandle to MutableFile and LockedFile to FileHandle; r=bent
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip