cleanTrace acts on the whole trace

GoogleChrome · Aug 5, 2016 · fa837c3 · fa837c3
1 parent c87fab5
commit fa837c3
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lighthouse-core/config/index.js b/lighthouse-core/config/index.js
@@ -28,7 +28,8 @@ const log = require('../lib/log');
 // and to change TracingStartedInBrowser events into TracingStartedInPage.
 // This is done by searching for most occuring threads and basing new events
 // off of those.
-function cleanTrace(traceEvents) {
+function cleanTrace(trace) {
+  const traceEvents = trace.traceEvents;
   // Keep track of most occuring threads
   const threads = [];
   const countsByThread = {};
@@ -103,7 +104,7 @@ function cleanTrace(traceEvents) {
   // and using TS of first found TracingStartedIn* event
   traceEvents.unshift(makeMockEvent(mostActiveFrame, ts));
 
-  return traceEvents;
+  return trace;
 }
 
 function filterPasses(passes, audits) {
@@ -196,7 +197,7 @@ function expandArtifacts(artifacts, includeSpeedline) {
           traceEvents: trace
         };
       }
-      cleanTrace(trace.traceEvents);
+      trace = cleanTrace(trace);
 
       expandedArtifacts.traces[key] = trace;
     });