Commit 67628e7
committed
Warn if addTransitionType is called when there are no pending Actions (#32793)
Stacked on #32792.
It's tricky to associate a specific `addTransitionType` call to a
specific `startTransition` call because we don't have `AsyncContext` in
browsers yet. However, we can keep track if there are any async
transitions running at all, and if not, warn. This should cover most
cases.
This also errors when inside a React render which might be a legit way
to associate a Transition Type to a specific render (e.g. based on props
changing) but we want to be a more conservative about allowing that yet.
If we wanted to support calling it in render, we might want to set which
Transition object is currently rendering but it's still tricky if the
render has `async function` components. So it might at least be
restricted to sync components (like Hooks).
DiffTrain build for [deca965](deca965)1 parent 70814b1 commit 67628e7
File tree
23 files changed
+133
-92
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
23 files changed
+133
-92
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: 14 additions & 7 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 | | |
| |||
7565 | 7565 | | |
7566 | 7566 | | |
7567 | 7567 | | |
7568 | | - | |
| 7568 | + | |
| 7569 | + | |
| 7570 | + | |
7569 | 7571 | | |
7570 | 7572 | | |
7571 | 7573 | | |
| |||
7964 | 7966 | | |
7965 | 7967 | | |
7966 | 7968 | | |
| 7969 | + | |
| 7970 | + | |
| 7971 | + | |
7967 | 7972 | | |
7968 | 7973 | | |
7969 | 7974 | | |
| |||
7991 | 7996 | | |
7992 | 7997 | | |
7993 | 7998 | | |
| 7999 | + | |
| 8000 | + | |
7994 | 8001 | | |
7995 | 8002 | | |
7996 | 8003 | | |
| |||
25668 | 25675 | | |
25669 | 25676 | | |
25670 | 25677 | | |
25671 | | - | |
| 25678 | + | |
25672 | 25679 | | |
25673 | 25680 | | |
25674 | 25681 | | |
25675 | | - | |
| 25682 | + | |
25676 | 25683 | | |
25677 | 25684 | | |
25678 | 25685 | | |
| |||
25709 | 25716 | | |
25710 | 25717 | | |
25711 | 25718 | | |
25712 | | - | |
| 25719 | + | |
25713 | 25720 | | |
25714 | 25721 | | |
25715 | | - | |
| 25722 | + | |
25716 | 25723 | | |
25717 | 25724 | | |
25718 | 25725 | | |
| |||
25856 | 25863 | | |
25857 | 25864 | | |
25858 | 25865 | | |
25859 | | - | |
| 25866 | + | |
25860 | 25867 | | |
Lines changed: 6 additions & 6 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 | | |
| |||
15983 | 15983 | | |
15984 | 15984 | | |
15985 | 15985 | | |
15986 | | - | |
| 15986 | + | |
15987 | 15987 | | |
15988 | 15988 | | |
15989 | 15989 | | |
15990 | 15990 | | |
15991 | 15991 | | |
15992 | 15992 | | |
15993 | | - | |
| 15993 | + | |
15994 | 15994 | | |
15995 | 15995 | | |
15996 | 15996 | | |
| |||
16012 | 16012 | | |
16013 | 16013 | | |
16014 | 16014 | | |
16015 | | - | |
| 16015 | + | |
16016 | 16016 | | |
16017 | 16017 | | |
16018 | | - | |
| 16018 | + | |
16019 | 16019 | | |
16020 | 16020 | | |
16021 | 16021 | | |
| |||
16119 | 16119 | | |
16120 | 16120 | | |
16121 | 16121 | | |
16122 | | - | |
| 16122 | + | |
Lines changed: 6 additions & 6 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 | | |
| |||
16631 | 16631 | | |
16632 | 16632 | | |
16633 | 16633 | | |
16634 | | - | |
| 16634 | + | |
16635 | 16635 | | |
16636 | 16636 | | |
16637 | 16637 | | |
16638 | 16638 | | |
16639 | 16639 | | |
16640 | 16640 | | |
16641 | | - | |
| 16641 | + | |
16642 | 16642 | | |
16643 | 16643 | | |
16644 | 16644 | | |
| |||
16660 | 16660 | | |
16661 | 16661 | | |
16662 | 16662 | | |
16663 | | - | |
| 16663 | + | |
16664 | 16664 | | |
16665 | 16665 | | |
16666 | | - | |
| 16666 | + | |
16667 | 16667 | | |
16668 | 16668 | | |
16669 | 16669 | | |
| |||
16782 | 16782 | | |
16783 | 16783 | | |
16784 | 16784 | | |
16785 | | - | |
| 16785 | + | |
Lines changed: 14 additions & 7 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 | | |
| |||
7573 | 7573 | | |
7574 | 7574 | | |
7575 | 7575 | | |
7576 | | - | |
| 7576 | + | |
| 7577 | + | |
| 7578 | + | |
7577 | 7579 | | |
7578 | 7580 | | |
7579 | 7581 | | |
| |||
7972 | 7974 | | |
7973 | 7975 | | |
7974 | 7976 | | |
| 7977 | + | |
| 7978 | + | |
| 7979 | + | |
7975 | 7980 | | |
7976 | 7981 | | |
7977 | 7982 | | |
| |||
7999 | 8004 | | |
8000 | 8005 | | |
8001 | 8006 | | |
| 8007 | + | |
| 8008 | + | |
8002 | 8009 | | |
8003 | 8010 | | |
8004 | 8011 | | |
| |||
25729 | 25736 | | |
25730 | 25737 | | |
25731 | 25738 | | |
25732 | | - | |
| 25739 | + | |
25733 | 25740 | | |
25734 | 25741 | | |
25735 | 25742 | | |
25736 | | - | |
| 25743 | + | |
25737 | 25744 | | |
25738 | 25745 | | |
25739 | 25746 | | |
| |||
25770 | 25777 | | |
25771 | 25778 | | |
25772 | 25779 | | |
25773 | | - | |
| 25780 | + | |
25774 | 25781 | | |
25775 | 25782 | | |
25776 | | - | |
| 25783 | + | |
25777 | 25784 | | |
25778 | 25785 | | |
25779 | 25786 | | |
| |||
26233 | 26240 | | |
26234 | 26241 | | |
26235 | 26242 | | |
26236 | | - | |
| 26243 | + | |
26237 | 26244 | | |
26238 | 26245 | | |
26239 | 26246 | | |
| |||
Lines changed: 6 additions & 6 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 | | |
| |||
15994 | 15994 | | |
15995 | 15995 | | |
15996 | 15996 | | |
15997 | | - | |
| 15997 | + | |
15998 | 15998 | | |
15999 | 15999 | | |
16000 | 16000 | | |
16001 | 16001 | | |
16002 | 16002 | | |
16003 | 16003 | | |
16004 | | - | |
| 16004 | + | |
16005 | 16005 | | |
16006 | 16006 | | |
16007 | 16007 | | |
| |||
16023 | 16023 | | |
16024 | 16024 | | |
16025 | 16025 | | |
16026 | | - | |
| 16026 | + | |
16027 | 16027 | | |
16028 | 16028 | | |
16029 | | - | |
| 16029 | + | |
16030 | 16030 | | |
16031 | 16031 | | |
16032 | 16032 | | |
| |||
16284 | 16284 | | |
16285 | 16285 | | |
16286 | 16286 | | |
16287 | | - | |
| 16287 | + | |
Lines changed: 6 additions & 6 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 | | |
| |||
16646 | 16646 | | |
16647 | 16647 | | |
16648 | 16648 | | |
16649 | | - | |
| 16649 | + | |
16650 | 16650 | | |
16651 | 16651 | | |
16652 | 16652 | | |
16653 | 16653 | | |
16654 | 16654 | | |
16655 | 16655 | | |
16656 | | - | |
| 16656 | + | |
16657 | 16657 | | |
16658 | 16658 | | |
16659 | 16659 | | |
| |||
16675 | 16675 | | |
16676 | 16676 | | |
16677 | 16677 | | |
16678 | | - | |
| 16678 | + | |
16679 | 16679 | | |
16680 | 16680 | | |
16681 | | - | |
| 16681 | + | |
16682 | 16682 | | |
16683 | 16683 | | |
16684 | 16684 | | |
| |||
16951 | 16951 | | |
16952 | 16952 | | |
16953 | 16953 | | |
16954 | | - | |
| 16954 | + | |
16955 | 16955 | | |
16956 | 16956 | | |
16957 | 16957 | | |
| |||
0 commit comments