core(tsc): don't use index signature in simulator.js (#5897)

GoogleChrome · Aug 22, 2018 · 23eca9d · 23eca9d
1 parent 846bb1c
commit 23eca9d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lighthouse-core/gather/computed/load-simulator.js b/lighthouse-core/gather/computed/load-simulator.js
@@ -23,6 +23,7 @@ class LoadSimulatorArtifact extends ComputedArtifact {
     const {throttlingMethod, throttling} = data.settings;
     const networkAnalysis = await artifacts.requestNetworkAnalysis(data.devtoolsLog);
 
+    /** @type {LH.Gatherer.Simulation.Options} */
     const options = {
       additionalRttByOrigin: networkAnalysis.additionalRttByOrigin,
       serverResponseTimeByOrigin: networkAnalysis.serverResponseTimeByOrigin,

diff --git a/lighthouse-core/lib/dependency-graph/simulator/simulator.js b/lighthouse-core/lib/dependency-graph/simulator/simulator.js
@@ -66,6 +66,7 @@ class Simulator {
     this._nodeTimings = new Map();
     /** @type {Map<string, number>} */
     this._numberInProgressByType = new Map();
+    /** @type {Record<number, Set<Node>>} */
     this._nodes = {};
     this._dns = new DNSCache({rtt: this._rtt});
     // @ts-ignore
@@ -95,8 +96,8 @@ class Simulator {
     this._numberInProgressByType = new Map();
 
     this._nodes = {};
-    for (const key of Object.keys(NodeState)) {
-      this._nodes[NodeState[key]] = new Set();
+    for (const state of Object.values(NodeState)) {
+      this._nodes[state] = new Set();
     }
   }