Commit f0c8a506 authored by Nicolas Chevobbe's avatar Nicolas Chevobbe
Browse files

Bug 1741172 - [devtools] Tune console.streamlog DAMP test.... 

Bug 1741172 - [devtools] Tune console.streamlog DAMP test. r=jdescottes,perftest-reviewers,AlexandruIonescu.

The test logs messages in a requestAnimationFrame loop to check if logging impact
the performance of requestAnimationFrame.
The issue is that the tests logs an average of rounded values, which makes the
test results oscillate between 16 and 17 (ms), and this extra ms in enough to
trigger a perf alert as it represents a few percent of variance.
This patch makes it so we don't average, nor round the values anymore, which
shouldn't trigger perf alerts.

Differential Revision: https://phabricator.services.mozilla.com/D131795
parent c7792dec

testing/talos/talos/tests/devtools/addon/content/tests/webconsole/streamlog.js

+1 −2
Original line number Diff line number Diff line
@@ -42,8 +42,7 @@ module.exports = async function() { 
                              new Array(100).join(" DAMP? DAMP! "));

          content.requestAnimationFrame(log);

        } else {

          let avgTime = (content.performance.now() - startTime) / ${TOTAL_MESSAGES};

          sendSyncMessage("done", Math.round(avgTime));

          sendSyncMessage("done", (content.performance.now() - startTime));

        }

      }

      log();