Skip to content

Commit

Permalink
Remove force commit phase
Browse files Browse the repository at this point in the history
This isn't necessary because we always call "updateWrapper" after the event handling.

We can't rely on even getting here if no props have changed since we could've bailed on the props object alone before getting here.
  • Loading branch information
sebmarkbage committed Mar 29, 2023
1 parent e4aeee8 commit 9c3ef63
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions packages/react-dom-bindings/src/client/ReactDOMComponent.js
Original file line number Diff line number Diff line change
Expand Up @@ -802,17 +802,6 @@ export function diffProperties(

let updatePayload: null | Array<any> = null;

switch (tag) {
case 'input':
case 'select':
case 'textarea':
// Schedule to ensure the commit phase happens
// TODO: Figure out if this is actually really necessary or if all scenarios
// will have changed props.
updatePayload = [];
break;
}

let propKey;
let styleName;
let styleUpdates = null;
Expand Down

0 comments on commit 9c3ef63

Please sign in to comment.