Speedometer icon indicating copy to clipboard operation
Speedometer copied to clipboard

Add measurePrepare param

Open camillobruni opened this issue 1 year ago • 3 comments

  • Include the suite prepare time in total time if param.debugMetrics is set
  • Add SuiteRunner._suiteResults local variable

camillobruni avatar Oct 23 '24 14:10 camillobruni

What is the purpose of this change? Why do we ever want to include the prepare time in the total?

rniwa avatar Oct 23 '24 15:10 rniwa

This is useful for debugging and internal testing.

We sometimes run the Speedometer workloads internally including the startup time with a live network. Given that Speedometer is well-controlled this makes for a nice end2end browser test.

camillobruni avatar Oct 23 '24 15:10 camillobruni

To simplify discussions on this PR, I'm splitting off the refactoring parts into separate PRs.

camillobruni avatar Nov 04 '24 10:11 camillobruni

Deploy Preview for webkit-speedometer-preview ready!

Name Link
Latest commit 768ab21691bc250018f8d01631dc63c18e313b66
Latest deploy log https://app.netlify.com/projects/webkit-speedometer-preview/deploys/6835cb75d06a9300088d0cd6
Deploy Preview https://deploy-preview-440--webkit-speedometer-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar May 21 '25 12:05 netlify[bot]