Refined -[RCTUIManager createView:]

Summary: Now we do not add newly created view to the registry at the preluminary step.

Reviewed By: mmmulani

Differential Revision: D6641403

fbshipit-source-id: c69077aaba871f3cdb3500c75e1efe07546e1b7f

Author: shergin

React/Modules/RCTUIManager.m

@@ -967,21 +967,19 @@ - (void)_manageChildren:(NSNumber *)containerTag
 
   // Dispatch view creation directly to the main thread instead of adding to
   // UIBlocks array. This way, it doesn't get deferred until after layout.
-  __weak RCTUIManager *weakManager = self;
+  __block UIView *preliminaryCreatedView;
+
   RCTExecuteOnMainQueue(^{
-    RCTUIManager *uiManager = weakManager;
-    if (!uiManager) {
+    preliminaryCreatedView = [componentData createViewWithTag:reactTag];
+  });
+
+  [self addUIBlock:^(RCTUIManager *uiManager, NSDictionary<NSNumber *, UIView *> *viewRegistry) {
+    if (!preliminaryCreatedView) {
       return;
     }
-    UIView *view = [componentData createViewWithTag:reactTag];
-    if (view) {
-      uiManager->_viewRegistry[reactTag] = view;
-    }
-  });
 
-  [self addUIBlock:^(__unused RCTUIManager *uiManager, NSDictionary<NSNumber *, UIView *> *viewRegistry) {
-    UIView *view = viewRegistry[reactTag];
-    [componentData setProps:props forView:view];
+    uiManager->_viewRegistry[reactTag] = preliminaryCreatedView;
+    [componentData setProps:props forView:preliminaryCreatedView];
   }];
 
   [self _shadowView:shadowView didReceiveUpdatedProps:[props allKeys]];