Skip to content

Free memory allocated by testElement after usage - #6309

Merged
zpao merged 1 commit into
react:masterfrom
victor-homyakov:patch-1
Mar 21, 2016
Merged

Free memory allocated by testElement after usage#6309
zpao merged 1 commit into
react:masterfrom
victor-homyakov:patch-1

Conversation

@victor-homyakov

Copy link
Copy Markdown
Contributor

IE11 reports testElement as detached DOM node consuming memory. It is captured in closure of setInnerHTML function and could be safely cleared after feature test.

IE11 reports `testElement` as detached DOM node consuming memory. It is captured in closure of `setInnerHTML` function and could be safely cleared after feature test.
@victor-homyakov

Copy link
Copy Markdown
Contributor Author

image

@zpao

zpao commented Mar 21, 2016

Copy link
Copy Markdown
Contributor

👍 Nice find, thanks!

zpao added a commit that referenced this pull request Mar 21, 2016
Free memory allocated by testElement after usage
@zpao
zpao merged commit 75e4ef1 into react:master Mar 21, 2016
@victor-homyakov
victor-homyakov deleted the patch-1 branch March 21, 2016 21:50
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
Free memory allocated by testElement after usage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants