core(network-request): backport rendererStartTime #14481
Conversation
| /** When the renderer process initially discovers a network request. */ | ||
| this.rendererStartTime = -1; |
There was a problem hiding this comment.
still in seconds because everything else is without the rest of #14311
| isLinkPreload: networkRecord.isLinkPreload, | ||
| }, | ||
| timestamp: networkRecord.redirectResponseTimestamp || networkRecord.startTime || 0, | ||
| timestamp: networkRecord.redirectResponseTimestamp || networkRecord.rendererStartTime || |
There was a problem hiding this comment.
no test in this PR explicitly sets rendererStartTime, but by default network-records-to-devtools-log checks that round-tripping networkRecords->DevtoolsLog->networkRecords reproduces the input and some tests (like uses-http2) use slightly altered but real network records that have an actual rendererStartTime that now needs to make its way to the output
|
not sure what's up with the |
Yeah we update smokehouse to be more flexible with screenshot elements somewhere along the way. We can safely ignore those. |
3 participants
backport of just what's needed to include
rendererStartTimefrom #14311 in thenetwork-requestshidden audit for richer timing information in the November HTTP Archive run.No other changes (or uses of
rendererStartTime) included.