From b31923544883c444328f03568fc040e72244e310 Mon Sep 17 00:00:00 2001
From: Nate Hunzaker <nate.hunzaker@gmail.com>
Date: Tue, 11 Sep 2018 07:35:46 -0700
Subject: [PATCH] Do not bind topLevelType to dispatch

A previous change made it such that all top level event types
correspond to their associated native event string values. This commit
eliminates the .bind attached to dispatch and fixes a related flow
type.
---
 packages/events/PluginModuleType.js                |  2 +-
 packages/events/ReactGenericBatching.js            |  8 ++++----
 .../react-dom/src/events/ReactDOMEventListener.js  | 14 +++++++-------
 .../react-dom/src/test-utils/ReactTestUtils.js     |  6 ++++--
 4 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/packages/events/PluginModuleType.js b/packages/events/PluginModuleType.js
index cd7a07661ab4e..0f79f749f5332 100644
--- a/packages/events/PluginModuleType.js
+++ b/packages/events/PluginModuleType.js
@@ -16,7 +16,7 @@ import type {TopLevelType} from './TopLevelEventTypes';
 
 export type EventTypes = {[key: string]: DispatchConfig};
 
-export type AnyNativeEvent = Event | KeyboardEvent | MouseEvent | Touch;
+export type AnyNativeEvent = Event | KeyboardEvent | MouseEvent | TouchEvent;
 
 export type PluginName = string;
 
diff --git a/packages/events/ReactGenericBatching.js b/packages/events/ReactGenericBatching.js
index 8347382f48a53..e37481e4deee8 100644
--- a/packages/events/ReactGenericBatching.js
+++ b/packages/events/ReactGenericBatching.js
@@ -20,8 +20,8 @@ import {
 let _batchedUpdatesImpl = function(fn, bookkeeping) {
   return fn(bookkeeping);
 };
-let _interactiveUpdatesImpl = function(fn, a, b) {
-  return fn(a, b);
+let _interactiveUpdatesImpl = function(fn, a) {
+  return fn(a);
 };
 let _flushInteractiveUpdatesImpl = function() {};
 
@@ -52,8 +52,8 @@ export function batchedUpdates(fn, bookkeeping) {
   }
 }
 
-export function interactiveUpdates(fn, a, b) {
-  return _interactiveUpdatesImpl(fn, a, b);
+export function interactiveUpdates(fn, a) {
+  return _interactiveUpdatesImpl(fn, a);
 }
 
 export function flushInteractiveUpdates() {
diff --git a/packages/react-dom/src/events/ReactDOMEventListener.js b/packages/react-dom/src/events/ReactDOMEventListener.js
index e49266d267f70..7458a08bf8fa0 100644
--- a/packages/react-dom/src/events/ReactDOMEventListener.js
+++ b/packages/react-dom/src/events/ReactDOMEventListener.js
@@ -21,6 +21,7 @@ import getEventTarget from './getEventTarget';
 import {getClosestInstanceFromNode} from '../client/ReactDOMComponentTree';
 import SimpleEventPlugin from './SimpleEventPlugin';
 import {getRawEventName} from './DOMTopLevelEventTypes';
+import {unsafeCastStringToDOMTopLevelType} from 'events/TopLevelEventTypes';
 
 const {isInteractiveTopLevelEventType} = SimpleEventPlugin;
 
@@ -48,7 +49,6 @@ function findRootContainerNode(inst) {
 
 // Used to store ancestor hierarchy in top level callback
 function getTopLevelCallbackBookKeeping(
-  topLevelType,
   nativeEvent,
   targetInst,
 ): {
@@ -57,6 +57,8 @@ function getTopLevelCallbackBookKeeping(
   targetInst: Fiber | null,
   ancestors: Array<Fiber>,
 } {
+  const topLevelType = unsafeCastStringToDOMTopLevelType(nativeEvent.type);
+
   if (callbackBookkeepingPool.length) {
     const instance = callbackBookkeepingPool.pop();
     instance.topLevelType = topLevelType;
@@ -149,7 +151,7 @@ export function trapBubbledEvent(
     element,
     getRawEventName(topLevelType),
     // Check if interactive and wrap in interactiveUpdates
-    dispatch.bind(null, topLevelType),
+    dispatch,
   );
 }
 
@@ -177,16 +179,15 @@ export function trapCapturedEvent(
     element,
     getRawEventName(topLevelType),
     // Check if interactive and wrap in interactiveUpdates
-    dispatch.bind(null, topLevelType),
+    dispatch,
   );
 }
 
-function dispatchInteractiveEvent(topLevelType, nativeEvent) {
-  interactiveUpdates(dispatchEvent, topLevelType, nativeEvent);
+function dispatchInteractiveEvent(nativeEvent) {
+  interactiveUpdates(dispatchEvent, nativeEvent);
 }
 
 export function dispatchEvent(
-  topLevelType: DOMTopLevelEventType,
   nativeEvent: AnyNativeEvent,
 ) {
   if (!_enabled) {
@@ -208,7 +209,6 @@ export function dispatchEvent(
   }
 
   const bookKeeping = getTopLevelCallbackBookKeeping(
-    topLevelType,
     nativeEvent,
     targetInst,
   );
diff --git a/packages/react-dom/src/test-utils/ReactTestUtils.js b/packages/react-dom/src/test-utils/ReactTestUtils.js
index dad3d0f14e2ef..88710e9518dc4 100644
--- a/packages/react-dom/src/test-utils/ReactTestUtils.js
+++ b/packages/react-dom/src/test-utils/ReactTestUtils.js
@@ -42,7 +42,9 @@ const [
   runEventsInBatch,
 ] = ReactDOM.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED.Events;
 
-function Event(suffix) {}
+function Event(type) {
+  this.type = type;
+}
 
 let hasWarnedAboutDeprecatedMockComponent = false;
 
@@ -59,7 +61,7 @@ let hasWarnedAboutDeprecatedMockComponent = false;
  */
 function simulateNativeEventOnNode(topLevelType, node, fakeNativeEvent) {
   fakeNativeEvent.target = node;
-  dispatchEvent(topLevelType, fakeNativeEvent);
+  dispatchEvent(fakeNativeEvent);
 }
 
 /**