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

Crashing memory leak when using budo in firefox 49+ #189

Closed
jaapz opened this issue Nov 30, 2016 · 8 comments
Closed

Crashing memory leak when using budo in firefox 49+ #189

jaapz opened this issue Nov 30, 2016 · 8 comments
Labels

Comments

@jaapz
Copy link

jaapz commented Nov 30, 2016

Since firefox 49+, I've been experiencing a major issue where firefox would completely eat up all available ram when the developer toolkit is open and budo triggers a live reload.

I can reproduce this 100% of the time.

I have no idea how I can debug this, could anyone help me out here?

@gregtatum
Copy link

I think I may have found it. We're looking into fixing it.

@gregtatum
Copy link

So it looks like the new debugger interface is leaking the sources on refresh, we're working on a fix for that one. The old debugger interface doesn't appear to be leaking, but it does duplicate the sources in memory like crazy, making it slow and bloated, which is a big problem for projects with large sources and sourcemaps. I found around 10-20 copies of the source maps in memory in the old interface 😮

The new debugger interface is currently in Dev Edition, and my guess is that we'll uplift any fixes for that, but probably not do anything with the old interface.

@mattdesl
Copy link
Owner

Awesome, thanks for the quick reply Greg! 😄

I've experienced something like this on a WebGL project a while back. Something I noticed is that Firefox likes to hang on to WebGL contexts on refresh — I'm not sure if the latest version still does this or not, or whether it's totally unrelated to the root problem.

@jaapz
Copy link
Author

jaapz commented Nov 30, 2016

@gregtatum that explains why even on 48, my development experience is less then ideal :) but at least on that version it doesn't crash my computer!

Is moving to firefox dev edition (for now) going to solve it for me?

@gregtatum
Copy link

gregtatum commented Dec 1, 2016

firefox-devtools/debugger#1369

There's a fix in place for the memory leak. I'll have to find out when that will land in Firefox. It will for sure be in the next release into Dev Edition (53), although I would guess they'll uplift it to the current Dev Edition (52). Before 51 and below are using the older debugger, and I doubt we'll address the memory issues for that.

Thanks for reporting it!

@jaapz
Copy link
Author

jaapz commented Dec 2, 2016

@gregtatum awesome, thanks! I'll keep using 48 for now until the fix is released :)

@mattdesl
Copy link
Owner

mattdesl commented Jan 3, 2017

I guess we can close this? 😄

@mattdesl mattdesl added the bug label Jan 3, 2017
@gregtatum
Copy link

Yup!

@jaapz jaapz closed this as completed Jan 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants