Clean up disableIEWorkarounds flag

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

@@ -50,7 +50,6 @@ import {
 } from './ReactDOMTextarea';
 import {validateTextNesting} from './validateDOMNesting';
 import {track} from './inputValueTracking';
-import setInnerHTML from './setInnerHTML';
 import setTextContent from './setTextContent';
 import {
   createDangerousStringForStyles,
@@ -66,7 +65,6 @@ import {validateProperties as validateUnknownProperties} from '../shared/ReactDO
 import sanitizeURL from '../shared/sanitizeURL';
 
 import {
-  disableIEWorkarounds,
   enableTrustedTypesIntegration,
   enableFilterEmptyStringAttributesDOM,
 } from 'shared/ReactFeatureFlags';
@@ -86,16 +84,7 @@ let didWarnPopoverTargetObject = false;
 let canDiffStyleForHydrationWarning;
 if (__DEV__) {
   didWarnForNewBooleanPropsWithEmptyValue = {};
-  // IE 11 parses & normalizes the style attribute as opposed to other
-  // browsers. It adds spaces and sorts the properties in some
-  // non-alphabetical order. Handling that would require sorting CSS
-  // properties in the client & server versions or applying
-  // `expectedStyle` to a temporary DOM node to read its `style` attribute
-  // normalized. Since it only affects IE, we're skipping style warnings
-  // in that browser completely in favor of doing all that work.
-  // See https://github.com/facebook/react/issues/11807
-  canDiffStyleForHydrationWarning =
-    disableIEWorkarounds || (canUseDOM && !document.documentMode);
+  canDiffStyleForHydrationWarning = canUseDOM && !document.documentMode;
 }
 
 function validatePropertiesInDevelopment(type: string, props: any) {
@@ -579,11 +568,7 @@ function setProp(
               'Can only set one of `children` or `props.dangerouslySetInnerHTML`.',
             );
           }
-          if (disableIEWorkarounds) {
-            domElement.innerHTML = nextHtml;
-          } else {
-            setInnerHTML(domElement, nextHtml);
-          }
+          domElement.innerHTML = nextHtml;
         }
       }
       break;
@@ -939,11 +924,7 @@ function setPropOnCustomElement(
               'Can only set one of `children` or `props.dangerouslySetInnerHTML`.',
             );
           }
-          if (disableIEWorkarounds) {
-            domElement.innerHTML = nextHtml;
-          } else {
-            setInnerHTML(domElement, nextHtml);
-          }
+          domElement.innerHTML = nextHtml;
         }
       }
       break;

packages/react-dom/src/client/__tests__/dangerouslySetInnerHTML-test.js

@@ -58,42 +58,5 @@ describe('dangerouslySetInnerHTML', () => {
         innerHTMLDescriptor,
       );
     });
-
-    // @gate !disableIEWorkarounds
-    it('sets innerHTML on it', async () => {
-      const html = '<circle></circle>';
-      const container = document.createElementNS(
-        'http://www.w3.org/2000/svg',
-        'svg',
-      );
-      const root = ReactDOMClient.createRoot(container);
-      await act(() => {
-        root.render(<g dangerouslySetInnerHTML={{__html: html}} />);
-      });
-      const circle = container.firstChild.firstChild;
-      expect(circle.tagName).toBe('circle');
-    });
-
-    // @gate !disableIEWorkarounds
-    it('clears previous children', async () => {
-      const firstHtml = '<rect></rect>';
-      const secondHtml = '<circle></circle>';
-
-      const container = document.createElementNS(
-        'http://www.w3.org/2000/svg',
-        'svg',
-      );
-      const root = ReactDOMClient.createRoot(container);
-      await act(() => {
-        root.render(<g dangerouslySetInnerHTML={{__html: firstHtml}} />);
-      });
-      const rect = container.firstChild.firstChild;
-      expect(rect.tagName).toBe('rect');
-      await act(() => {
-        root.render(<g dangerouslySetInnerHTML={{__html: secondHtml}} />);
-      });
-      const circle = container.firstChild.firstChild;
-      expect(circle.tagName).toBe('circle');
-    });
   });
 });

packages/react-dom/src/client/__tests__/trustedTypes-test.internal.js

@@ -206,56 +206,6 @@ describe('when Trusted Types are available in global object', () => {
     }
   });
 
