Bug 1475441 - Add requestLongerTimeout(2) and remove unnecessary code of removing animations. r=daisuke
authorMantaroh Yoshinaga <mantaroh@gmail.com>
Wed, 15 Aug 2018 00:38:25 +0000
changeset 486677 e6584141d58175ba0df1339493bdd069f71cec07
parent 486676 3208575a228a9f2228c0efb0d2a453894f10860e
child 486678 b725384bc0cec68d4a671edc4cfc3ed67e11be75
push id9719
push userffxbld-merge
push dateFri, 24 Aug 2018 17:49:46 +0000
treeherdermozilla-beta@719ec98fba77 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdaisuke
bugs1475441
milestone63.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 1475441 - Add requestLongerTimeout(2) and remove unnecessary code of removing animations. r=daisuke This test removed the animations after opening the animation inspector. This removing animation code is unnecessary since the target document has not these animation class. As a result of it, that test will take the long times. However, this test will timeout even if removing that code. So this patch will add requestLongerTimeout(2) for intermittent failure of the ccov test. Differential Revision: https://phabricator.services.mozilla.com/D3292
devtools/client/inspector/animation/test/browser_animation_rewind-button.js
--- a/devtools/client/inspector/animation/test/browser_animation_rewind-button.js
+++ b/devtools/client/inspector/animation/test/browser_animation_rewind-button.js
@@ -1,23 +1,25 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
 
 "use strict";
 
+requestLongerTimeout(2);
+
 // Test for following RewindButton component:
 // * element existence
 // * make animations to rewind to zero
 // * the state should be always paused after rewinding
 
 add_task(async function() {
   await addTab(URL_ROOT + "doc_multi_timings.html");
+  await removeAnimatedElementsExcept([".delay-negative",
+                                      ".delay-positive"]);
   const { animationInspector, panel } = await openAnimationInspector();
-  await removeAnimatedElementsExcept([".animated",
-                                      ".negative-delay"]);
 
   info("Checking button existence");
   ok(panel.querySelector(".rewind-button"), "Rewind button should exist");
 
   info("Checking rewind button makes animations to rewind to zero");
   await clickOnRewindButton(animationInspector, panel);
   assertAnimationsCurrentTime(animationInspector, 0);
   assertAnimationsPausing(animationInspector, panel);