core(lantern): remove last usages of Lighthouse NetworkRequest

GoogleChrome · Jun 11, 2024 · db61764 · db61764
1 parent a0d78ce
commit db61764
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 31 deletions.
diff --git a/core/lib/lantern/metric.js b/core/lib/lantern/metric.js
@@ -6,7 +6,7 @@
 
 import * as Lantern from './types/lantern.js';
 import {BaseNode} from '../../lib/lantern/base-node.js';
-import {RESOURCE_TYPES} from '../../lib/network-request.js';
+import {RESOURCE_TYPES} from '../network-request.js';
 
 /** @typedef {import('./base-node.js').Node} Node */
 /** @typedef {import('./network-node.js').NetworkNode} NetworkNode */

diff --git a/core/test/lib/lantern/simulator/network-analyzer-test.js b/core/test/lib/lantern/simulator/network-analyzer-test.js
@@ -9,7 +9,6 @@ import assert from 'assert/strict';
 import {NetworkAnalyzer} from '../../../../lib/lantern/simulator/network-analyzer.js';
 import {NetworkRecords} from '../../../../computed/network-records.js';
 import {readJson} from '../../../test-utils.js';
-import {NetworkRequest} from '../../../../lib/network-request.js';
 
 const devtoolsLog = readJson('../../../fixtures/artifacts/paul/devtoolslog.json', import.meta);
 const devtoolsLogWithRedirect = readJson('../../../fixtures/artifacts/redirect/devtoolslog.json', import.meta);
@@ -25,7 +24,7 @@ describe('DependencyGraph/Simulator/NetworkAnalyzer', () => {
     const url = opts.url || 'https://example.com';
     if (opts.networkRequestTime) opts.networkRequestTime *= 1000;
     if (opts.networkEndTime) opts.networkEndTime *= 1000;
-    const request = Object.assign(
+    return Object.assign(
       {
         url,
         requestId: recordId++,
@@ -40,7 +39,6 @@ describe('DependencyGraph/Simulator/NetworkAnalyzer', () => {
       },
       opts
     );
-    return NetworkRequest.asLanternNetworkRequest(request);
   }
 
   beforeEach(() => {
@@ -264,22 +262,6 @@ describe('DependencyGraph/Simulator/NetworkAnalyzer', () => {
         assertCloseEnough(result.median, resultApprox.median, 30);
       });
     });
-
-    it('should use lrStatistics when needed', () => {
-      global.isLightrider = true;
-      const request = createRecord({timing: {}, lrStatistics: {TCPMs: 100}});
-      const result = NetworkAnalyzer.estimateRTTByOrigin([request]);
-      const expected = {min: 50, max: 50, avg: 50, median: 50};
-      assert.deepStrictEqual(result.get('https://example.com'), expected);
-    });
-
-    it('should use lrStatistics when needed (h3)', () => {
-      global.isLightrider = true;
-      const request = createRecord({protocol: 'h3', timing: {}, lrStatistics: {TCPMs: 100}});
-      const result = NetworkAnalyzer.estimateRTTByOrigin([request]);
-      const expected = {min: 100, max: 100, avg: 100, median: 100};
-      assert.deepStrictEqual(result.get('https://example.com'), expected);
-    });
   });
 
   describe('#estimateServerResponseTimeByOrigin', () => {
@@ -331,14 +313,6 @@ describe('DependencyGraph/Simulator/NetworkAnalyzer', () => {
         assertCloseEnough(result.median, resultApprox.median, 30);
       });
     });
-
-    it('should use lrStatistics when needed', () => {
-      global.isLightrider = true;
-      const request = createRecord({timing: {}, lrStatistics: {TCPMs: 1, requestMs: 100}});
-      const result = NetworkAnalyzer.estimateServerResponseTimeByOrigin([request]);
-      const expected = {min: 100, max: 100, avg: 100, median: 100};
-      assert.deepStrictEqual(result.get('https://example.com'), expected);
-    });
   });
 
   describe('#estimateThroughput', () => {

diff --git a/core/test/lib/lantern/simulator/simulator-test.js b/core/test/lib/lantern/simulator/simulator-test.js
@@ -12,7 +12,6 @@ import {CPUNode} from '../../../../lib/lantern/cpu-node.js';
 import {Simulator} from '../../../../lib/lantern/simulator/simulator.js';
 import {DNSCache} from '../../../../lib/lantern/simulator/dns-cache.js';
 import {getURLArtifactFromDevtoolsLog, readJson} from '../../../test-utils.js';
-import {NetworkRequest} from '../../../../lib/network-request.js';
 import * as TraceEngineComputationData from '../../../../lib/lantern/trace-engine-computation-data.js';
 import {runTraceEngine} from '../metrics/metric-test-utils.js';
 
@@ -40,7 +39,7 @@ function request(opts) {
   delete opts.startTime;
   delete opts.endTime;
 
-  const request = Object.assign({
+  return Object.assign({
     requestId: opts.requestId || nextRequestId++,
     url,
     transferSize: opts.transferSize || 1000,
@@ -50,7 +49,6 @@ function request(opts) {
     rendererStartTime,
     networkEndTime,
   }, opts);
-  return NetworkRequest.asLanternNetworkRequest(request);
 }
 
 function cpuTask({tid, ts, duration}) {

diff --git a/core/test/lib/network-request-test.js b/core/test/lib/network-request-test.js
@@ -303,6 +303,28 @@ describe('NetworkRequest', () => {
     });
   });
 
+  describe('#asLanternNetworkRequest', () => {
+    it('uses lrStatistics to make timings', () => {
+      global.isLightrider = true;
+      const request = NetworkRequest.asLanternNetworkRequest({protocol: 'h2', timing: {}, lrStatistics: {TCPMs: 100}});
+      expect(request.timing).toStrictEqual({
+        connectStart: 0,
+        connectEnd: 100,
+        sslStart: 50,
+        sslEnd: 100,
+      });
+    });
+
+    it('uses lrStatistics to make timings (h3)', () => {
+      global.isLightrider = true;
+      const request = NetworkRequest.asLanternNetworkRequest({protocol: 'h3', timing: {}, lrStatistics: {TCPMs: 100}});
+      expect(request.timing).toStrictEqual({
+        connectStart: 0,
+        connectEnd: 100,
+      });
+    });
+  });
+
   describe('#isSecureRequest', () => {
     const isSecureRequest = NetworkRequest.isSecureRequest;