-  describe('dangerouslySetInnerHTML in svg elements in Internet Explorer', () => {
-    let innerHTMLDescriptor;
-
-    // simulate svg elements in Internet Explorer which don't have 'innerHTML' property
-    beforeEach(() => {
-      innerHTMLDescriptor = Object.getOwnPropertyDescriptor(
-        Element.prototype,
-        'innerHTML',
-      );
-      delete Element.prototype.innerHTML;
-      Object.defineProperty(
-        HTMLDivElement.prototype,
-        'innerHTML',
-        innerHTMLDescriptor,
-      );
-    });
-
-    afterEach(() => {
-      delete HTMLDivElement.prototype.innerHTML;
-      Object.defineProperty(
-        Element.prototype,
-        'innerHTML',
-        innerHTMLDescriptor,
-      );
-    });
-
-    // @gate !disableIEWorkarounds
-    it('should log a warning', async () => {
-      class Component extends React.Component {
-        render() {
-          return <svg dangerouslySetInnerHTML={{__html: 'unsafe html'}} />;
-        }
-      }
-      const root = ReactDOMClient.createRoot(container);
-      await expect(async () => {
-        await act(() => {
-          root.render(<Component />);
-        });
-      }).toErrorDev(
-        "Using 'dangerouslySetInnerHTML' in an svg element with " +
-          'Trusted Types enabled in an Internet Explorer will cause ' +
-          'the trusted value to be converted to string. Assigning string ' +
-          "to 'innerHTML' will throw an error if Trusted Types are enforced. " +
-          "You can try to wrap your svg element inside a div and use 'dangerouslySetInnerHTML' " +
-          'on the enclosing div instead.',
-      );
-      expect(container.innerHTML).toBe('<svg>unsafe html</svg>');
-    });
-  });
-
   it('should warn once when rendering script tag in jsx on client', async () => {
     const root = ReactDOMClient.createRoot(container);
     await expect(async () => {

packages/shared/ReactFeatureFlags.js

@@ -159,10 +159,6 @@ export const disableLegacyContext = true;
 // TODO: clean up legacy <StrictMode /> once tests pass WWW.
 export const useModernStrictMode = true;
 
-// Not ready to break experimental yet.
-// Remove IE and MsApp specific workarounds for innerHTML
-export const disableIEWorkarounds = true;
-
 // Filter certain DOM attributes (e.g. src, href) if their values are empty
 // strings. This prevents e.g. <img src=""> from making an unnecessary HTTP
 // request for certain browsers.

packages/shared/forks/ReactFeatureFlags.native-fb.js

@@ -35,7 +35,6 @@ export const allowConcurrentByDefault = false;
 export const debugRenderPhaseSideEffectsForStrictMode = __DEV__;
 export const disableClientCache = true;
 export const disableCommentsAsDOMContainers = true;
-export const disableIEWorkarounds = true;
 export const disableInputAttributeSyncing = false;
 export const disableLegacyContext = false;
 export const disableLegacyMode = false;

packages/shared/forks/ReactFeatureFlags.native-oss.js

@@ -25,7 +25,6 @@ export const consoleManagedByDevToolsDuringStrictMode = true;
 export const disableClientCache = true;
 export const disableCommentsAsDOMContainers = true;
 export const disableDefaultPropsExceptForClasses = true;
-export const disableIEWorkarounds = true;
 export const disableInputAttributeSyncing = false;
 export const disableLegacyContext = true;
 export const disableLegacyMode = false;

packages/shared/forks/ReactFeatureFlags.test-renderer.js

@@ -27,7 +27,6 @@ export const enableTaint = true;
 export const enablePostpone = false;
 export const disableCommentsAsDOMContainers = true;
 export const disableInputAttributeSyncing = false;
-export const disableIEWorkarounds = true;
 export const enableScopeAPI = false;
 export const enableCreateEventHandleAPI = false;
 export const enableSuspenseCallback = false;

packages/shared/forks/ReactFeatureFlags.test-renderer.native-fb.js

@@ -17,7 +17,6 @@ export const debugRenderPhaseSideEffectsForStrictMode = false;
 export const disableClientCache = true;
 export const disableCommentsAsDOMContainers = true;
 export const disableDefaultPropsExceptForClasses = false;
-export const disableIEWorkarounds = true;
 export const disableInputAttributeSyncing = false;
 export const disableLegacyContext = false;
 export const disableLegacyMode = false;

packages/shared/forks/ReactFeatureFlags.test-renderer.www.js

@@ -27,7 +27,6 @@ export const enableTaint = true;
 export const enablePostpone = false;
 export const disableCommentsAsDOMContainers = true;
 export const disableInputAttributeSyncing = false;
-export const disableIEWorkarounds = true;
 export const enableScopeAPI = true;
 export const enableCreateEventHandleAPI = false;
 export const enableSuspenseCallback = true;

packages/shared/forks/ReactFeatureFlags.www.js

@@ -50,7 +50,6 @@ export const enableUpdaterTracking = __PROFILE__;
 export const enableSuspenseAvoidThisFallback = true;
 export const enableSuspenseAvoidThisFallbackFizz = false;
 
-export const disableIEWorkarounds = true;
 export const enableCPUSuspense = true;
 export const enableUseMemoCacheHook = true;
 export const enableUseEffectEventHook = true;

scripts/jest/setupTests.www.js

@@ -18,7 +18,6 @@ jest.mock('shared/ReactFeatureFlags', () => {
   // These are hardcoded to true for the next release,
   // but still run the tests against both variants until
   // we remove the flag.
-  actual.disableIEWorkarounds = __VARIANT__;
   actual.disableClientCache = __VARIANT__;
 
   return actual;