Flow: complete types first migration (#25389)

This complete the "types first" migration and enables the config everywhere.

Author: kassens

packages/react-devtools-shared/src/node_modules/react-window/src/FixedSizeGrid.js

@@ -59,7 +59,7 @@ function isControlled(props) {
  * See http://www.w3.org/TR/2012/WD-html5-20121025/the-input-element.html
  */
 
-export function getHostProps(element: Element, props: Object) {
+export function getHostProps(element: Element, props: Object): Object {
   const node = ((element: any): InputWithWrapperState);
   const checked = props.checked;
 

packages/react-devtools-shared/src/node_modules/react-window/src/FixedSizeList.js

@@ -134,7 +134,7 @@ function updateOptions(
  * selected.
  */
 
-export function getHostProps(element: Element, props: Object) {
+export function getHostProps(element: Element, props: Object): Object {
   return assign({}, props, {
     value: undefined,
   });

packages/react-devtools-shared/src/node_modules/react-window/src/VariableSizeGrid.js

@@ -37,7 +37,7 @@ type TextAreaWithWrapperState = HTMLTextAreaElement & {
  * `defaultValue` if specified, or the children content (deprecated).
  */
 
-export function getHostProps(element: Element, props: Object) {
+export function getHostProps(element: Element, props: Object): Object {
   const node = ((element: any): TextAreaWithWrapperState);
 
   if (props.dangerouslySetInnerHTML != null) {

packages/react-devtools-shared/src/node_modules/react-window/src/VariableSizeList.js

@@ -21,7 +21,10 @@ let reusableSVGContainer;
  * @param {string} html
  * @internal
  */
-const setInnerHTML = createMicrosoftUnsafeLocalFunction(function(
+const setInnerHTML: (
+  node: Element,
+  html: {valueOf(): {toString(): string, ...}, ...},
+) => void = createMicrosoftUnsafeLocalFunction(function(
   node: Element,
   html: {valueOf(): {toString(): string, ...}, ...},
 ): void {

packages/react-dom-bindings/src/client/ReactDOMInput.js

@@ -21,15 +21,19 @@ if (enableCreateEventHandleAPI) {
 /**
  * Mapping from registration name to event name
  */
-export const registrationNameDependencies = {};
+export const registrationNameDependencies: {
+  [registrationName: string]: Array<DOMEventName>,
+} = {};
 
 /**
  * Mapping from lowercase registration names to the properly cased version,
  * used to warn in the case of missing event handlers. Available
  * only in __DEV__.
  * @type {Object}
  */
-export const possibleRegistrationNames = __DEV__ ? {} : (null: any);
+export const possibleRegistrationNames: {
+  [lowerCasedName: string]: string,
+} = __DEV__ ? {} : (null: any);
 // Trust the developer to only use possibleRegistrationNames in __DEV__
 
 export function registerTwoPhaseEvent(

packages/react-dom-bindings/src/client/ReactDOMSelect.js

@@ -8,7 +8,7 @@
  */
 
 import type {AnyNativeEvent} from '../events/PluginModuleType';
-import type {FiberRoot} from 'react-reconciler/src/ReactInternalTypes';
+import type {Fiber, FiberRoot} from 'react-reconciler/src/ReactInternalTypes';
 import type {Container, SuspenseInstance} from '../client/ReactDOMHostConfig';
 import type {DOMEventName} from '../events/DOMEventNames';
 import {enableCapturePhaseSelectiveHydrationWithoutDiscreteEventReplay} from 'shared/ReactFeatureFlags';
@@ -58,15 +58,15 @@ import {isRootDehydrated} from 'react-reconciler/src/ReactFiberShellHydration';
 const {ReactCurrentBatchConfig} = ReactSharedInternals;
 
 // TODO: can we stop exporting these?
-export let _enabled = true;
+export let _enabled: boolean = true;
 
 // This is exported in FB builds for use by legacy FB layer infra.
 // We'd like to remove this but it's not clear if this is safe.
-export function setEnabled(enabled: ?boolean) {
+export function setEnabled(enabled: ?boolean): void {
   _enabled = !!enabled;
 }
 
-export function isEnabled() {
+export function isEnabled(): boolean {
   return _enabled;
 }
 
@@ -348,7 +348,7 @@ function dispatchEventWithEnableCapturePhaseSelectiveHydrationWithoutDiscreteEve
   );
 }
 
-export let return_targetInst = null;
+export let return_targetInst: null | Fiber = null;
 
 // Returns a SuspenseInstance or Container if it's blocked.
 // The return_targetInst field above is conceptually part of the return value.