Commit bb4b9e4
committed
Reset currentEventTransitionLane after flushing sync work (#33159)
This keeps track of the transition lane allocated for this event. I want
to be able to use the current one within sync work flushing to know
which lane needs its loading indicator cleared.
It's also a bit weird that transition work scheduled inside sync updates
in the same event aren't entangled with other transitions in that event
when `flushSync` is.
Therefore this moves it to reset after flushing.
It should have no impact. Just splitting it out into a separate PR for
an abundance of caution.
The only thing this might affect would be if the React internals throws
and it doesn't reset after. But really it doesn't really have to reset
and they're all entangled anyway.
DiffTrain build for [676f087](676f087)1 parent 4a0575d commit bb4b9e4
File tree
24 files changed
+159
-177
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
+159
-177
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: 10 additions & 12 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 | | |
| |||
15616 | 15616 | | |
15617 | 15617 | | |
15618 | 15618 | | |
15619 | | - | |
| 15619 | + | |
| 15620 | + | |
15620 | 15621 | | |
15621 | 15622 | | |
15622 | 15623 | | |
| |||
17667 | 17668 | | |
17668 | 17669 | | |
17669 | 17670 | | |
17670 | | - | |
17671 | | - | |
17672 | | - | |
17673 | | - | |
17674 | | - | |
| 17671 | + | |
| 17672 | + | |
17675 | 17673 | | |
17676 | 17674 | | |
17677 | 17675 | | |
| |||
26960 | 26958 | | |
26961 | 26959 | | |
26962 | 26960 | | |
26963 | | - | |
| 26961 | + | |
26964 | 26962 | | |
26965 | 26963 | | |
26966 | 26964 | | |
26967 | | - | |
| 26965 | + | |
26968 | 26966 | | |
26969 | 26967 | | |
26970 | 26968 | | |
| |||
27001 | 26999 | | |
27002 | 27000 | | |
27003 | 27001 | | |
27004 | | - | |
| 27002 | + | |
27005 | 27003 | | |
27006 | 27004 | | |
27007 | | - | |
| 27005 | + | |
27008 | 27006 | | |
27009 | 27007 | | |
27010 | 27008 | | |
| |||
27142 | 27140 | | |
27143 | 27141 | | |
27144 | 27142 | | |
27145 | | - | |
| 27143 | + | |
27146 | 27144 | | |
Lines changed: 10 additions & 12 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 | | |
| |||
11350 | 11350 | | |
11351 | 11351 | | |
11352 | 11352 | | |
11353 | | - | |
| 11353 | + | |
| 11354 | + | |
11354 | 11355 | | |
11355 | 11356 | | |
11356 | 11357 | | |
| |||
12828 | 12829 | | |
12829 | 12830 | | |
12830 | 12831 | | |
12831 | | - | |
12832 | | - | |
12833 | | - | |
12834 | | - | |
12835 | | - | |
| 12832 | + | |
| 12833 | + | |
12836 | 12834 | | |
12837 | 12835 | | |
12838 | 12836 | | |
| |||
17051 | 17049 | | |
17052 | 17050 | | |
17053 | 17051 | | |
17054 | | - | |
| 17052 | + | |
17055 | 17053 | | |
17056 | 17054 | | |
17057 | 17055 | | |
17058 | 17056 | | |
17059 | 17057 | | |
17060 | 17058 | | |
17061 | | - | |
| 17059 | + | |
17062 | 17060 | | |
17063 | 17061 | | |
17064 | 17062 | | |
| |||
17080 | 17078 | | |
17081 | 17079 | | |
17082 | 17080 | | |
17083 | | - | |
| 17081 | + | |
17084 | 17082 | | |
17085 | 17083 | | |
17086 | | - | |
| 17084 | + | |
17087 | 17085 | | |
17088 | 17086 | | |
17089 | 17087 | | |
| |||
17181 | 17179 | | |
17182 | 17180 | | |
17183 | 17181 | | |
17184 | | - | |
| 17182 | + | |
Lines changed: 10 additions & 12 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 | | |
| |||
11911 | 11911 | | |
11912 | 11912 | | |
11913 | 11913 | | |
11914 | | - | |
| 11914 | + | |
| 11915 | + | |
11915 | 11916 | | |
11916 | 11917 | | |
11917 | 11918 | | |
| |||
13530 | 13531 | | |
13531 | 13532 | | |
13532 | 13533 | | |
13533 | | - | |
13534 | | - | |
13535 | | - | |
13536 | | - | |
13537 | | - | |
| 13534 | + | |
| 13535 | + | |
13538 | 13536 | | |
13539 | 13537 | | |
13540 | 13538 | | |
| |||
17761 | 17759 | | |
17762 | 17760 | | |
17763 | 17761 | | |
17764 | | - | |
| 17762 | + | |
17765 | 17763 | | |
17766 | 17764 | | |
17767 | 17765 | | |
17768 | 17766 | | |
17769 | 17767 | | |
17770 | 17768 | | |
17771 | | - | |
| 17769 | + | |
17772 | 17770 | | |
17773 | 17771 | | |
17774 | 17772 | | |
| |||
17790 | 17788 | | |
17791 | 17789 | | |
17792 | 17790 | | |
17793 | | - | |
| 17791 | + | |
17794 | 17792 | | |
17795 | 17793 | | |
17796 | | - | |
| 17794 | + | |
17797 | 17795 | | |
17798 | 17796 | | |
17799 | 17797 | | |
| |||
17906 | 17904 | | |
17907 | 17905 | | |
17908 | 17906 | | |
17909 | | - | |
| 17907 | + | |
Lines changed: 10 additions & 12 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 | | |
| |||
15624 | 15624 | | |
15625 | 15625 | | |
15626 | 15626 | | |
15627 | | - | |
| 15627 | + | |
| 15628 | + | |
15628 | 15629 | | |
15629 | 15630 | | |
15630 | 15631 | | |
| |||
17675 | 17676 | | |
17676 | 17677 | | |
17677 | 17678 | | |
17678 | | - | |
17679 | | - | |
17680 | | - | |
17681 | | - | |
17682 | | - | |
| 17679 | + | |
| 17680 | + | |
17683 | 17681 | | |
17684 | 17682 | | |
17685 | 17683 | | |
| |||
27016 | 27014 | | |
27017 | 27015 | | |
27018 | 27016 | | |
27019 | | - | |
| 27017 | + | |
27020 | 27018 | | |
27021 | 27019 | | |
27022 | 27020 | | |
27023 | | - | |
| 27021 | + | |
27024 | 27022 | | |
27025 | 27023 | | |
27026 | 27024 | | |
| |||
27057 | 27055 | | |
27058 | 27056 | | |
27059 | 27057 | | |
27060 | | - | |
| 27058 | + | |
27061 | 27059 | | |
27062 | 27060 | | |
27063 | | - | |
| 27061 | + | |
27064 | 27062 | | |
27065 | 27063 | | |
27066 | 27064 | | |
| |||
27514 | 27512 | | |
27515 | 27513 | | |
27516 | 27514 | | |
27517 | | - | |
| 27515 | + | |
27518 | 27516 | | |
27519 | 27517 | | |
27520 | 27518 | | |
| |||
Lines changed: 10 additions & 12 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 | | |
| |||
11350 | 11350 | | |
11351 | 11351 | | |
11352 | 11352 | | |
11353 | | - | |
| 11353 | + | |
| 11354 | + | |
11354 | 11355 | | |
11355 | 11356 | | |
11356 | 11357 | | |
| |||
12828 | 12829 | | |
12829 | 12830 | | |
12830 | 12831 | | |
12831 | | - | |
12832 | | - | |
12833 | | - | |
12834 | | - | |
12835 | | - | |
| 12832 | + | |
| 12833 | + | |
12836 | 12834 | | |
12837 | 12835 | | |
12838 | 12836 | | |
| |||
17062 | 17060 | | |
17063 | 17061 | | |
17064 | 17062 | | |
17065 | | - | |
| 17063 | + | |
17066 | 17064 | | |
17067 | 17065 | | |
17068 | 17066 | | |
17069 | 17067 | | |
17070 | 17068 | | |
17071 | 17069 | | |
17072 | | - | |
| 17070 | + | |
17073 | 17071 | | |
17074 | 17072 | | |
17075 | 17073 | | |
| |||
17091 | 17089 | | |
17092 | 17090 | | |
17093 | 17091 | | |
17094 | | - | |
| 17092 | + | |
17095 | 17093 | | |
17096 | 17094 | | |
17097 | | - | |
| 17095 | + | |
17098 | 17096 | | |
17099 | 17097 | | |
17100 | 17098 | | |
| |||
17345 | 17343 | | |
17346 | 17344 | | |
17347 | 17345 | | |
17348 | | - | |
| 17346 | + | |
0 commit comments