Memory leak at refresh of Youtube pages (non-video)
Categories
(Core :: Performance, defect, P2)
Tracking
()
Performance Impact | medium |
People
(Reporter: dreadhead90, Unassigned)
Details
(Keywords: perf:resource-use)
Attachments
(2 files)
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Firefox/91.0
Steps to reproduce:
Starting Firefox with all addons (exept HTTPS everywhere) disabled and load up one tab with the youtube start page and one tab with my youtube subscriptions. Keep these tabs open during the day and reload them throughout the day.
For the attached memory reports I "simulated" the daily usage by repeatedly refreshing the tabs during a shorter time spread.
Actual results:
The web containers holding these tabs increase significantly in memory. At the end of the test, Firefox uses around 2 GB more memory than at the start.
Expected results:
Since the only actions taken between the measurements are refreshing of the tabs, I don't think that the memory consumption should increase. Or at least not to this extent.
Reporter | ||
Comment 1•3 years ago
|
||
Comment 2•3 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Core::Audio/Video: Playback' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.
Updated•3 years ago
|
Updated•3 years ago
|
Comment 3•3 years ago
|
||
Thanks for reporting this problem! In about:config, can you please try setting accessibility.force_disabled
to 1
and see if they resolves the issue?
Reporter | ||
Comment 4•3 years ago
|
||
(In reply to Denis Palmeiro [:denispal] from comment #3)
Thanks for reporting this problem! In about:config, can you please try setting
accessibility.force_disabled
to1
and see if they resolves the issue?
I made the change and tried a similar test as in my original post again. Unfortunately it didn't help and I still have the memory leak upon multiple refresh.
i am dealing with the same exact issue, i have tested in Firefox nightly and stable with accessibility.force_disabled being set 1.
clicking on minimize memory from about:memory page didn't free the excessive memory usage.
to replicate all you need is to keep reloading a youtube page.
Comment 6•3 years ago
|
||
dthayer was able to reproduce this by reloading the youtube page repeatedly. The about:memory dump shows most of the memory under js-realm(https://www.youtube.com/feed/subscriptions)
and dom
.
Jon, do you know how to proceed here?
Comment 7•3 years ago
|
||
I can reproduce this by loading two tabs containing youtube homepage and subscriptions page, and alternately reloading them.
Using the devtools memory shapshot doesn't show any memory increase (it stays at roughly 100MB).
Memory usage reported by about:memory increases each time, even after using 'minimize memory usage'. E.g.:
1,425.92 MB (100.0%) -- explicit
├──1,066.94 MB (74.82%) -- window-objects
│ ├──1,031.18 MB (72.32%) -- top(https://www.youtube.com/feed/subscriptions, id=137)
│ │ ├────939.79 MB (65.91%) -- active/window(https://www.youtube.com/feed/subscriptions)
It seems we are keeping the realm for the subscription page alive somehow:
40 (100.0%) -- js-main-runtime-realms
├──36 (90.00%) -- user
│ ├──21 (52.50%) ── https://www.youtube.com/feed/subscriptions [21]
Updated•3 years ago
|
Comment 8•3 years ago
|
||
This should have been fixed by the patch for bug 1717553. Can you confirm this works as expected in latest nightly?
(In reply to Jon Coppeard (:jonco) from comment #8)
This should have been fixed by the patch for bug 1717553. Can you confirm this works as expected in latest nightly?
i can confirm that it's fixed, thanks c:
Reporter | ||
Comment 10•3 years ago
|
||
(In reply to Jon Coppeard (:jonco) from comment #8)
This should have been fixed by the patch for bug 1717553. Can you confirm this works as expected in latest nightly?
Yes, I also tried to recreate my original experiment after downloading nightly, and it seems that the leak has been fixed!
Thanks to everyone who helped in this matter.
Comment 11•3 years ago
|
||
(In reply to ryn.and0 from comment #9)
Great, thanks both for confirming.
Updated•3 years ago
|
Updated•3 years ago
|
Description
•