Commit 9ea059c
authored
## Summary:
We've had a recurring bug where `RCTDevLoadingView` stick around
blocking the whole app even after it's dismissed. As far as I can tell,
this is a race condition where we have multiple devloadingviews at once
(each setting and nilling the same `self->_window` property), which is
causing a race condition. Rather than figure out how to make this class
only ever show one window or race-safe, for now I will just make sure to
dismiss _all_ sheets when `hide` is called.
While we're at it, a couple of other changes:
1. Change the type from `NSPanel` to `NSWindow`. Panels don't release
when closed, which may be causing them to stick around longer than we
like
2. Move the iOS only variable `mainWindow` into the iOS ifdef, and
remove the diffs and diff tag around it.
## Test Plan:
Initial testing shows RNTester no longer getting blocked. It could still
happen (race conditions are hard), but I think the change is safe to add
regardless.
1 parent 1129083 commit 9ea059c
File tree
1 file changed
+11
-4
lines changed- packages/react-native/React/CoreModules
1 file changed
+11
-4
lines changedLines changed: 11 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| 27 | + | |
| 28 | + | |
27 | 29 | | |
28 | 30 | | |
29 | 31 | | |
| |||
121 | 123 | | |
122 | 124 | | |
123 | 125 | | |
124 | | - | |
125 | 126 | | |
| 127 | + | |
126 | 128 | | |
127 | 129 | | |
128 | 130 | | |
129 | 131 | | |
130 | | - | |
| 132 | + | |
131 | 133 | | |
132 | 134 | | |
133 | 135 | | |
| 136 | + | |
134 | 137 | | |
135 | 138 | | |
136 | 139 | | |
| |||
220 | 223 | | |
221 | 224 | | |
222 | 225 | | |
223 | | - | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
224 | 231 | | |
225 | 232 | | |
226 | 233 | | |
| |||
350 | 357 | | |
351 | 358 | | |
352 | 359 | | |
353 | | - | |
| 360 | + | |
0 commit comments