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

Memory leak in test svg-in-object-auto.html #3173

Open
1 task
InvalidUsernameException opened this issue Jan 7, 2025 · 0 comments
Open
1 task

Memory leak in test svg-in-object-auto.html #3173

InvalidUsernameException opened this issue Jan 7, 2025 · 0 comments
Labels
bug Something isn't working performance

Comments

@InvalidUsernameException
Copy link
Contributor

Summary

Executing the test under Tests/LibWeb/Text/input/wpt-import/html/rendering/replaced-elements/svg-embedded-sizing/svg-in-object-auto.html repeatedly in the same tab causes memory usage of the content process to grow. Invoking garbage collection manually has no or very little effect on memory usage.

I cannot reproduce the memory increase in other browsers. So it seems unlikely that this is a bug in the test itself.

Worth noting: Some of the tests in the same folder exhibit similar behavior, but the memory usage for them seems to stabilize after several tests executions.

Operating system

Linux

Steps to reproduce

Run ./Meta/ladybird.sh run ladybird Tests/LibWeb/Text/input/wpt-import/html/rendering/replaced-elements/svg-embedded-sizing/svg-in-object-auto.html and refresh the page several times

Expected behavior

Memory usage stays constant after explicitly invoking garbage collection.

Actual behavior

  • Memory usage increases with every page reload and does not decrease by running garbage collection.
  • Also after the test has been re-executed several times in the same tab, test execution becomes noticeably slower and idle CPU usage higher as well.

URL for a reduced test case

Tests/LibWeb/Text/input/wpt-import/html/rendering/replaced-elements/svg-embedded-sizing/svg-in-object-auto.html

HTML/SVG/etc. source for a reduced test case

N/A

Log output and (if possible) backtrace

N/A

Screenshots or screen recordings

No response

Build flags or config settings

No response

Contribute a patch?

  • I’ll contribute a patch for this myself.
@AtkinsSJ AtkinsSJ added bug Something isn't working performance labels Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working performance
Projects
None yet
Development

No branches or pull requests

2 participants