Bug 1415346 - Use const instead of var in file_event-dispatch.html. r?birtles draft
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Wed, 08 Nov 2017 12:15:00 +0900
changeset 694699 05aeb96b8fc197b379296be378007af3b266d53b
parent 694698 3cb0d0c7823d6a63b9e949b8604b1def12714937
child 694700 8002e3ec351fb292dcfe8de5a15fe309e4d64edc
push id88205
push userhikezoe@mozilla.com
push dateWed, 08 Nov 2017 03:47:13 +0000
reviewersbirtles
bugs1415346
milestone58.0a1
Bug 1415346 - Use const instead of var in file_event-dispatch.html. r?birtles MozReview-Commit-ID: Al9lYAMDVNZ
dom/animation/test/css-animations/file_event-dispatch.html
dom/animation/test/css-transitions/file_event-dispatch.html
--- a/dom/animation/test/css-animations/file_event-dispatch.html
+++ b/dom/animation/test/css-animations/file_event-dispatch.html
@@ -37,22 +37,22 @@ function AnimationEventHandler(target) {
 AnimationEventHandler.prototype.clear = () => {
   this.animationstart     = undefined;
   this.animationiteration = undefined;
   this.animationend       = undefined;
   this.animationcancel    = undefined;
 }
 
 function setupAnimation(t, animationStyle) {
-  var div = addDiv(t, { style: 'animation: ' + animationStyle });
-  var watcher = new EventWatcher(t, div, [ 'animationstart',
+  const div = addDiv(t, { style: 'animation: ' + animationStyle });
+  const watcher = new EventWatcher(t, div, [ 'animationstart',
                                            'animationiteration',
                                            'animationend',
                                            'animationcancel' ]);
-  var animation = div.getAnimations()[0];
+  const animation = div.getAnimations()[0];
 
   return [animation, watcher, div];
 }
 
 promise_test(t => {
   // Add 1ms delay to ensure that the delay is not included in the elapsedTime.
   const [animation, watcher] = setupAnimation(t, 'anim 100s 1ms');
 
--- a/dom/animation/test/css-transitions/file_event-dispatch.html
+++ b/dom/animation/test/css-transitions/file_event-dispatch.html
@@ -32,172 +32,172 @@ function TransitionEventHandler(target) 
 TransitionEventHandler.prototype.clear = () => {
   this.transitionrun    = undefined;
   this.transitionstart  = undefined;
   this.transitionend    = undefined;
   this.transitioncancel = undefined;
 };
 
 function setupTransition(t, transitionStyle) {
-  var div = addDiv(t, { style: 'transition: ' + transitionStyle });
-  var watcher = new EventWatcher(t, div, [ 'transitionrun',
+  const div = addDiv(t, { style: 'transition: ' + transitionStyle });
+  const watcher = new EventWatcher(t, div, [ 'transitionrun',
                                            'transitionstart',
                                            'transitionend',
                                            'transitioncancel' ]);
   flushComputedStyle(div);
 
   div.style.marginLeft = '100px';
-  var transition = div.getAnimations()[0];
+  const transition = div.getAnimations()[0];
 
   return [transition, watcher, div];
 }
 
 // On the next frame (i.e. when events are queued), whether or not the
 // transition is still pending depends on the implementation.
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
   return watcher.wait_for('transitionrun').then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Idle -> Pending or Before');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
   // Force the transition to leave the idle phase
   transition.startTime = document.timeline.currentTime;
   return watcher.wait_for('transitionrun').then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Idle -> Before');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   // Seek to Active phase.
   transition.currentTime = 100 * MS_PER_SEC;
   transition.pause();
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     assert_equals(handler.transitionrun, 0.0);
     assert_equals(handler.transitionstart, 0.0);
   });
 }, 'Idle or Pending -> Active');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   // Seek to After phase.
   transition.finish();
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart',
                             'transitionend' ]).then(evt => {
     assert_equals(handler.transitionrun, 0.0);
     assert_equals(handler.transitionstart, 0.0);
     assert_equals(handler.transitionend, 100.0);
   });
 }, 'Idle or Pending -> After');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return Promise.all([ watcher.wait_for('transitionrun'),
                        transition.ready ]).then(() => {
     // Make idle
     div.style.display = 'none';
     flushComputedStyle(div);
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Before -> Idle (display: none)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return Promise.all([ watcher.wait_for('transitionrun'),
                        transition.ready ]).then(() => {
     // Make idle
     transition.timeline = null;
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Before -> Idle (Animation.timeline = null)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return Promise.all([ watcher.wait_for('transitionrun'),
                        transition.ready ]).then(() => {
     transition.currentTime = 100 * MS_PER_SEC;
     return watcher.wait_for('transitionstart');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Before -> Active');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   return Promise.all([ watcher.wait_for('transitionrun'),
                        transition.ready ]).then(() => {
     // Seek to After phase.
     transition.currentTime = 200 * MS_PER_SEC;
     return watcher.wait_for([ 'transitionstart', 'transitionend' ]);
   }).then(evt => {
     assert_equals(handler.transitionstart, 0.0);
     assert_equals(handler.transitionend, 100.0);
   });
 }, 'Before -> After');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s');
 
   // Seek to Active start position.
   transition.pause();
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Make idle
     div.style.display = 'none';
     flushComputedStyle(div);
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Idle, no delay (display: none)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s');
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Make idle
     transition.currentTime = 0;
     transition.timeline = null;
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Idle, no delay (Animation.timeline = null)');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
   // Pause so the currentTime is fixed and we can accurately compare the event
   // time in transition cancel events.
   transition.pause();
 
   // Seek to Active phase.
   transition.currentTime = 100 * MS_PER_SEC;
   return watcher.wait_for([ 'transitionrun',
@@ -207,34 +207,34 @@ promise_test(t => {
     flushComputedStyle(div);
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Idle, with positive delay (display: none)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
 
   // Seek to Active phase.
   transition.currentTime = 100 * MS_PER_SEC;
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Make idle
     transition.currentTime = 100 * MS_PER_SEC;
     transition.timeline = null;
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Idle, with positive delay (Animation.timeline = null)');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s -50s');
 
   // Pause so the currentTime is fixed and we can accurately compare the event
   // time in transition cancel events.
   transition.pause();
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
@@ -243,115 +243,115 @@ promise_test(t => {
     flushComputedStyle(div);
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 50.0);
   });
 }, 'Active -> Idle, with negative delay (display: none)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s -50s');
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Make idle
     transition.currentTime = 50 * MS_PER_SEC;
     transition.timeline = null;
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Idle, with negative delay (Animation.timeline = null)');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
   // Seek to Active phase.
   transition.currentTime = 100 * MS_PER_SEC;
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Seek to Before phase.
     transition.currentTime = 0;
     return watcher.wait_for('transitionend');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 0.0);
   });
 }, 'Active -> Before');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
   // Seek to Active phase.
   transition.currentTime = 100 * MS_PER_SEC;
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Seek to After phase.
     transition.currentTime = 200 * MS_PER_SEC;
     return watcher.wait_for('transitionend');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 100.0);
   });
 }, 'Active -> After');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   // Seek to After phase.
   transition.finish();
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart',
                             'transitionend' ]).then(evt => {
     // Seek to Before phase.
     transition.currentTime = 0;
     return watcher.wait_for([ 'transitionstart', 'transitionend' ]);
   }).then(evt => {
     assert_equals(handler.transitionstart, 100.0);
     assert_equals(handler.transitionend, 0.0);
   });
 }, 'After -> Before');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s 100s');
   // Seek to After phase.
   transition.finish();
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart',
                             'transitionend' ]).then(evt => {
     // Seek to Active phase.
     transition.currentTime = 100 * MS_PER_SEC;
     return watcher.wait_for('transitionstart');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 100.0);
   });
 }, 'After -> Active');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s -50s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(() => {
     assert_equals(handler.transitionrun, 50.0);
     assert_equals(handler.transitionstart, 50.0);
     transition.finish();
     return watcher.wait_for('transitionend');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 100.0);
   });
 }, 'Calculating the interval start and end time with negative start delay.');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
