Address feedback

Author: trueadm

packages/react-debug-tools/src/ReactDebugHooks.js

@@ -262,8 +262,7 @@ function useEvent(event: any): any {
   hookLog.push({primitive: 'Event', stackError: new Error(), value: event});
   return {
     clear: noOp,
-    listen: noOp,
-    unlisten: noOp,
+    setListener: noOp,
   };
 }
 

packages/react-dom/src/events/__tests__/DOMModernPluginEventSystem-test.internal.js

@@ -1067,7 +1067,7 @@ describe('DOMModernPluginEventSystem', () => {
       ReactDOM.render(<Test />, container);
       ReactDOM.render(<Test />, container);
       expect(listenerMaps.length).toEqual(2);
-      expect(listenerMaps[1]).toEqual(listenerMaps[1]);
+      expect(listenerMaps[0]).toEqual(listenerMaps[1]);
     });
   });
 });

packages/react-reconciler/src/ReactFiberHooks.js

@@ -1394,7 +1394,9 @@ function mountEventListener(event: ReactListenerEvent): ReactListenerMap {
       },
     };
     // In order to clear up upon the hook unmounting,
-    /// we ensure we push an effect that handles the use-case.
+    // we ensure we set the effecrt tag so that we visit
+    // this effect in the commit phase, so we can handle
+    // clean-up accordingly.
     currentlyRenderingFiber.effectTag |= UpdateEffect;
     pushEffect(NoHookEffect, noOpMount, clear, null);
     hook.memoizedState = [reactListenerMap, event, clear];
@@ -1435,7 +1437,9 @@ function updateEventListener(event: ReactListenerEvent): ReactListenerMap {
       }
     }
     // In order to clear up upon the hook unmounting,
-    /// we ensure we push an effect that handles the use-case.
+    // we ensure we set the effecrt tag so that we visit
+    // this effect in the commit phase, so we can handle
+    // clean-up accordingly.
     currentlyRenderingFiber.effectTag |= UpdateEffect;
     pushEffect(NoHookEffect, noOpMount, clear, null);
     return reactListenerMap;