Closed
Bug 1498890
Opened 6 years ago
Closed 6 years ago
The "confirm you are about to close tabs" dialogue box uses 200MB RAM
Categories
(Core :: Graphics: WebRender, defect, P2)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
People
(Reporter: mayankleoboy1, Unassigned)
References
Details
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:64.0) Gecko/20100101 Firefox/64.0
Steps to reproduce:
create new profile. Enable WR. Restart
1.Open 2 tabs
1.5 Check RAM usage
2. Click on the "X" button on the top right corner of the window, to close the window
3. A dialogue bow will appear saying something like "Please confirm yiu are about to close 2 tabs"
4.Dont click on yes/ok. Keep this dialogue box open
Actual results:
Check RAM usage after step 3
It would be increased by ~200MB
Expected results:
not so
Reporter | ||
Comment 1•6 years ago
|
||
while I have 2 windows. Each has 2 tabs
Reporter | ||
Comment 2•6 years ago
|
||
After I try to close one of the windows by clicking on the "x" button on the top right corner, and a "confirm"dialogue box opens
Comment 3•6 years ago
|
||
Thanks for those memory reports Mayank, that's really helpful.
It looks like that dialog is triggering the lazy creation of one of the texture caches, which will consume 64 MB of vram straight out of the gate. And up until bug 1496168 landed this weekend, that would actually consume 33% more memory, because of mipmaps.
So that explains ~90MB of the regression. It looks like the total increase in committed memory is ~130MB, so there may be something else going on there (I do see the GPU cache growing). But I think we should fix the texture stuff first and then remeasure.
Depends on: 1494760
Updated•6 years ago
|
Updated•6 years ago
|
Blocks: stage-wr-trains
Priority: -- → P3
Reporter | ||
Comment 4•6 years ago
|
||
This is much less now if there is only 1 window open. If there are more than 1 windows, the dialogue box uses ~100MB RAM
Comment 5•6 years ago
|
||
Can you re-test this for us?
Flags: needinfo?(mayankleoboy1)
Priority: P3 → P2
Reporter | ||
Comment 6•6 years ago
|
||
This is much better now. I will close this bug. We can file a new bug if needed in future.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(mayankleoboy1)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•