Bug 1230735 - AddonWatcher.alerts is now a map;r=Felipe
authorDavid Rajchenbach-Teller <dteller@mozilla.com>
Mon, 14 Dec 2015 22:57:13 +0100
changeset 278850 ab085ca3826b4c8880a93f7ad2d3c5e37d4a95e8
parent 278849 51c62defb59707b9de153e1741d9965b8ef9b0ea
child 278851 7a2f776d40c55c669370112fc559de7e8506ed50
push id29860
push usercbook@mozilla.com
push dateThu, 07 Jan 2016 10:51:20 +0000
treeherdermozilla-central@e0bcd16e1d4b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersFelipe
bugs1230735
milestone46.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 1230735 - AddonWatcher.alerts is now a map;r=Felipe
toolkit/components/aboutperformance/content/aboutPerformance.js
--- a/toolkit/components/aboutperformance/content/aboutPerformance.js
+++ b/toolkit/components/aboutperformance/content/aboutPerformance.js
@@ -563,18 +563,18 @@ var View = {
 
       let processes = delta.diff.processes.map(proc => `${proc.processId} (${proc.isChildProcess?"child":"parent"})`);
       cachedElements.eltProcess.textContent = `Processes: ${processes.join(", ")}`;
       let jankSuffix = "";
       let cpowSuffix = "";
       if (watcherAlerts) {
         let deltaAlerts = watcherAlerts.get(delta.diff.addonId);
         if (deltaAlerts) {
-          jankSuffix = ` (${deltaAlerts.alerts.longestDuration || 0} alerts)`;
-          cpowSuffix = ` (${deltaAlerts.alerts.totalCPOWTime || 0} alerts)`;
+          jankSuffix = ` (${deltaAlerts.get("longestDuration") || 0} alerts)`;
+          cpowSuffix = ` (${deltaAlerts.get("totalCPOWTime") || 0} alerts)`;
         }
       }
 
       let eltImpact = cachedElements.eltImpact;
       if (currentMode == MODE_RECENT) {
         cachedElements.eltRoot.setAttribute("impact", delta.diff.jank.longestDuration + 1);
         if (Delta.compare(delta, Delta.MAX_DELTA_FOR_GOOD_RECENT_PERFORMANCE) <= 0) {
           eltImpact.textContent = ` currently performs well.`;