devtools-tracing 1.1.0 → 1.2.0

package/lib/front_end/models/trace/Processor.ts

@@ -1,652 +0,0 @@
-// Copyright 2023 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import * as Handlers from './handlers/handlers.js';
-import * as Helpers from './helpers/helpers.js';
-import * as Insights from './insights/insights.js';
-import * as Lantern from './lantern/lantern.js';
-import * as LanternComputationData from './LanternComputationData.js';
-import type * as Model from './ModelImpl.js';
-import * as Types from './types/types.js';
-
-const enum Status {
-  IDLE = 'IDLE',
-  PARSING = 'PARSING',
-  FINISHED_PARSING = 'FINISHED_PARSING',
-  ERRORED_WHILE_PARSING = 'ERRORED_WHILE_PARSING',
-}
-
-export class TraceParseProgressEvent extends Event {
-  static readonly eventName = 'traceparseprogress';
-  constructor(public data: Model.TraceParseEventProgressData, init: EventInit = {bubbles: true}) {
-    super(TraceParseProgressEvent.eventName, init);
-  }
-}
-
-/**
- * Parsing a trace can take time. On large traces we see a breakdown of time like so:
- *   - handleEvent() loop:  ~20%
- *   - finalize() loop:     ~60%
- *   - shallowClone calls:  ~20%
- * The numbers below are set so we can report a progress percentage of [0...1]
- */
-const enum ProgressPhase {
-  HANDLE_EVENT = 0.2,
-  FINALIZE = 0.8,
-  CLONE = 1.0,
-}
-function calculateProgress(value: number, phase: ProgressPhase): number {
-  // Finalize values should be [0.2...0.8]
-  if (phase === ProgressPhase.FINALIZE) {
-    return (value * (ProgressPhase.FINALIZE - ProgressPhase.HANDLE_EVENT)) + ProgressPhase.HANDLE_EVENT;
-  }
-  return value * phase;
-}
-
-declare global {
-  interface HTMLElementEventMap {
-    [TraceParseProgressEvent.eventName]: TraceParseProgressEvent;
-  }
-}
-
-export class TraceProcessor extends EventTarget {
-  // We force the Meta handler to be enabled, so the TraceHandlers type here is
-  // the model handlers the user passes in and the Meta handler.
-  readonly #traceHandlers: Partial<Handlers.Types.Handlers>;
-  #status = Status.IDLE;
-  #modelConfiguration = Types.Configuration.defaults();
-  #data: Handlers.Types.HandlerData|null = null;
-  #insights: Insights.Types.TraceInsightSets|null = null;
-
-  static createWithAllHandlers(): TraceProcessor {
-    return new TraceProcessor(Handlers.ModelHandlers, Types.Configuration.defaults());
-  }
-
-  /**
-   * This function is kept for testing with `stub`.
-   */
-  static getInsightRunners(): Insights.Types.InsightModelsType {
-    return {...Insights.Models};
-  }
-
-  constructor(traceHandlers: Partial<Handlers.Types.Handlers>, modelConfiguration?: Types.Configuration.Configuration) {
-    super();
-
-    this.#verifyHandlers(traceHandlers);
-    this.#traceHandlers = {
-      Meta: Handlers.ModelHandlers.Meta,
-      ...traceHandlers,
-    };
-    if (modelConfiguration) {
-      this.#modelConfiguration = modelConfiguration;
-    }
-    this.#passConfigToHandlers();
-  }
-
-  #passConfigToHandlers(): void {
-    for (const handler of Object.values(this.#traceHandlers)) {
-      // Bit of an odd double check, but without this TypeScript refuses to let
-      // you call the function as it thinks it might be undefined.
-      if ('handleUserConfig' in handler && handler.handleUserConfig) {
-        handler.handleUserConfig(this.#modelConfiguration);
-      }
-    }
-  }
-
-  /**
-   * When the user passes in a set of handlers, we want to ensure that we have all
-   * the required handlers. Handlers can depend on other handlers, so if the user
-   * passes in FooHandler which depends on BarHandler, they must also pass in
-   * BarHandler too. This method verifies that all dependencies are met, and
-   * throws if not.
-   **/
-  #verifyHandlers(providedHandlers: Partial<Handlers.Types.Handlers>): void {
-    // Tiny optimisation: if the amount of provided handlers matches the amount
-    // of handlers in the Handlers.ModelHandlers object, that means that the
-    // user has passed in every handler we have. So therefore they cannot have
-    // missed any, and there is no need to iterate through the handlers and
-    // check the dependencies.
-    if (Object.keys(providedHandlers).length === Object.keys(Handlers.ModelHandlers).length) {
-      return;
-    }
-    const requiredHandlerKeys = new Set<Handlers.Types.HandlerName>();
-    for (const [handlerName, handler] of Object.entries(providedHandlers)) {
-      requiredHandlerKeys.add(handlerName as Handlers.Types.HandlerName);
-      const deps = 'deps' in handler ? handler.deps() : [];
-      for (const depName of deps) {
-        requiredHandlerKeys.add(depName);
-      }
-    }
-
-    const providedHandlerKeys = new Set(Object.keys(providedHandlers));
-    // We always force the Meta handler to be enabled when creating the
-    // Processor, so if it is missing from the set the user gave us that is OK,
-    // as we will have enabled it anyway.
-    requiredHandlerKeys.delete('Meta');
-
-    for (const requiredKey of requiredHandlerKeys) {
-      if (!providedHandlerKeys.has(requiredKey)) {
-        throw new Error(`Required handler ${requiredKey} not provided.`);
-      }
-    }
-  }
-
-  reset(): void {
-    if (this.#status === Status.PARSING) {
-      throw new Error('Trace processor can\'t reset while parsing.');
-    }
-
-    const handlers = Object.values(this.#traceHandlers);
-    for (const handler of handlers) {
-      handler.reset();
-    }
-
-    this.#data = null;
-    this.#insights = null;
-    this.#status = Status.IDLE;
-  }
-
-  async parse(traceEvents: readonly Types.Events.Event[], options: Types.Configuration.ParseOptions): Promise<void> {
-    if (this.#status !== Status.IDLE) {
-      throw new Error(`Trace processor can't start parsing when not idle. Current state: ${this.#status}`);
-    }
-
-    if (typeof options.isCPUProfile === 'undefined' && options.metadata) {
-      options.isCPUProfile = options.metadata.dataOrigin === Types.File.DataOrigin.CPU_PROFILE;
-    }
-
-    options.logger?.start('total');
-    try {
-      this.#status = Status.PARSING;
-      options.logger?.start('parse');
-      await this.#computeParsedTrace(traceEvents, options);
-      options.logger?.end('parse');
-      if (this.#data && !options.isCPUProfile) {  // We do not calculate insights for CPU Profiles.
-        options.logger?.start('insights');
-        this.#computeInsights(this.#data, traceEvents, options);
-        options.logger?.end('insights');
-      }
-      this.#status = Status.FINISHED_PARSING;
-    } catch (e) {
-      this.#status = Status.ERRORED_WHILE_PARSING;
-      throw e;
-    } finally {
-      options.logger?.end('total');
-    }
-  }
-
-  /**
-   * Run all the handlers and set the result to `#data`.
-   */
-  async #computeParsedTrace(traceEvents: readonly Types.Events.Event[], options: Types.Configuration.ParseOptions):
-      Promise<void> {
-    /**
-     * We want to yield regularly to maintain responsiveness. If we yield too often, we're wasting idle time.
-     * We could do this by checking `performance.now()` regularly, but it's an expensive call in such a hot loop.
-     * `eventsPerChunk` is an approximated proxy metric.
-     * But how big a chunk? We're aiming for long tasks that are no smaller than 100ms and not bigger than 200ms.
-     * It's CPU dependent, so it should be calibrated on oldish hardware.
-     * Illustration of a previous change to `eventsPerChunk`: https://imgur.com/wzp8BnR
-     */
-    const eventsPerChunk = 50_000;
-    // Convert to array so that we are able to iterate all handlers multiple times.
-    const sortedHandlers = [...sortHandlers(this.#traceHandlers).entries()];
-
-    // Reset.
-    for (const [, handler] of sortedHandlers) {
-      handler.reset();
-    }
-
-    options.logger?.start('parse:handleEvent');
-
-    // Handle each event.
-    for (let i = 0; i < traceEvents.length; ++i) {
-      // Every so often we take a break just to render.
-      if (i % eventsPerChunk === 0 && i) {
-        // Take the opportunity to provide status update events.
-        const percent = calculateProgress(i / traceEvents.length, ProgressPhase.HANDLE_EVENT);
-        this.dispatchEvent(new TraceParseProgressEvent({percent}));
-        // TODO(paulirish): consider using `scheduler.yield()` or `scheduler.postTask(() => {}, {priority: 'user-blocking'})`
-        await new Promise(resolve => setTimeout(resolve, 0));
-      }
-      const event = traceEvents[i];
-      for (let j = 0; j < sortedHandlers.length; ++j) {
-        const [, handler] = sortedHandlers[j];
-        handler.handleEvent(event);
-      }
-    }
-
-    options.logger?.end('parse:handleEvent');
-
-    // Finalize.
-    const finalizeOptions: Handlers.Types.FinalizeOptions = {
-      ...options,
-      allTraceEvents: traceEvents,
-    };
-    for (let i = 0; i < sortedHandlers.length; i++) {
-      const [name, handler] = sortedHandlers[i];
-      if (handler.finalize) {
-        options.logger?.start(`parse:${name}:finalize`);
-        // Yield to the UI because finalize() calls can be expensive
-        // TODO(jacktfranklin): consider using `scheduler.yield()` or `scheduler.postTask(() => {}, {priority: 'user-blocking'})`
-        await new Promise(resolve => setTimeout(resolve, 0));
-        await handler.finalize(finalizeOptions);
-        options.logger?.end(`parse:${name}:finalize`);
-      }
-      const percent = calculateProgress(i / sortedHandlers.length, ProgressPhase.FINALIZE);
-      this.dispatchEvent(new TraceParseProgressEvent({percent}));
-    }
-
-    options.logger?.start('parse:handler.data()');
-    const parsedTrace = {};
-    for (const [name, handler] of Object.entries(this.#traceHandlers)) {
-      Object.assign(parsedTrace, {[name]: handler.data()});
-    }
-    options.logger?.end('parse:handler.data()');
-
-    this.dispatchEvent(new TraceParseProgressEvent({percent: ProgressPhase.CLONE}));
-
-    this.#data = parsedTrace as Handlers.Types.HandlerData;
-  }
-
-  get data(): Handlers.Types.HandlerData|null {
-    if (this.#status !== Status.FINISHED_PARSING) {
-      return null;
-    }
-
-    return this.#data;
-  }
-
-  get insights(): Insights.Types.TraceInsightSets|null {
-    if (this.#status !== Status.FINISHED_PARSING) {
-      return null;
-    }
-
-    return this.#insights;
-  }
-
-  #createLanternContext(
-      data: Handlers.Types.HandlerData, traceEvents: readonly Types.Events.Event[], frameId: string,
-      navigationId: string, options: Types.Configuration.ParseOptions): Insights.Types.LanternContext|undefined {
-    // Check for required handlers.
-    if (!data.NetworkRequests || !data.Workers || !data.PageLoadMetrics) {
-      return;
-    }
-    if (!data.NetworkRequests.byTime.length) {
-      throw new Lantern.Core.LanternError('No network requests found in trace');
-    }
-
-    const navStarts = data.Meta.navigationsByFrameId.get(frameId);
-    const navStartIndex = navStarts?.findIndex(n => n.args.data?.navigationId === navigationId);
-    if (!navStarts || navStartIndex === undefined || navStartIndex === -1) {
-      throw new Lantern.Core.LanternError('Could not find navigation start');
-    }
-
-    const startTime = navStarts[navStartIndex].ts;
-    const endTime = navStartIndex + 1 < navStarts.length ? navStarts[navStartIndex + 1].ts : Number.POSITIVE_INFINITY;
-    const boundedTraceEvents = traceEvents.filter(e => e.ts >= startTime && e.ts < endTime);
-
-    // Lantern.Types.TraceEvent and Types.Events.Event represent the same
-    // object - a trace event - but one is more flexible than the other. It should be safe to cast between them.
-    const trace: Lantern.Types.Trace = {
-      traceEvents: boundedTraceEvents as unknown as Lantern.Types.TraceEvent[],
-    };
-
-    const requests = LanternComputationData.createNetworkRequests(trace, data, startTime, endTime);
-    const graph = LanternComputationData.createGraph(requests, trace, data);
-    const processedNavigation = LanternComputationData.createProcessedNavigation(data, frameId, navigationId);
-
-    const networkAnalysis = Lantern.Core.NetworkAnalyzer.analyze(requests);
-    if (!networkAnalysis) {
-      return;
-    }
-
-    const lanternSettings: Lantern.Types.Simulation.Settings = {
-      // TODO(crbug.com/372674229): if devtools throttling was on, does this network analysis capture
-      // that? Do we need to set 'devtools' throttlingMethod?
-      networkAnalysis,
-      throttlingMethod: 'provided',
-      ...options.lanternSettings,
-    };
-    const simulator: Lantern.Simulation.Simulator<Types.Events.SyntheticNetworkRequest> =
-        Lantern.Simulation.Simulator.createSimulator(lanternSettings);
-
-    const computeData = {graph, simulator, processedNavigation};
-    const fcpResult = Lantern.Metrics.FirstContentfulPaint.compute(computeData);
-    const lcpResult = Lantern.Metrics.LargestContentfulPaint.compute(computeData, {fcpResult});
-    const interactiveResult = Lantern.Metrics.Interactive.compute(computeData, {lcpResult});
-    const tbtResult = Lantern.Metrics.TotalBlockingTime.compute(computeData, {fcpResult, interactiveResult});
-    const metrics = {
-      firstContentfulPaint: fcpResult,
-      interactive: interactiveResult,
-      largestContentfulPaint: lcpResult,
-      totalBlockingTime: tbtResult,
-    };
-
-    return {requests, graph, simulator, metrics};
-  }
-
-  /**
-   * Sort the insight models based on the impact of each insight's estimated savings, additionally weighted by the
-   * worst metrics according to field data (if present).
-   */
-  sortInsightSet(insightSet: Insights.Types.InsightSet, metadata: Types.File.MetaData|null): void {
-    // The initial order of the insights is alphabetical, based on `front_end/models/trace/insights/Models.ts`.
-    // The order here provides a baseline that groups insights in a more logical way.
-    const baselineOrder: Record<keyof Insights.Types.InsightModels, null> = {
-      INPBreakdown: null,
-      LCPBreakdown: null,
-      LCPDiscovery: null,
-      CLSCulprits: null,
-      RenderBlocking: null,
-      NetworkDependencyTree: null,
-      ImageDelivery: null,
-      DocumentLatency: null,
-      FontDisplay: null,
-      Viewport: null,
-      DOMSize: null,
-      ThirdParties: null,
-      DuplicatedJavaScript: null,
-      SlowCSSSelector: null,
-      ForcedReflow: null,
-      Cache: null,
-      ModernHTTP: null,
-      LegacyJavaScript: null,
-    };
-
-    // Determine the weights for each metric based on field data, utilizing the same scoring curve that Lighthouse uses.
-    const weights = Insights.Common.calculateMetricWeightsForSorting(insightSet, metadata);
-
-    // Normalize the estimated savings to a single number, weighted by its relative impact
-    // to the page experience based on the same scoring curve that Lighthouse uses.
-    const observedLcpMicro = Insights.Common.getLCP(insightSet)?.value;
-    const observedLcp = observedLcpMicro ? Helpers.Timing.microToMilli(observedLcpMicro) : Types.Timing.Milli(0);
-    const observedCls = Insights.Common.getCLS(insightSet).value;
-
-    // INP is special - if users did not interact with the page, we'll have no INP, but we should still
-    // be able to prioritize insights based on this metric. When we observe no interaction, instead use
-    // a default value for the baseline INP.
-    const observedInpMicro = Insights.Common.getINP(insightSet)?.value;
-    const observedInp = observedInpMicro ? Helpers.Timing.microToMilli(observedInpMicro) : Types.Timing.Milli(200);
-
-    const observedLcpScore =
-        observedLcp !== undefined ? Insights.Common.evaluateLCPMetricScore(observedLcp) : undefined;
-    const observedInpScore = Insights.Common.evaluateINPMetricScore(observedInp);
-    const observedClsScore = Insights.Common.evaluateCLSMetricScore(observedCls);
-
-    const insightToSortingRank = new Map<string, number>();
-    for (const [name, model] of Object.entries(insightSet.model)) {
-      const lcp = model.metricSavings?.LCP ?? 0;
-      const inp = model.metricSavings?.INP ?? 0;
-      const cls = model.metricSavings?.CLS ?? 0;
-
-      const lcpPostSavings =
-          observedLcp !== undefined ? Math.max(0, observedLcp - lcp) as Types.Timing.Milli : undefined;
-      const inpPostSavings = Math.max(0, observedInp - inp) as Types.Timing.Milli;
-      const clsPostSavings = Math.max(0, observedCls - cls);
-
-      let score = 0;
-      if (weights.lcp && lcp && observedLcpScore !== undefined && lcpPostSavings !== undefined) {
-        score += weights.lcp * (Insights.Common.evaluateLCPMetricScore(lcpPostSavings) - observedLcpScore);
-      }
-      if (weights.inp && inp && observedInpScore !== undefined) {
-        score += weights.inp * (Insights.Common.evaluateINPMetricScore(inpPostSavings) - observedInpScore);
-      }
-      if (weights.cls && cls && observedClsScore !== undefined) {
-        score += weights.cls * (Insights.Common.evaluateCLSMetricScore(clsPostSavings) - observedClsScore);
-      }
-
-      insightToSortingRank.set(name, score);
-    }
-
-    // Now perform the actual sorting.
-    const baselineOrderKeys = Object.keys(baselineOrder);
-    const orderedKeys = Object.keys(insightSet.model);
-    orderedKeys.sort((a, b) => {
-      const a1 = baselineOrderKeys.indexOf(a);
-      const b1 = baselineOrderKeys.indexOf(b);
-      if (a1 >= 0 && b1 >= 0) {
-        return a1 - b1;
-      }
-      if (a1 >= 0) {
-        return -1;
-      }
-      if (b1 >= 0) {
-        return 1;
-      }
-      return 0;
-    });
-    orderedKeys.sort((a, b) => (insightToSortingRank.get(b) ?? 0) - (insightToSortingRank.get(a) ?? 0));
-
-    const newModel = {} as Insights.Types.InsightModels;
-    for (const key of orderedKeys as Array<keyof Insights.Types.InsightModels>) {
-      const model = insightSet.model[key];
-      // @ts-expect-error Maybe someday typescript will be powerful enough to handle this.
-      newModel[key] = model;
-    }
-    insightSet.model = newModel;
-  }
-
-  #computeInsightSet(data: Handlers.Types.HandlerData, context: Insights.Types.InsightSetContext): void {
-    const logger = context.options.logger;
-
-    let id, urlString, navigation;
-    if (context.navigation) {
-      id = context.navigationId;
-      urlString = data.Meta.finalDisplayUrlByNavigationId.get(context.navigationId) ?? data.Meta.mainFrameURL;
-      navigation = context.navigation;
-    } else {
-      id = Types.Events.NO_NAVIGATION;
-      urlString = data.Meta.finalDisplayUrlByNavigationId.get('') ?? data.Meta.mainFrameURL;
-    }
-
-    const insightSetModel = {} as Insights.Types.InsightSet['model'];
-
-    for (const [name, insight] of Object.entries(TraceProcessor.getInsightRunners())) {
-      let model: Insights.Types.InsightModel|Error;
-      try {
-        logger?.start(`insights:${name}`);
-        model = insight.generateInsight(data, context);
-        model.frameId = context.frameId;
-        const navId = context.navigation?.args.data?.navigationId;
-        if (navId) {
-          model.navigationId = navId;
-        }
-        model.createOverlays = () => {
-          // @ts-expect-error: model is a union of all possible insight model types.
-          return insight.createOverlays(model);
-        };
-      } catch (err) {
-        model = err;
-      } finally {
-        logger?.end(`insights:${name}`);
-      }
-      Object.assign(insightSetModel, {[name]: model});
-    }
-
-    // We may choose to exclude the insightSet if it's trivial. Trivial means:
-    //   1. There's no navigation (it's an initial trace period)
-    //   2. The duration is short.
-    //   3. All the insights are passing (aka no insights to show the user)
-    //   4. It has no metrics to report (apart from a CLS of 0, which is default)
-    // Generally, these cases are the short time ranges before a page reload starts.
-    const isNavigation = id === Types.Events.NO_NAVIGATION;
-    const trivialThreshold = Helpers.Timing.milliToMicro(Types.Timing.Milli(5000));
-    const everyInsightPasses = Object.values(insightSetModel)
-                                   .filter(model => !(model instanceof Error))
-                                   .every(model => model.state === 'pass');
-
-    const noLcp = !insightSetModel.LCPBreakdown.lcpEvent;
-    const noInp = !insightSetModel.INPBreakdown.longestInteractionEvent;
-    const noLayoutShifts = insightSetModel.CLSCulprits.shifts?.size === 0;
-    const shouldExclude = isNavigation && context.bounds.range < trivialThreshold && everyInsightPasses && noLcp &&
-        noInp && noLayoutShifts;
-    if (shouldExclude) {
-      return;
-    }
-
-    let url;
-    try {
-      url = new URL(urlString);
-    } catch {
-      // We're pretty sure this only happens for our test fixture: missing-url.json.gz. Shouldn't
-      // happen for real traces.
-      return;
-    }
-
-    const insightSet: Insights.Types.InsightSet = {
-      id,
-      url,
-      navigation,
-      frameId: context.frameId,
-      bounds: context.bounds,
-      model: insightSetModel,
-    };
-    if (!this.#insights) {
-      this.#insights = new Map();
-    }
-    this.#insights.set(insightSet.id, insightSet);
-    this.sortInsightSet(insightSet, context.options.metadata ?? null);
-  }
-
-  /**
-   * Run all the insights and set the result to `#insights`.
-   */
-  #computeInsights(
-      data: Handlers.Types.HandlerData, traceEvents: readonly Types.Events.Event[],
-      options: Types.Configuration.ParseOptions): void {
-    // This insights map will be populated by the helper methods.
-    this.#insights = new Map();
-
-    // Filter main frame navigations to those that have the necessary data (frameId and navigationId).
-    // TODO(cjamcl): Does this filtering makes the "use the next nav as the end time" logic potentially broken? Are navs without nav id or frame even real?
-    const navigations = data.Meta.mainFrameNavigations.filter(
-        navigation => navigation.args.frame && navigation.args.data?.navigationId);
-
-    this.#computeInsightsForInitialTracePeriod(data, navigations, options);
-
-    for (const [index, navigation] of navigations.entries()) {
-      const min = navigation.ts;
-      // Use trace end for the last navigation, otherwise use the start of the next navigation.
-      const max = index + 1 < navigations.length ? navigations[index + 1].ts : data.Meta.traceBounds.max;
-      const bounds = Helpers.Timing.traceWindowFromMicroSeconds(min, max);
-      this.#computeInsightsForNavigation(navigation, bounds, data, traceEvents, options);
-    }
-  }
-
-  /**
-   * Computes insights for the period before the first navigation, or for the entire trace if no navigations exist.
-   */
-  #computeInsightsForInitialTracePeriod(
-      data: Handlers.Types.HandlerData, navigations: readonly Types.Events.NavigationStart[],
-      options: Types.Configuration.ParseOptions): void {
-    // Determine bounds: Use the period before the first navigation if navigations exist, otherwise use the entire trace bounds.
-    const bounds = navigations.length > 0 ?
-        Helpers.Timing.traceWindowFromMicroSeconds(data.Meta.traceBounds.min, navigations[0].ts) :
-        data.Meta.traceBounds;
-
-    const context: Insights.Types.InsightSetContext = {
-      options,
-      bounds,
-      frameId: data.Meta.mainFrameId,
-      // No navigation or lantern context applies to this initial/no-navigation period.
-    };
-    this.#computeInsightSet(data, context);
-  }
-
-  /**
-   * Computes insights for a specific navigation event.
-   */
-  #computeInsightsForNavigation(
-      navigation: Types.Events.NavigationStart, bounds: Types.Timing.TraceWindowMicro, data: Handlers.Types.HandlerData,
-      traceEvents: readonly Types.Events.Event[], options: Types.Configuration.ParseOptions): void {
-    const frameId = navigation.args.frame;
-    // Guaranteed by the filter in #computeInsights
-    const navigationId = navigation.args.data?.navigationId as string;
-
-    // The lantern sub-context is optional on InsightSetContext, so not setting it is OK.
-    // This is also a hedge against an error inside Lantern resulting in breaking the entire performance panel.
-    // Additionally, many trace fixtures are too old to be processed by Lantern.
-    let lantern: Insights.Types.LanternContext|undefined;
-    try {
-      options.logger?.start('insights:createLanternContext');
-      lantern = this.#createLanternContext(data, traceEvents, frameId, navigationId, options);
-    } catch (e) {
-      // Handle Lantern errors gracefully
-      // Don't allow an error in constructing the Lantern graphs to break the rest of the trace processor.
-      // Log unexpected errors, but suppress anything that occurs from a trace being too old.
-      // Otherwise tests using old fixtures become way too noisy.
-      const expectedErrors = [
-        'mainDocumentRequest not found',
-        'missing metric scores for main frame',
-        'missing metric: FCP',
-        'missing metric: LCP',
-        'No network requests found in trace',
-        'Trace is too old',
-      ];
-      if (!(e instanceof Lantern.Core.LanternError)) {
-        // If this wasn't a managed LanternError, the stack trace is likely needed for debugging.
-        console.error(e);
-      } else if (!expectedErrors.some(err => e.message === err)) {
-        // To reduce noise from tests, only print errors that are not expected to occur because a trace is
-        // too old (for which there is no single check).
-        console.error(e);
-      }
-    } finally {
-      options.logger?.end('insights:createLanternContext');
-    }
-
-    const context: Insights.Types.InsightSetContext = {
-      options,
-      bounds,
-      frameId,
-      navigation,
-      navigationId,
-      lantern,
-    };
-    this.#computeInsightSet(data, context);
-  }
-}
-
-/**
- * Some Handlers need data provided by others. Dependencies of a handler handler are
- * declared in the `deps` field.
- * @returns A map from trace event handler name to trace event handler whose entries
- * iterate in such a way that each handler is visited after its dependencies.
- */
-export function sortHandlers(traceHandlers: Partial<Record<Handlers.Types.HandlerName, Handlers.Types.Handler>>):
-    Map<Handlers.Types.HandlerName, Handlers.Types.Handler> {
-  const sortedMap = new Map<Handlers.Types.HandlerName, Handlers.Types.Handler>();
-  const visited = new Set<Handlers.Types.HandlerName>();
-  const visitHandler = (handlerName: Handlers.Types.HandlerName): void => {
-    if (sortedMap.has(handlerName)) {
-      return;
-    }
-    if (visited.has(handlerName)) {
-      let stackPath = '';
-      for (const handler of visited) {
-        if (stackPath || handler === handlerName) {
-          stackPath += `${handler}->`;
-        }
-      }
-      stackPath += handlerName;
-      throw new Error(`Found dependency cycle in trace event handlers: ${stackPath}`);
-    }
-    visited.add(handlerName);
-    const handler = traceHandlers[handlerName];
-    if (!handler) {
-      return;
-    }
-    const deps = handler.deps?.();
-    if (deps) {
-      deps.forEach(visitHandler);
-    }
-    sortedMap.set(handlerName, handler);
-  };
-
-  for (const handlerName of Object.keys(traceHandlers)) {
-    visitHandler(handlerName as Handlers.Types.HandlerName);
-  }
-  return sortedMap;
-}