Bug 1406285 - Part 18: Rename tests for consistency. r=gl

MozReview-Commit-ID: L6F7rA2ibV3

--HG--
rename : devtools/client/inspector/animation/test/browser_animation_animation_list_exists.js => devtools/client/inspector/animation/test/browser_animation_animation-list.js
rename : devtools/client/inspector/animation/test/browser_animation_AnimationTarget.js => devtools/client/inspector/animation/test/browser_animation_animation-target.js
rename : devtools/client/inspector/animation/test/browser_animation_animation_list_time_tick.js => devtools/client/inspector/animation/test/browser_animation_animation-timeline-tick.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_AnimationName.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_animation-name.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_compositor.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_compositor.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_ComputedTimingPath.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_computed-timing-path.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_DelaySign.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_delay-sign.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_EffectTimingPath.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_effect-timing-path.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_EndDelaySign.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_end-delay-sign.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_NegativeDelayPath.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_negative-delay-path.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_NegativeEndDelayPath.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_negative-end-delay-path.js
rename : devtools/client/inspector/animation/test/browser_animation_SummaryGraph_tooltip.js => devtools/client/inspector/animation/test/browser_animation_summary-graph_tooltip.js
extra : rebase_source : 8824436fbc9455e9812363195b8ace0e16299a54

devtools/client/inspector/animation/test/browser.ini

@@ -11,17 +11,17 @@ support-files =
   !/devtools/client/shared/test/test-actor-registry.js
   !/devtools/client/shared/test/test-actor.js
 
-[browser_animation_animation_list_exists.js]
-[browser_animation_animation_list_time_tick.js]
-[browser_animation_AnimationTarget.js]
+[browser_animation_animation-list.js]
+[browser_animation_animation-target.js]
+[browser_animation_animation-timeline-tick.js]
 [browser_animation_empty_on_invalid_nodes.js]
 [browser_animation_inspector_exists.js]
-[browser_animation_SummaryGraph_AnimationName.js]
-[browser_animation_SummaryGraph_compositor.js]
-[browser_animation_SummaryGraph_ComputedTimingPath.js]
-[browser_animation_SummaryGraph_DelaySign.js]
-[browser_animation_SummaryGraph_EndDelaySign.js]
-[browser_animation_SummaryGraph_EffectTimingPath.js]
-[browser_animation_SummaryGraph_NegativeDelayPath.js]
-[browser_animation_SummaryGraph_NegativeEndDelayPath.js]
-[browser_animation_SummaryGraph_tooltip.js]
+[browser_animation_summary-graph_animation-name.js]
+[browser_animation_summary-graph_compositor.js]
+[browser_animation_summary-graph_computed-timing-path.js]
+[browser_animation_summary-graph_delay-sign.js]
+[browser_animation_summary-graph_end-delay-sign.js]
+[browser_animation_summary-graph_effect-timing-path.js]
+[browser_animation_summary-graph_negative-delay-path.js]
+[browser_animation_summary-graph_negative-end-delay-path.js]
+[browser_animation_summary-graph_tooltip.js]

devtools/client/inspector/animation/test/browser_animation_animation-timeline-tick.js

@@ -3,10 +3,10 @@
 
 "use strict";
 
-// Test for following time tick items.
+// Test for following timeline tick items.
 // * animation list header elements existence
-// * time tick item elements existence
-// * count and label of time tick elements changing by the sidebar width
+// * timeline tick item elements existence
+// * count and label of timeline tick elements changing by the sidebar width
 
 const TimeScale = require("devtools/client/inspector/animation/utils/timescale");
 const { findOptimalTimeInterval } =

devtools/client/inspector/animation/test/browser_animation_summary-graph_tooltip.js

@@ -11,7 +11,6 @@ const TEST_CASES = [
     expectedResult: {
       nameAndType: "cssanimation - CSS Animation",
       duration: "100s",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -20,7 +19,6 @@ const TEST_CASES = [
       nameAndType: "cssanimation - CSS Animation",
       duration: "100s",
       animationTimingFunction: "linear",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -29,7 +27,6 @@ const TEST_CASES = [
       nameAndType: "test-delay-animation - Script Animation",
       delay: "50s",
       duration: "100s",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -38,7 +35,6 @@ const TEST_CASES = [
       nameAndType: "test-negative-delay-animation - Script Animation",
       delay: "-50s",
       duration: "100s",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -47,7 +43,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       easing: "steps(2)",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -56,7 +51,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       endDelay: "50s",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -65,7 +59,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       endDelay: "-50s",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -75,7 +68,6 @@ const TEST_CASES = [
       duration: "100s",
       endDelay: "50s",
       fill: "forwards",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -85,7 +77,6 @@ const TEST_CASES = [
       duration: "100s",
       endDelay: "50s",
       iterations: "\u221E",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -95,7 +86,6 @@ const TEST_CASES = [
       duration: "100s",
       direction: "alternate",
       iterations: "\u221E",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -105,7 +95,6 @@ const TEST_CASES = [
       duration: "100s",
       direction: "alternate-reverse",
       iterations: "\u221E",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -115,7 +104,6 @@ const TEST_CASES = [
       duration: "100s",
       direction: "reverse",
       iterations: "\u221E",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -124,7 +112,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       fill: "backwards",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -135,7 +122,6 @@ const TEST_CASES = [
       duration: "100s",
       fill: "backwards",
       iterationStart: "0.5",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -144,7 +130,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       fill: "both",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -155,7 +140,6 @@ const TEST_CASES = [
       duration: "100s",
       fill: "both",
       iterationStart: "0.5",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -164,7 +148,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       fill: "forwards",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -173,7 +156,6 @@ const TEST_CASES = [
       nameAndType: "Script Animation",
       duration: "100s",
       iterationStart: "0.5",
-      isAllOnCompositor: true,
     },
   },
   {
@@ -188,7 +170,6 @@ const TEST_CASES = [
     expectedResult: {
       nameAndType: "Script Animation",
       duration: "100s",
-      isAllOnCompositor: true,
     },
   },
 ];
@@ -224,7 +205,6 @@ add_task(async function () {
       iterations,
       iterationStart,
       nameAndType,
-      isAllOnCompositor,
     } = expectedResult;
 
     ok(tooltip.startsWith(nameAndType), "Tooltip should start with name and type");
@@ -293,13 +273,5 @@ add_task(async function () {
       ok(!tooltip.includes("Iteration start:"),
          "Tooltip should not include iterationStart");
     }
-
-    if (isAllOnCompositor) {
-      const expected = "All animation properties are optimized";
-      ok(tooltip.includes(expected), `Tooltip should include '${ expected }'`);
-    } else {
-      ok(!tooltip.includes("optimized"),
-         "Tooltip should not include a message for optmization");
-    }
   }
 });