Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

WIP Profiling Report #53

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

gioragutt
Copy link
Contributor

@gioragutt gioragutt commented Oct 9, 2020

This PR is above the HTML Viewer PR, and should not be merged before it.

First commit is a cleanup, to move all hooks into the hooks folder.

Related to #54


Available reports:

  • profile.cpuprofile: Devtools -> Menu -> More Tools -> JavaScript Profiler -> Load.
  • performanceTrace.json: Devtools -> Performance -> Load Profile... (upload icon)

TIP: on mac, dotfiles are hidden by default. Use + + . to toggle viewing dotfiles in finder.


Another possible report we can generate is lighthouse report. It can be html, to more easily integrate it into the client, and/or json to be able to extract data from it.

@github-actions
Copy link

github-actions bot commented Oct 9, 2020

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@gioragutt gioragutt marked this pull request as draft October 10, 2020 13:38
instrumentor: IAutomationFrameworkInstrumentor,
page: RootCausePage
) {
if (isPlaywrightPage(page)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y? Is this not possible in playwright?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants