Bug 1180125 part 0 - Fix an obvious bug in animation_utils.js; r=dbaron
authorBrian Birtles <birtles@gmail.com>
Wed, 29 Jul 2015 10:57:38 +0900
changeset 288373 94a07b627f21a086015e45941b33ea4f8280dead
parent 288372 a01bf3016c0824cc48f301c95507358d3bbdc5eb
child 288374 f30ecbdba384066bf0e5b8ac7ded27c88160304c
push id5067
push userraliiev@mozilla.com
push dateMon, 21 Sep 2015 14:04:52 +0000
treeherdermozilla-beta@14221ffe5b2f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs1180125, 1070745
milestone42.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1180125 part 0 - Fix an obvious bug in animation_utils.js; r=dbaron There was a bug introduced in to animation_utils.js in https://hg.mozilla.org/mozilla-central/rev/496e867cd2cd (bug 1070745). This patch fixes this bug so we can be sure testing events correctly before messing with them.
layout/style/test/animation_utils.js
--- a/layout/style/test/animation_utils.js
+++ b/layout/style/test/animation_utils.js
@@ -50,17 +50,17 @@ function advance_clock(milliseconds) {
     gElem.addEventListener("animationiteration", listener, false);
     gElem.addEventListener("animationend", listener, false);
   }
 
   function check_events(eventsExpected, desc) {
     // This function checks that the list of eventsExpected matches
     // the received events -- but it only checks the properties that
     // are present on eventsExpected.
-    is(gEventsReceived.length, gEventsReceived.length,
+    is(gEventsReceived.length, eventsExpected.length,
        "number of events received for " + desc);
     for (var i = 0,
          i_end = Math.min(eventsExpected.length, gEventsReceived.length);
          i != i_end; ++i) {
       var exp = eventsExpected[i];
       var rec = gEventsReceived[i];
       for (var prop in exp) {
         if (prop == "elapsedTime") {