[DevTools Bug]: Consistent memory leak when refreshing on React + non-React websites. about react

[hoxyq]

Thanks for reporting this and recording a demo, I will try to triage this early next week, hopefully shipping a fix soon.
If someone is already triaging this, feel free to open a PR and mentioning this issue.

[pwbriggs]

This was in incognito mode without any other extensions enabled

Just wondering, how'd you install React DevTools in incognito?

[braeden]

Also cannot reproduce anymore 😬 (same chrome version as @hoxyq ).

I'm slightly concerned that this might reappear unless we can pinpoint the Chromium commit that might have changed the behavior here...

[braeden]

This was in incognito mode without any other extensions enabled

Just wondering, how'd you install React DevTools in incognito?

There's an Allow in Incognito switch in the Details of any Chrome extension.

[erikasby]

Chrome Version 122.0.6261.129 (Official Build) (64-bit)
Can't reproduce. Refreshes accumulate MB infinitely, but waiting for ~5 seconds or pressing "Collect garbage" flushes everything instantly down to ~10 MB.

Firefox Version 123.0.1 (64-bit)
Measured private window in about:processes with and without React Dev Tools - memory behavior seems to be the same. Was able to instantly free up memory in about:memory with Free memory - GC on both occasions. From ~500 MB to ~210 MB instantly.

I guess not a Chromium or Firefox issue then if both browsers suddenly behave properly, or it can be?

[pwbriggs]

I see, thanks.

[mkdynamic]

I see similar behavior when the React Dev Tools is enabled, across all sites. Looking at the heap snapshots, you can see the previous window objects retained and accumulate on every reload. Maybe the Dev Tools is hanging on to a reference for every window object?

[hoxyq]

Attempted to triage this, noticed that the required condition for this to happen is chrome.devtools.network.onNavigated listeners here:

react/packages/react-devtools-extensions/src/main/index.js

Line 431 in ddd736d

chrome.devtools.network.onNavigated.addListener(syncSavedPreferences);

react/packages/react-devtools-extensions/src/main/index.js

Line 444 in ddd736d

chrome.devtools.network.onNavigated.addListener(debouncedOnNavigatedListener);

Removing both of them fixes the issue.

Having at least one of them still reproduces the memory leak, which is surprising, given the fact that the first one (syncSavedPreferences) is only using chrome.devtools.inspectedWindow.eval:

react/packages/react-devtools-extensions/src/main/syncSavedPreferences.js

Lines 12 to 36 in ddd736d

	// The renderer interface can't read saved component filters directly,
	// because they are stored in localStorage within the context of the extension.
	// Instead it relies on the extension to pass filters through.
	function syncSavedPreferences() {
	chrome.devtools.inspectedWindow.eval(
	`window.__REACT_DEVTOOLS_APPEND_COMPONENT_STACK__ = ${JSON.stringify(
	getAppendComponentStack(),
	)};
	window.__REACT_DEVTOOLS_BREAK_ON_CONSOLE_ERRORS__ = ${JSON.stringify(
	getBreakOnConsoleErrors(),
	)};
	window.__REACT_DEVTOOLS_COMPONENT_FILTERS__ = ${JSON.stringify(
	getSavedComponentFilters(),
	)};
	window.__REACT_DEVTOOLS_SHOW_INLINE_WARNINGS_AND_ERRORS__ = ${JSON.stringify(
	getShowInlineWarningsAndErrors(),
	)};
	window.__REACT_DEVTOOLS_HIDE_CONSOLE_LOGS_IN_STRICT_MODE__ = ${JSON.stringify(
	getHideConsoleLogsInStrictMode(),
	)};
	window.__REACT_DEVTOOLS_BROWSER_THEME__ = ${JSON.stringify(
	getBrowserTheme(),
	)};`,
	);
	}

Few ideas:

1. I don't have the time right now, but maybe its worth trying to create a small reproducible example of an extension, which uses both chrome.devtools.network.onNavigated and chrome.devtools.inspectedWindow.eval, which has the same memory leak issues?
2. Does this memory leak happen in Firefox as well?

[chrishoage]

Does this memory leak happen in Firefox as well?

I believe it does, yes. Today I noticed excessive memory usage in Firefox. I opened about:processes and saw "Extensions" using 5GB of memory.

One by one I turned off my extensions (not many, just 4) and watched the report on the memory usage for extensions.

Once I turned off "React Developer Tools" the memory usage for the "Extensions" line in about:processes went from over 5GB down to 600MB.

While I admit this anecdote isn't conclusive proof I think it serves as evidence that it could happen in firefox as well.

[jryans]

Similar to @chrishoage, I have also observed this memory leak in Firefox in the form of high memory usage (though I saw it in happening in a React-using content process) which seemed to go away after disabling this extension.

[hoxyq]

I am currently exploring an option to switch from chrome.devtools.inspectedWindow.eval to chrome.scripting.executeScript.

The main problem for us with this approach is the requirement for executing scripts in MAIN world, which Firefox doesn't support yet.

[WizardAlice]

It took me a whole Friday afternoon to finally find the issue came from this plugin.

[hoxyq]

My Chrome got updated, and I can't reproduce this on Chrome v122.0.6261.69, can someone confirm this as well?

[liy9426]

chrome v122.0.6261.112 still having this problem.....even if the webpage is not using React at all
subscribed to window 'message' event but consistently receving message with source 'react-devtools-content-sceipt'

[hoxyq]

Since we can't reproduce this, I am going to close this. Would be nice if someone can grep the changes from between Chrome 121 and Chrome 122 to identify the affecting commit, but this would require some time.

[torchesburn]

Since we can't reproduce this, I am going to close this. Would be nice if someone can grep the changes from between Chrome 121 and Chrome 122 to identify the affecting commit, but this would require some time.

you literally just refresh the tab and see the memory grow.