Skip to content

DevTools only records Timeline data when Profiling#23137

Closed
bvaughn wants to merge 1 commit into
react:mainfrom
bvaughn:devtools-profiler-refactor-phase-2
Closed

DevTools only records Timeline data when Profiling#23137
bvaughn wants to merge 1 commit into
react:mainfrom
bvaughn:devtools-profiler-refactor-phase-2

Conversation

@bvaughn

@bvaughn bvaughn commented Jan 18, 2022

Copy link
Copy Markdown
Contributor

React will call the DevTools profiling hooks unconditionally (for DEV and profiling builds) but DevTools will only log the data to the User Timing API when DevTools is profiling.

Related to #22529


Note that this PR (in isolation) makes the Timeline profiler feature unusable. It will be part of a stack of PRs that will be merged together once finished.

@bvaughn bvaughn requested a review from lunaruan January 18, 2022 15:09
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jan 18, 2022
React will call the DevTools profiling hooks unconditionally (for DEV and profiling builds) but DevTools will only log the data to the User Timing API when DevTools is profiling.
@bvaughn bvaughn force-pushed the devtools-profiler-refactor-phase-2 branch from 19416aa to 79aef6a Compare January 20, 2022 18:52
@bvaughn

bvaughn commented Jan 20, 2022

Copy link
Copy Markdown
Contributor Author

Squashed into #23158

@bvaughn bvaughn closed this Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Component: Developer Tools React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants