Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Video Recording has become super slow within Chrome #16696

Closed
chasemgray opened this issue May 26, 2021 · 4 comments
Closed

Video Recording has become super slow within Chrome #16696

chasemgray opened this issue May 26, 2021 · 4 comments
Labels
stale no activity on this issue for a long period type: performance 🏃‍♀️ Performance related

Comments

@chasemgray
Copy link

Current behavior

Our test runs within Chrome are very slow after upgrading from 6.6.0 to 6.7.0. We also had to update to a more recent version of Chrome due to something required in cypress that only newer Chrome has.
We have frames per second tracking within some of our tests which run in a canvas and after the upgrade we see a drop of 50-75% of the FPS that we used to have before the upgrade.
We've tried all the recent versions of Cypress and they all have the same issues.
We also tried adjusting the windowSize as suggested in some solutions but this didn't help.
For example, launchOptions.args.push('--window-size=1280,768');

We tested out electron and it doesn't have the same issue.

We finally completely disabled video recording and our FPS shot back up to even faster than before so it's definitely something around the video recording.

The drop in performance immediately following our 6.7.0 upgrade and continuing into our 7.2.0 upgrade:
Screen Shot 2021-05-26 at 5 14 02 PM

After disabling video recording within Cypress 7.2.0:
Screen Shot 2021-05-26 at 5 07 09 PM

Desired behavior

Enabling recording should not have such a significant performance impact on our app. It becomes unusable with the latest cypress and causes constant flakes due to the slow framerate.

Versions

Cypress 6.6.0 worked for us, anything from 6.7.0 to 7.2.0 has not helped.

@sainthkh sainthkh added the type: performance 🏃‍♀️ Performance related label Jul 5, 2021
@helenanull
Copy link

helenanull commented Jul 20, 2021

Having the same exact issue, last usable Cypress version is 6.6.0, last attempt was to upgrade to 8.0.0 but still very slow (tests are taking twice as long if not more)
Browser Chrome and both headless and headed versions [both locally and in CI]
Even with video disabled, 6.6.0 is faster than newer versions

@jennifer-shehane
Copy link
Member

Chrome 89+ introduced a change to the amount of frames captured when video recording, so that it does slow down Cypress performance when recording video. This is the only performance affecting thing that we have diagnosed so far. #16152

7.2.0 introduced an issue that could cause some sites to load slower than before (cy.visit taking longer to load). #16671

🙏 Please provide a reproducible example to show a test suite taking twice as long in 8.0.0 versus 6.6.0. We have spent a lot of hours tracking this down and testing suites in many different containers and OS's and have not come up with any other causes than the ones I listed.

@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label May 17, 2023
@cypress-app-bot
Copy link
Collaborator

This issue has been closed due to inactivity.

@cypress-app-bot cypress-app-bot closed this as not planned Won't fix, can't repro, duplicate, stale May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale no activity on this issue for a long period type: performance 🏃‍♀️ Performance related
Projects
None yet
Development

No branches or pull requests

5 participants