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

DefaultPrinter for Javascript #19

Open
Richard-Walton opened this issue Apr 23, 2014 · 0 comments
Open

DefaultPrinter for Javascript #19

Richard-Walton opened this issue Apr 23, 2014 · 0 comments

Comments

@Richard-Walton
Copy link

The default printer for JS targets (ConsoleView) leaks memory in the background (Tested in the latest Chrome, Firefox and Safari) (Interestingly IE 11 seems to be unaffected).

Basically the ConsoleView creates (but does not add to the document) DOM elements for all logged messages regardless of weather the printer has been "attached" and visible on screen. Because the printer is added to the Console by default, but not attached, it silently logs and creates DOM elements to the point of causing memory issues.

Because it creates DOM elements the chrome debugger doesn't show it as a javascript memory leak(!) (But pressing shift-escape and viewing the chrome task manager (or even the using your system task manager) will show the issue).

To fix the issue, it is as simple as "Console.removePrinter(Console.defaultPrinter)" but I'm still raising the issue as this issue is present by default and so will cause memory leaks by default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant