Commit be952bb
committed
Handle fabric root level fragment with compareDocumentPosition (#34533)
The root instance doesn't have a canonical property so we were not
returning a public instance that we can call compareDocumentPosition on
when a Fragment had no other host parent in Fabric. In this case we need
to get the ReactNativeElement from the ReactNativeDocument.
I've also added test coverage for this case in DOM for consistency,
though it was already working there because we use DOM elements as root.
This same test will be copied to RN using Fantom.
DiffTrain build for [83c88ad](83c88ad)1 parent cc7cac5 commit be952bb
File tree
24 files changed
+170
-185
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
+170
-185
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: 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 | | |
| |||
29923 | 29923 | | |
29924 | 29924 | | |
29925 | 29925 | | |
29926 | | - | |
| 29926 | + | |
29927 | 29927 | | |
29928 | 29928 | | |
29929 | 29929 | | |
29930 | | - | |
| 29930 | + | |
29931 | 29931 | | |
29932 | 29932 | | |
29933 | 29933 | | |
| |||
29964 | 29964 | | |
29965 | 29965 | | |
29966 | 29966 | | |
29967 | | - | |
| 29967 | + | |
29968 | 29968 | | |
29969 | 29969 | | |
29970 | | - | |
| 29970 | + | |
29971 | 29971 | | |
29972 | 29972 | | |
29973 | 29973 | | |
| |||
30116 | 30116 | | |
30117 | 30117 | | |
30118 | 30118 | | |
30119 | | - | |
| 30119 | + | |
30120 | 30120 | | |
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 | | |
| |||
17529 | 17529 | | |
17530 | 17530 | | |
17531 | 17531 | | |
17532 | | - | |
| 17532 | + | |
17533 | 17533 | | |
17534 | 17534 | | |
17535 | 17535 | | |
17536 | 17536 | | |
17537 | 17537 | | |
17538 | 17538 | | |
17539 | | - | |
| 17539 | + | |
17540 | 17540 | | |
17541 | 17541 | | |
17542 | 17542 | | |
| |||
17558 | 17558 | | |
17559 | 17559 | | |
17560 | 17560 | | |
17561 | | - | |
| 17561 | + | |
17562 | 17562 | | |
17563 | 17563 | | |
17564 | | - | |
| 17564 | + | |
17565 | 17565 | | |
17566 | 17566 | | |
17567 | 17567 | | |
| |||
17668 | 17668 | | |
17669 | 17669 | | |
17670 | 17670 | | |
17671 | | - | |
| 17671 | + | |
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 | | |
| |||
19624 | 19624 | | |
19625 | 19625 | | |
19626 | 19626 | | |
19627 | | - | |
| 19627 | + | |
19628 | 19628 | | |
19629 | 19629 | | |
19630 | 19630 | | |
19631 | 19631 | | |
19632 | 19632 | | |
19633 | 19633 | | |
19634 | | - | |
| 19634 | + | |
19635 | 19635 | | |
19636 | 19636 | | |
19637 | 19637 | | |
| |||
19653 | 19653 | | |
19654 | 19654 | | |
19655 | 19655 | | |
19656 | | - | |
| 19656 | + | |
19657 | 19657 | | |
19658 | 19658 | | |
19659 | | - | |
| 19659 | + | |
19660 | 19660 | | |
19661 | 19661 | | |
19662 | 19662 | | |
| |||
19779 | 19779 | | |
19780 | 19780 | | |
19781 | 19781 | | |
19782 | | - | |
| 19782 | + | |
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 | | |
| |||
29979 | 29979 | | |
29980 | 29980 | | |
29981 | 29981 | | |
29982 | | - | |
| 29982 | + | |
29983 | 29983 | | |
29984 | 29984 | | |
29985 | 29985 | | |
29986 | | - | |
| 29986 | + | |
29987 | 29987 | | |
29988 | 29988 | | |
29989 | 29989 | | |
| |||
30020 | 30020 | | |
30021 | 30021 | | |
30022 | 30022 | | |
30023 | | - | |
| 30023 | + | |
30024 | 30024 | | |
30025 | 30025 | | |
30026 | | - | |
| 30026 | + | |
30027 | 30027 | | |
30028 | 30028 | | |
30029 | 30029 | | |
| |||
30488 | 30488 | | |
30489 | 30489 | | |
30490 | 30490 | | |
30491 | | - | |
| 30491 | + | |
30492 | 30492 | | |
30493 | 30493 | | |
30494 | 30494 | | |
| |||
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 | | |
| |||
17540 | 17540 | | |
17541 | 17541 | | |
17542 | 17542 | | |
17543 | | - | |
| 17543 | + | |
17544 | 17544 | | |
17545 | 17545 | | |
17546 | 17546 | | |
17547 | 17547 | | |
17548 | 17548 | | |
17549 | 17549 | | |
17550 | | - | |
| 17550 | + | |
17551 | 17551 | | |
17552 | 17552 | | |
17553 | 17553 | | |
| |||
17569 | 17569 | | |
17570 | 17570 | | |
17571 | 17571 | | |
17572 | | - | |
| 17572 | + | |
17573 | 17573 | | |
17574 | 17574 | | |
17575 | | - | |
| 17575 | + | |
17576 | 17576 | | |
17577 | 17577 | | |
17578 | 17578 | | |
| |||
17832 | 17832 | | |
17833 | 17833 | | |
17834 | 17834 | | |
17835 | | - | |
| 17835 | + | |
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 | | |
| |||
19639 | 19639 | | |
19640 | 19640 | | |
19641 | 19641 | | |
19642 | | - | |
| 19642 | + | |
19643 | 19643 | | |
19644 | 19644 | | |
19645 | 19645 | | |
19646 | 19646 | | |
19647 | 19647 | | |
19648 | 19648 | | |
19649 | | - | |
| 19649 | + | |
19650 | 19650 | | |
19651 | 19651 | | |
19652 | 19652 | | |
| |||
19668 | 19668 | | |
19669 | 19669 | | |
19670 | 19670 | | |
19671 | | - | |
| 19671 | + | |
19672 | 19672 | | |
19673 | 19673 | | |
19674 | | - | |
| 19674 | + | |
19675 | 19675 | | |
19676 | 19676 | | |
19677 | 19677 | | |
| |||
19947 | 19947 | | |
19948 | 19948 | | |
19949 | 19949 | | |
19950 | | - | |
| 19950 | + | |
19951 | 19951 | | |
19952 | 19952 | | |
19953 | 19953 | | |
| |||
0 commit comments