Bug 1419079 - Revise a comment for a test case for missing keyframes. r?birtles draft
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Wed, 17 Jan 2018 08:55:00 +0900
changeset 721313 4b83a75750ce9f1472a6d187c0cd454c3e376dca
parent 721312 86d96131839d70b87fad95a74de822dbbf47e186
child 721314 4d2384cbcd11c4f998d1853c451cc9845825da71
push id95792
push userhikezoe@mozilla.com
push dateWed, 17 Jan 2018 01:53:19 +0000
reviewersbirtles
bugs1419079
milestone59.0a1
Bug 1419079 - Revise a comment for a test case for missing keyframes. r?birtles The animation in the test case is not actually additive animation, it's just a missing keyframe animation and its composite operation is actually 'replace'. MozReview-Commit-ID: 4A29V5Ke2hF
dom/animation/test/mozilla/file_restyles.html
--- a/dom/animation/test/mozilla/file_restyles.html
+++ b/dom/animation/test/mozilla/file_restyles.html
@@ -950,17 +950,17 @@ waitForAllPaints(() => {
          '!important rule begins running on the compositor even if the ' +
          '!important rule had been dropped before the target element was ' +
          'removed');
 
       await ensureElementRemoval(div);
     }
   );
 
-  // Tests that additive animations don't throttle at all.
+  // Tests that missing keyframes animations don't throttle at all.
   add_task(async function no_throttling_animations_out_of_view_element() {
     var div = addDiv(null, { style: 'transform: translateY(-400px);' });
     var animation =
       div.animate([{ visibility: 'visible' }], 100 * MS_PER_SEC);
 
     await animation.ready;
 
     const expectedRestyleCount = tweakExpectedRestyleCount(animation, 5);