Commit 7723329
committed
Log Performance Track Entries for View Transitions (facebook#34510)
Stacked on facebook#34509.
View Transitions introduces a bunch of new types of gaps in the commit
phase which needs to be logged differently in the performance track.
One thing that can happen is that a `flushSync` update forces the View
Transition to abort before it has started if it happens in the gap
before the transition is ready. In that case we log "Interrupted View
Transition".
Otherwise, when we're done in `startViewTransition` there's some work to
finalize the animations before the `ready` calllback. This is logged as
"Starting Animation".
Then there's a gap before the passive effects fire which we log as
"Animating". This can be long unless they're forced to flush early e.g.
due to another lane updating.
The "Animating" track should then pick up which doesn't do yet. This one
is tricky because this is after the actual commit phase and needs to be
interrupted by new renders which themselves can be suspended on the
animation finshing.
This PR is just a subset of all the cases. Will need a lot more work.
<img width="679" height="161" alt="Screenshot 2025-09-16 at 10 19 06 PM"
src="https://github.com/user-attachments/assets/0407372d-aaed-41f5-a262-059b2686ae87"
/>
DiffTrain build for [84af908](facebook@84af908)1 parent 70fc333 commit 7723329
File tree
26 files changed
+2311
-1711
lines changed- compiled-rn
- facebook-fbsource/xplat/js
- RKJSModules/vendor/react
- react-dom/cjs
- react-test-renderer/cjs
- react/cjs
- react-native-github/Libraries/Renderer
- implementations
- tools/eslint-plugin-react-hooks
- cjs
26 files changed
+2311
-1711
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
404 | 404 | | |
405 | 405 | | |
406 | 406 | | |
407 | | - | |
| 407 | + | |
408 | 408 | | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 415 additions & 312 deletions
Large diffs are not rendered by default.
Lines changed: 32 additions & 32 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
12354 | 12354 | | |
12355 | 12355 | | |
12356 | 12356 | | |
12357 | | - | |
| 12357 | + | |
12358 | 12358 | | |
12359 | 12359 | | |
12360 | 12360 | | |
| |||
12642 | 12642 | | |
12643 | 12643 | | |
12644 | 12644 | | |
12645 | | - | |
| 12645 | + | |
12646 | 12646 | | |
12647 | 12647 | | |
12648 | 12648 | | |
12649 | | - | |
| 12649 | + | |
12650 | 12650 | | |
12651 | 12651 | | |
12652 | 12652 | | |
| |||
13016 | 13016 | | |
13017 | 13017 | | |
13018 | 13018 | | |
13019 | | - | |
| 13019 | + | |
13020 | 13020 | | |
13021 | 13021 | | |
13022 | 13022 | | |
| |||
13187 | 13187 | | |
13188 | 13188 | | |
13189 | 13189 | | |
13190 | | - | |
13191 | | - | |
13192 | | - | |
| 13190 | + | |
| 13191 | + | |
| 13192 | + | |
13193 | 13193 | | |
13194 | | - | |
13195 | | - | |
13196 | | - | |
13197 | | - | |
13198 | | - | |
13199 | | - | |
13200 | | - | |
| 13194 | + | |
| 13195 | + | |
| 13196 | + | |
| 13197 | + | |
| 13198 | + | |
| 13199 | + | |
| 13200 | + | |
13201 | 13201 | | |
13202 | | - | |
13203 | | - | |
| 13202 | + | |
| 13203 | + | |
13204 | 13204 | | |
13205 | 13205 | | |
13206 | 13206 | | |
| |||
17523 | 17523 | | |
17524 | 17524 | | |
17525 | 17525 | | |
17526 | | - | |
| 17526 | + | |
17527 | 17527 | | |
17528 | | - | |
17529 | | - | |
| 17528 | + | |
| 17529 | + | |
17530 | 17530 | | |
17531 | 17531 | | |
17532 | 17532 | | |
17533 | 17533 | | |
17534 | | - | |
17535 | | - | |
| 17534 | + | |
| 17535 | + | |
17536 | 17536 | | |
17537 | 17537 | | |
17538 | 17538 | | |
| |||
17552 | 17552 | | |
17553 | 17553 | | |
17554 | 17554 | | |
17555 | | - | |
| 17555 | + | |
17556 | 17556 | | |
17557 | | - | |
| 17557 | + | |
17558 | 17558 | | |
17559 | 17559 | | |
17560 | | - | |
| 17560 | + | |
17561 | 17561 | | |
17562 | 17562 | | |
17563 | | - | |
| 17563 | + | |
17564 | 17564 | | |
17565 | | - | |
17566 | | - | |
| 17565 | + | |
| 17566 | + | |
17567 | 17567 | | |
17568 | 17568 | | |
17569 | | - | |
17570 | | - | |
| 17569 | + | |
| 17570 | + | |
17571 | 17571 | | |
17572 | | - | |
| 17572 | + | |
17573 | 17573 | | |
17574 | 17574 | | |
17575 | 17575 | | |
| |||
17664 | 17664 | | |
17665 | 17665 | | |
17666 | 17666 | | |
17667 | | - | |
| 17667 | + | |
0 commit comments