Bug 1461899: Use mouseover of another element instead of mouseout. r?gl,r?jmaher draft
authorDaisuke Akatsuka <dakatsuka@mozilla.com>
Fri, 08 Jun 2018 10:01:29 +0900
changeset 805571 06220842773f2d094670d50368923ed1798d89aa
parent 805066 f7fd9b08c0156be5b5cd99de5ed0ed0b98d93051
push id112700
push userbmo:dakatsuka@mozilla.com
push dateFri, 08 Jun 2018 01:02:18 +0000
reviewersgl, jmaher
bugs1461899
milestone62.0a1
Bug 1461899: Use mouseover of another element instead of mouseout. r?gl,r?jmaher MozReview-Commit-ID: 15ykKWTJMOW
devtools/client/inspector/animation/test/browser.ini
devtools/client/inspector/animation/test/browser_animation_keyframes-graph_computed-value-path_easing-hint.js
--- a/devtools/client/inspector/animation/test/browser.ini
+++ b/devtools/client/inspector/animation/test/browser.ini
@@ -34,17 +34,16 @@ support-files =
 [browser_animation_current-time-label.js]
 [browser_animation_current-time-scrubber.js]
 [browser_animation_current-time-scrubber_each-different-creation-time-animations.js]
 [browser_animation_empty_on_invalid_nodes.js]
 [browser_animation_inspector_exists.js]
 [browser_animation_keyframes-graph_computed-value-path-01.js]
 [browser_animation_keyframes-graph_computed-value-path-02.js]
 [browser_animation_keyframes-graph_computed-value-path_easing-hint.js]
-skip-if = (os == "win") || (os == "linux") #1461899
 [browser_animation_keyframes-graph_keyframe-marker.js]
 [browser_animation_keyframes-progress-bar.js]
 [browser_animation_keyframes-progress-bar_after-resuming.js]
 [browser_animation_logic_auto-stop.js]
 [browser_animation_logic_avoid-updating-during-hiding.js]
 [browser_animation_logic_created-time.js]
 [browser_animation_logic_mutations.js]
 [browser_animation_logic_mutations_fast.js]
--- a/devtools/client/inspector/animation/test/browser_animation_keyframes-graph_computed-value-path_easing-hint.js
+++ b/devtools/client/inspector/animation/test/browser_animation_keyframes-graph_computed-value-path_easing-hint.js
@@ -239,25 +239,26 @@ add_task(async function() {
             `x of <rect> in ${ hintTarget } should be ${ expectedHint.rect.x }`);
           is(interactionEl.getAttribute("width"), expectedHint.rect.width,
             `width of <rect> in ${ hintTarget } should be ${ expectedHint.rect.width }`);
         }
 
         info(`Checking interaction for ${ hintTarget }`);
         interactionEl.scrollIntoView(false);
         const win = hintEl.ownerGlobal;
-        // Mouse out once from pathEl.
-        EventUtils.synthesizeMouse(interactionEl, -1, -1, { type: "mouseout" }, win);
-        is(win.getComputedStyle(displayedEl).strokeOpacity, 0,
-          `stroke-opacity of hintEl for ${ hintTarget } should be 0` +
-          " while mouse is out from the element");
         // Mouse over the pathEl.
         ok(isStrokeChangedByMouseOver(interactionEl, displayedEl, win),
-          `stroke-opacity of hintEl for ${ hintTarget } should be 1` +
-          " while mouse is over the element");
+          `stroke-opacity of hintEl for ${ hintTarget } should be 1 ` +
+          "while mouse is over the element");
+        // Mouse out from pathEl.
+        EventUtils.synthesizeMouse(panel.querySelector(".animation-toolbar"),
+                                   0, 0, { type: "mouseover" }, win);
+        is(win.getComputedStyle(displayedEl).strokeOpacity, 0,
+          `stroke-opacity of hintEl for ${ hintTarget } should be 0 ` +
+          "while mouse is out from the element");
       }
     }
   }
 });
 
 function isStrokeChangedByMouseOver(mouseoverEl, displayedEl, win) {
   const boundingBox = mouseoverEl.getBoundingClientRect();
   const x = boundingBox.width / 2;