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 issue: Leaking ImageFragmentInfo #25928

Open
paulrouget opened this issue Mar 9, 2020 · 1 comment
Open

Memory issue: Leaking ImageFragmentInfo #25928

paulrouget opened this issue Mar 9, 2020 · 1 comment
Labels
A-content/images Interacting with images from web content I-memory-leak A memory leak has been observed.

Comments

@paulrouget
Copy link
Contributor

For a simple <img>, ImageFragmentInfo are cloned/new 28 times and released 26 times. Holding the image bytes in memory.

After the pipeline is destroyed, there is still 2 arcs in memory:

Some(Arc::new(Image {

@vbkaisetsu
Copy link
Contributor

layout thread unreachable - leaking layout data occurs on reloading a page that only contains <img> tag:

% uname -a
Linux koichi-ThinkPad-T470s 5.3.0-40-generic #32-Ubuntu SMP Fri Jan 31 20:24:34 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
% echo '<img src="./resources/servo-64.png">' > ./test.html
% RUST_LOG=warn ./target/debug/servo ./test.html
[2020-03-25T08:23:27Z WARN  servo::context] Making an already current context current
[2020-03-25T08:23:27Z WARN  constellation::constellation] Visibility change for closed browsing context PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(1) }.
[2020-03-25T08:23:29Z WARN  script::timers] Resuming an already resumed timer.

After pressing Ctrl+R:

[2020-03-25T08:23:36Z WARN  script::timers] Resuming an already resumed timer.
[2020-03-25T08:23:36Z WARN  compositing::compositor] Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(1) }).
[2020-03-25T08:23:36Z WARN  servo_media_gstreamer] Trying to exec media action on an unknown client context
[2020-03-25T08:23:36Z WARN  script::dom::node] layout thread unreachable - leaking layout data
[2020-03-25T08:23:36Z WARN  script::dom::node] layout thread unreachable - leaking layout data
[2020-03-25T08:23:36Z WARN  script::dom::node] layout thread unreachable - leaking layout data
[2020-03-25T08:23:36Z WARN  script::dom::node] layout thread unreachable - leaking layout data
[2020-03-25T08:23:36Z WARN  script::dom::node] layout thread unreachable - leaking layout data

@jdm jdm added I-memory-leak A memory leak has been observed. A-content/images Interacting with images from web content labels Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-content/images Interacting with images from web content I-memory-leak A memory leak has been observed.
Projects
None yet
Development

No branches or pull requests

3 participants