-  var handler = new TransitionEventHandler(div);
+  const handler = new TransitionEventHandler(div);
 
   return watcher.wait_for('transitionrun').then(evt => {
     // We can't set the end delay via generated effect timing.
     // Because CSS-Transition use the AnimationEffectTimingReadOnly.
     transition.effect = new KeyframeEffect(div,
                                            { marginleft: [ '0px', '100px' ]},
                                            { duration: 100 * MS_PER_SEC,
                                              endDelay: -50 * MS_PER_SEC });
@@ -368,99 +368,99 @@ promise_test(t => {
     transition.finish();
     return watcher.wait_for('transitionend');
   }).then(evt => {
     assert_equals(evt.elapsedTime, 50.0);
   });
 }, 'Calculating the interval start and end time with negative end delay.');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return watcher.wait_for('transitionrun').then(() => {
     // Make idle
     div.style.display = 'none';
     flushComputedStyle(div);
     return watcher.wait_for('transitioncancel');
   }).then(() => {
     transition.cancel();
     // Then wait a couple of frames and check that no event was dispatched
     return waitForAnimationFrames(2);
   });
 }, 'Call Animation.cancel after cancelling transition.');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return watcher.wait_for('transitionrun').then(evt => {
     // Make idle
     div.style.display = 'none';
     flushComputedStyle(div);
     transition.play();
     watcher.wait_for([ 'transitioncancel',
                        'transitionrun',
                        'transitionstart' ]);
   });
 }, 'Restart transition after cancelling transition immediately');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s 100s');
 
   return watcher.wait_for('transitionrun').then(evt => {
     // Make idle
     div.style.display = 'none';
     flushComputedStyle(div);
     transition.play();
     transition.cancel();
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     // Then wait a couple of frames and check that no event was dispatched
     return waitForAnimationFrames(2);
   });
 }, 'Call Animation.cancel after restarting transition immediately');
 
 promise_test(t => {
-  var [transition, watcher] =
+  const [transition, watcher] =
     setupTransition(t, 'margin-left 100s');
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     // Make idle
     transition.timeline = null;
     return watcher.wait_for('transitioncancel');
   }).then(evt => {
     transition.timeline = document.timeline;
     transition.play();
 
     return watcher.wait_for(['transitionrun', 'transitionstart']);
   });
 }, 'Set timeline and play transition after clear the timeline');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s');
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(() => {
     transition.cancel();
     return watcher.wait_for('transitioncancel');
   }).then(() => {
     // Make After phase
     transition.effect = null;
 
     // Then wait a couple of frames and check that no event was dispatched
     return waitForAnimationFrames(2);
   });
 }, 'Set null target effect after cancel the transition');
 
 promise_test(t => {
-  var [transition, watcher, div] =
+  const [transition, watcher, div] =
     setupTransition(t, 'margin-left 100s');
 
   return watcher.wait_for([ 'transitionrun',
                             'transitionstart' ]).then(evt => {
     transition.effect = null;
     return watcher.wait_for('transitionend');
   }).then(evt => {
     transition.cancel();