Add timesJSLoadToRefreshDevTools config option #417
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jhen0409
force-pushed
the
refresh-devtools-on-reload
branch
from
e1c15cc to
592435e
Compare
jhen0409
force-pushed
the
refresh-devtools-on-reload
branch
from
10faf33 to
9c76508
Compare
jhen0409
force-pushed
the
refresh-devtools-on-reload
branch
from
9c76508 to
f8cbc4a
Compare
jhen0409
changed the title
jhen0409
added a commit
that referenced
this pull request
Oct 24, 2019
* Add timesJSReloadToRefreshDevTools config option * Update snapshot * Fix isDevToolsOpened check * Add log after refresh devtools * Update reloadCount order * Do selectRNDebuggerWorkerContext on devtools-opened * Update comment of option * Rename to timesJSLoadToRefreshDevTools * checkJSLoadCount for delta available * Update docs/troubleshooting.md
cryptodev523
pushed a commit
to cryptodev523/react-native-debugger
that referenced
this pull request
Jan 27, 2022
* Add timesJSReloadToRefreshDevTools config option * Update snapshot * Fix isDevToolsOpened check * Add log after refresh devtools * Update reloadCount order * Do selectRNDebuggerWorkerContext on devtools-opened * Update comment of option * Rename to timesJSLoadToRefreshDevTools * checkJSLoadCount for delta available * Update docs/troubleshooting.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related #151 and #405.
I add the
timesJSLoadToRefreshDevToolsconfig option, it will refresh devtools when doing JS reload every N times (-1 for disabled). This can effectively avoid possible memory leaks in devtools. (SeeConfig file in home directory)This is a compromise for solving the DevTools memory leak issue. Since it affects the use of devtools, it is disabled by default.