Manual touch ups

A few manual changes I made after the codemod ran.

Author: acdlite

packages/react-art/src/ReactARTHostConfig.js

@@ -248,8 +248,6 @@ export function appendInitialChild(parentInstance, child) {
   if (typeof child === 'string') {
     // Noop for string children of Text (eg <Text>{'foo'}{'bar'}</Text>)
     throw new Error('Text children should already be flattened.');
-
-    return;
   }
 
   child.inject(parentInstance);

packages/react-dom/src/client/ReactDOMTextarea.js

@@ -104,8 +104,6 @@ export function initWrapperState(element: Element, props: Object) {
         }
 
         if (isArray(children)) {
-          // FIXME: Review this condition before merging
-          // Should be equivalent to: !(children.length <= 1)
           if (children.length > 1) {
             throw new Error('<textarea> can only have at most one child.');
           }

packages/react-dom/src/server/ReactDOMServerFormatConfig.js

@@ -896,8 +896,6 @@ function pushStartTextArea(
     }
 
     if (isArray(children)) {
-      // FIXME: Review this condition before merging
-      // Should be equivalent to: !(children.length <= 1)
       if (children.length > 1) {
         throw new Error('<textarea> can only have at most one child.');
       }

packages/react-dom/src/server/ReactPartialRenderer.js

@@ -976,8 +976,6 @@ class ReactDOMServerRenderer {
         } catch (err) {
           if (err != null && typeof err.then === 'function') {
             if (enableSuspenseServerRenderer) {
-              // FIXME: Review this condition before merging
-              // Should be equivalent to: !(this.suspenseDepth > 0)
               if (this.suspenseDepth <= 0) {
                 throw new Error(
                   // TODO: include component name. This is a bit tricky with current factoring.
@@ -1477,8 +1475,6 @@ class ReactDOMServerRenderer {
           }
 
           if (isArray(textareaChildren)) {
-            // FIXME: Review this condition before merging
-            // Should be equivalent to: !(textareaChildren.length <= 1)
             if (textareaChildren.length > 1) {
               throw new Error('<textarea> can only have at most one child.');
             }

packages/react-dom/src/server/ReactPartialRendererHooks.js

@@ -424,8 +424,6 @@ function dispatchAction<A>(
   queue: UpdateQueue<A>,
   action: A,
 ) {
-  // FIXME: Review this condition before merging
-  // Should be equivalent to: !(numberOfReRenders < RE_RENDER_LIMIT)
   if (numberOfReRenders >= RE_RENDER_LIMIT) {
     throw new Error(
       'Too many re-renders. React limits the number of renders to prevent ' +

packages/react-dom/src/server/ReactThreadIDAllocator.js

@@ -24,8 +24,6 @@ function growThreadCountAndReturnNextAvailable() {
   const oldSize = oldArray.length;
   const newSize = oldSize * 2;
 
-  // FIXME: Review this condition before merging
-  // Should be equivalent to: !(newSize <= 0x10000)
   if (newSize > 0x10000) {
     throw new Error(
       'Maximum number of concurrent React renderers exceeded. ' +

packages/react-native-renderer/src/ReactNativeBridgeEventPlugin.js

@@ -150,6 +150,7 @@ const ReactNativeBridgeEventPlugin = {
 
     if (!bubbleDispatchConfig && !directDispatchConfig) {
       throw new Error(
+        // $FlowFixMe - Flow doesn't like this string coercion because DOMTopLevelEventType is opaque
         `Unsupported top level event type "${topLevelType}" dispatched`,
       );
     }

packages/react-native-renderer/src/__mocks__/react-native/Libraries/ReactPrivate/UIManager.js

@@ -40,8 +40,6 @@ function insertSubviewAtIndex(parent, child, index) {
     );
   }
 
-  // FIXME: Review this condition before merging
-  // Should be equivalent to: !(0 <= index && index <= parentInfo.children.length)
   if (0 > index || index > parentInfo.children.length) {
     throw new Error(
       `Invalid index ${index} for children ${parentInfo.children}`,
@@ -57,8 +55,6 @@ function removeChild(parent, child) {
   const childInfo = views.get(child);
   const index = parentInfo.children.indexOf(child);
 
-  // FIXME: Review this condition before merging
-  // Should be equivalent to: !(index >= 0)
   if (index < 0) {
     throw new Error(`Missing view ${child} during removal`);
   }

packages/react-native-renderer/src/legacy-events/EventPluginRegistry.js

@@ -44,8 +44,6 @@ function recomputePluginOrdering(): void {
     const pluginModule = namesToPlugins[pluginName];
     const pluginIndex = eventPluginOrder.indexOf(pluginName);
 
-    // FIXME: Review this condition before merging
-    // Should be equivalent to: !(pluginIndex > -1)
     if (pluginIndex <= -1) {
       throw new Error(
         'EventPluginRegistry: Cannot inject event plugins that do not exist in ' +

packages/react-reconciler/src/ReactChildFiber.new.js

@@ -169,6 +169,8 @@ function coerceRef(
             'bug in React. Please file an issue.',
         );
       }
+      // Assigning this to a const so Flow knows it won't change in the closure
+      const resolvedInst = inst;
 
       const stringRef = '' + mixedRef;
       // Check if previous string ref matches new string ref
@@ -181,10 +183,10 @@ function coerceRef(
         return current.ref;
       }
       const ref = function(value) {
-        let refs = inst.refs;
+        let refs = resolvedInst.refs;
         if (refs === emptyRefsObject) {
           // This is a lazy pooled frozen object, so we need to initialize.
-          refs = inst.refs = {};
+          refs = resolvedInst.refs = {};
         }
         if (value === null) {
           delete refs[stringRef];