Commit 533b5a9
committed
[Fiber] Trigger default indicator for isomorphic async actions with no root associated (#33190)
Stacked on #33160, #33162, #33186 and #33188.
We have a special case that's awkward for default indicators. When you
start a new async Transition from `React.startTransition` then there's
not yet any associated root with the Transition because you haven't
necessarily `setState` on anything yet until the promise resolves.
That's what `entangleAsyncAction` handles by creating a lane that
everything entangles with until all async actions are done.
If there are no sync updates before the end of the event, we should
trigger a default indicator until either the async action completes
without update or if it gets entangled with some roots we should keep it
going until those roots are done.
DiffTrain build for [3a5b326](3a5b326)1 parent 9c65b7e commit 533b5a9
File tree
24 files changed
+449
-449
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
24 files changed
+449
-449
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: 119 additions & 119 deletions
Large diffs are not rendered by default.
Lines changed: 28 additions & 28 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 | | |
| |||
12965 | 12965 | | |
12966 | 12966 | | |
12967 | 12967 | | |
12968 | | - | |
12969 | | - | |
12970 | | - | |
| 12968 | + | |
| 12969 | + | |
| 12970 | + | |
12971 | 12971 | | |
12972 | | - | |
12973 | | - | |
12974 | | - | |
12975 | | - | |
12976 | | - | |
12977 | | - | |
12978 | | - | |
| 12972 | + | |
| 12973 | + | |
| 12974 | + | |
| 12975 | + | |
| 12976 | + | |
| 12977 | + | |
| 12978 | + | |
12979 | 12979 | | |
12980 | | - | |
12981 | | - | |
| 12980 | + | |
| 12981 | + | |
12982 | 12982 | | |
12983 | 12983 | | |
12984 | 12984 | | |
| |||
17092 | 17092 | | |
17093 | 17093 | | |
17094 | 17094 | | |
17095 | | - | |
| 17095 | + | |
17096 | 17096 | | |
17097 | | - | |
17098 | | - | |
| 17097 | + | |
| 17098 | + | |
17099 | 17099 | | |
17100 | 17100 | | |
17101 | 17101 | | |
17102 | 17102 | | |
17103 | | - | |
17104 | | - | |
| 17103 | + | |
| 17104 | + | |
17105 | 17105 | | |
17106 | 17106 | | |
17107 | 17107 | | |
| |||
17121 | 17121 | | |
17122 | 17122 | | |
17123 | 17123 | | |
17124 | | - | |
| 17124 | + | |
17125 | 17125 | | |
17126 | | - | |
| 17126 | + | |
17127 | 17127 | | |
17128 | 17128 | | |
17129 | | - | |
| 17129 | + | |
17130 | 17130 | | |
17131 | 17131 | | |
17132 | | - | |
| 17132 | + | |
17133 | 17133 | | |
17134 | | - | |
17135 | | - | |
| 17134 | + | |
| 17135 | + | |
17136 | 17136 | | |
17137 | 17137 | | |
17138 | | - | |
17139 | | - | |
| 17138 | + | |
| 17139 | + | |
17140 | 17140 | | |
17141 | | - | |
| 17141 | + | |
17142 | 17142 | | |
17143 | 17143 | | |
17144 | 17144 | | |
| |||
17224 | 17224 | | |
17225 | 17225 | | |
17226 | 17226 | | |
17227 | | - | |
| 17227 | + | |
Lines changed: 28 additions & 28 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 | | |
| |||
13667 | 13667 | | |
13668 | 13668 | | |
13669 | 13669 | | |
13670 | | - | |
13671 | | - | |
13672 | | - | |
| 13670 | + | |
| 13671 | + | |
| 13672 | + | |
13673 | 13673 | | |
13674 | | - | |
13675 | | - | |
13676 | | - | |
13677 | | - | |
13678 | | - | |
13679 | | - | |
13680 | | - | |
| 13674 | + | |
| 13675 | + | |
| 13676 | + | |
| 13677 | + | |
| 13678 | + | |
| 13679 | + | |
| 13680 | + | |
13681 | 13681 | | |
13682 | | - | |
13683 | | - | |
| 13682 | + | |
| 13683 | + | |
13684 | 13684 | | |
13685 | 13685 | | |
13686 | 13686 | | |
| |||
17802 | 17802 | | |
17803 | 17803 | | |
17804 | 17804 | | |
17805 | | - | |
| 17805 | + | |
17806 | 17806 | | |
17807 | | - | |
17808 | | - | |
| 17807 | + | |
| 17808 | + | |
17809 | 17809 | | |
17810 | 17810 | | |
17811 | 17811 | | |
17812 | 17812 | | |
17813 | | - | |
17814 | | - | |
| 17813 | + | |
| 17814 | + | |
17815 | 17815 | | |
17816 | 17816 | | |
17817 | 17817 | | |
| |||
17831 | 17831 | | |
17832 | 17832 | | |
17833 | 17833 | | |
17834 | | - | |
| 17834 | + | |
17835 | 17835 | | |
17836 | | - | |
| 17836 | + | |
17837 | 17837 | | |
17838 | 17838 | | |
17839 | | - | |
| 17839 | + | |
17840 | 17840 | | |
17841 | 17841 | | |
17842 | 17842 | | |
| |||
17854 | 17854 | | |
17855 | 17855 | | |
17856 | 17856 | | |
17857 | | - | |
| 17857 | + | |
17858 | 17858 | | |
17859 | | - | |
17860 | | - | |
| 17859 | + | |
| 17860 | + | |
17861 | 17861 | | |
17862 | 17862 | | |
17863 | | - | |
17864 | | - | |
| 17863 | + | |
| 17864 | + | |
17865 | 17865 | | |
17866 | | - | |
| 17866 | + | |
17867 | 17867 | | |
17868 | 17868 | | |
17869 | 17869 | | |
| |||
17949 | 17949 | | |
17950 | 17950 | | |
17951 | 17951 | | |
17952 | | - | |
| 17952 | + | |
0 commit comments