Skip to content

Commit e88db99

Browse files
sherginfacebook-github-bot
authored andcommitted
Fabric: Removing deprecated leftovers from RCTScheduler and RCTSurfacePresenter
Summary: Trivial. We don't use it anymore. Reviewed By: mdvacca Differential Revision: D12876743 fbshipit-source-id: dc979aaea1fef443b8caf2e58d44b0c7aad90246
1 parent df4521e commit e88db99

File tree

4 files changed

+0
-34
lines changed

4 files changed

+0
-34
lines changed

React/Fabric/RCTScheduler.h

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
#import <React/RCTPrimitives.h>
1212
#import <fabric/core/LayoutConstraints.h>
1313
#import <fabric/core/LayoutContext.h>
14-
#import <fabric/uimanager/FabricUIManager.h>
1514
#import <fabric/uimanager/ShadowViewMutation.h>
1615

1716
NS_ASSUME_NONNULL_BEGIN
@@ -57,10 +56,4 @@ NS_ASSUME_NONNULL_BEGIN
5756

5857
@end
5958

60-
@interface RCTScheduler (Deprecated)
61-
62-
- (std::shared_ptr<facebook::react::FabricUIManager>)uiManager_DO_NOT_USE;
63-
64-
@end
65-
6659
NS_ASSUME_NONNULL_END

React/Fabric/RCTScheduler.mm

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -97,12 +97,3 @@ - (void)constraintSurfaceLayoutWithLayoutConstraints:(LayoutConstraints)layoutCo
9797
}
9898

9999
@end
100-
101-
@implementation RCTScheduler (Deprecated)
102-
103-
- (std::shared_ptr<FabricUIManager>)uiManager_DO_NOT_USE
104-
{
105-
return _scheduler->getUIManager_DO_NOT_USE();
106-
}
107-
108-
@end

React/Fabric/RCTSurfacePresenter.h

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010

1111
#import <React/RCTBridge.h>
1212
#import <React/RCTPrimitives.h>
13-
#import <fabric/uimanager/FabricUIManager.h>
1413

1514
NS_ASSUME_NONNULL_BEGIN
1615

@@ -27,12 +26,6 @@ NS_ASSUME_NONNULL_BEGIN
2726

2827
- (instancetype)initWithBridge:(RCTBridge *)bridge;
2928

30-
/*
31-
* Deprecated. Do not use.
32-
*/
33-
@property (nonatomic) std::function<facebook::react::UIManagerInstaller> uiManagerInstaller;
34-
@property (nonatomic) std::function<facebook::react::UIManagerUninstaller> uiManagerUninstaller;
35-
3629
@end
3730

3831
@interface RCTSurfacePresenter (Surface)
@@ -66,12 +59,6 @@ NS_ASSUME_NONNULL_BEGIN
6659

6760
@interface RCTSurfacePresenter (Deprecated)
6861

69-
/**
70-
* We need to expose `uiManager` for registration
71-
* purposes. Eventually, we will move this down to C++ side.
72-
*/
73-
- (std::shared_ptr<facebook::react::FabricUIManager>)uiManager_DO_NOT_USE;
74-
7562
/**
7663
* Returns a underlying bridge.
7764
*/

React/Fabric/RCTSurfacePresenter.mm

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -306,11 +306,6 @@ - (void)handleJavaScriptDidLoadNotification:(NSNotification *)notification
306306

307307
@implementation RCTSurfacePresenter (Deprecated)
308308

309-
- (std::shared_ptr<FabricUIManager>)uiManager_DO_NOT_USE
310-
{
311-
return _scheduler.uiManager_DO_NOT_USE;
312-
}
313-
314309
- (RCTBridge *)bridge_DO_NOT_USE
315310
{
316311
return _bridge;

0 commit comments

Comments
 (0)