Bug 1463621 - Part 3: Rename application name. r=pbro

Differential Revision: https://phabricator.services.mozilla.com/D5500

--HG--
extra : rebase_source : 58e2edeb00e21dc3b7b3de9b655b9ee15e5a1835

devtools/client/inspector/animation/animation.js

@@ -110,8 +110,8 @@ class AnimationInspector {
 
     const provider = createElement(Provider,
       {
-        id: "newanimationinspector",
-        key: "newanimationinspector",
+        id: "animationinspector",
+        key: "animationinspector",
         store: this.inspector.store
       },
       App(
@@ -275,7 +275,7 @@ class AnimationInspector {
   isPanelVisible() {
     return this.inspector && this.inspector.toolbox && this.inspector.sidebar &&
            this.inspector.toolbox.currentToolId === "inspector" &&
-           this.inspector.sidebar.getCurrentTabID() === "newanimationinspector";
+           this.inspector.sidebar.getCurrentTabID() === "animationinspector";
   }
 
   onAnimationStateChanged() {
@@ -625,7 +625,7 @@ class AnimationInspector {
   }
 
   async update() {
-    const done = this.inspector.updating("newanimationinspector");
+    const done = this.inspector.updating("animationinspector");
 
     const selection = this.inspector.selection;
     const animations =

devtools/client/inspector/animation/test/head.js

@@ -10,7 +10,7 @@ Services.scriptloader.loadSubScript(
   "chrome://mochitests/content/browser/devtools/client/inspector/test/head.js", this);
 
 const FRAME_SCRIPT_URL = CHROME_URL_ROOT + "doc_frame_script.js";
-const TAB_NAME = "newanimationinspector";
+const TAB_NAME = "animationinspector";
 
 const ANIMATION_L10N =
   new LocalizationHelper("devtools/client/locales/animationinspector.properties");
@@ -405,7 +405,7 @@ const mouseOutOnTargetNode = function(animationInspector, panel, index) {
 const selectAnimationInspector = async function(inspector) {
   await inspector.toolbox.selectTool("inspector");
   const onUpdated = inspector.once("inspector-updated");
-  inspector.sidebar.select("newanimationinspector");
+  inspector.sidebar.select("animationinspector");
   await onUpdated;
   await waitForRendering(inspector.animationinspector);
 };

devtools/client/inspector/inspector.js

@@ -935,7 +935,7 @@ Inspector.prototype = {
       INSPECTOR_L10N.getStr("inspector.sidebar.computedViewTitle"),
       defaultTab == "computedview");
 
-    const animationId = "newanimationinspector";
+    const animationId = "animationinspector";
     const animationTitle =
       INSPECTOR_L10N.getStr("inspector.sidebar.animationInspectorTitle");
     this.sidebar.queueTab(

devtools/client/shared/telemetry.js

@@ -575,10 +575,6 @@ class Telemetry {
     }
 
     if (charts.useTimedEvent) {
-      if (id === "newanimationinspector") {
-        id = "animationinspector";
-      }
-
       this.preparePendingEvent("devtools.main", "tool_timer", id, null, [
         "os",
         "time_open",
@@ -619,10 +615,6 @@ class Telemetry {
     }
 
     if (charts.useTimedEvent) {
-      if (id === "newanimationinspector") {
-        id = "animationinspector";
-      }
-
       const sig = `devtools.main,tool_timer,${id},null`;
       const event = PENDING_EVENTS.get(sig);
       const time = this.msSystemNow() - event.extra.time_open;
@@ -664,9 +656,6 @@ function getChartsFromToolId(id) {
   if (id === "PERFORMANCE") {
     id = "JSPROFILER";
   }
-  if (id === "NEWANIMATIONINSPECTOR") {
-    id = "ANIMATIONINSPECTOR";
-  }
 
   switch (id) {
     case "ABOUTDEBUGGING":

devtools/client/shared/test/browser_telemetry_sidebar.js

@@ -20,7 +20,7 @@ const DATA = [
     value: null,
     extra: {
       oldpanel: "computedview",
-      newpanel: "newanimationinspector"
+      newpanel: "animationinspector"
     }
   },
   {
@@ -30,7 +30,7 @@ const DATA = [
     object: "inspector",
     value: null,
     extra: {
-      oldpanel: "newanimationinspector",
+      oldpanel: "animationinspector",
       newpanel: "fontinspector"
     }
   },
@@ -64,7 +64,7 @@ const DATA = [
     value: null,
     extra: {
       oldpanel: "computedview",
-      newpanel: "newanimationinspector"
+      newpanel: "animationinspector"
     }
   },
   {
@@ -74,7 +74,7 @@ const DATA = [
     object: "inspector",
     value: null,
     extra: {
-      oldpanel: "newanimationinspector",
+      oldpanel: "animationinspector",
       newpanel: "fontinspector"
     }
   },
@@ -130,7 +130,7 @@ function testSidebar(toolbox) {
 
   const inspector = toolbox.getCurrentPanel();
   let sidebarTools = ["computedview", "layoutview", "fontinspector",
-                      "newanimationinspector"];
+                      "animationinspector"];
 
   // Concatenate the array with itself so that we can open each tool twice.
   sidebarTools = [...sidebarTools, ...sidebarTools];