diff --git a/scripts/fiber/tests-passing-except-dev.txt b/scripts/fiber/tests-passing-except-dev.txt index 5b7ce16d92..3ff930bd86 100644 --- a/scripts/fiber/tests-passing-except-dev.txt +++ b/scripts/fiber/tests-passing-except-dev.txt @@ -1,5 +1,4 @@ src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js -* should not warn when server-side rendering `onScroll` * should warn about incorrect casing on properties (ssr) * should warn about incorrect casing on event handlers (ssr) * should warn about class (ssr) diff --git a/scripts/fiber/tests-passing.txt b/scripts/fiber/tests-passing.txt index 951646da67..be1d9751e1 100644 --- a/scripts/fiber/tests-passing.txt +++ b/scripts/fiber/tests-passing.txt @@ -951,7 +951,6 @@ src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js * should report component containing invalid styles * should properly escape text content and attributes values * unmounts children before unsetting DOM node info -* should warn about the `onScroll` issue when unsupported (IE8) * should throw when an invalid tag name is used server-side * should throw when an attack vector is used server-side * should throw when an invalid tag name is used diff --git a/src/renderers/dom/fiber/ReactDOMFiberComponent.js b/src/renderers/dom/fiber/ReactDOMFiberComponent.js index 89df93b868..bea2bf0ca5 100644 --- a/src/renderers/dom/fiber/ReactDOMFiberComponent.js +++ b/src/renderers/dom/fiber/ReactDOMFiberComponent.js @@ -28,7 +28,6 @@ var {getCurrentFiberOwnerName} = require('ReactDebugCurrentFiber'); var emptyFunction = require('fbjs/lib/emptyFunction'); var invariant = require('fbjs/lib/invariant'); -var isEventSupported = require('isEventSupported'); var setInnerHTML = require('setInnerHTML'); var setTextContent = require('setTextContent'); var inputValueTracking = require('inputValueTracking'); @@ -145,14 +144,6 @@ if (__DEV__) { } function ensureListeningTo(rootContainerElement, registrationName) { - if (__DEV__) { - // IE8 has no API for event capturing and the `onScroll` event doesn't - // bubble. - warning( - registrationName !== 'onScroll' || isEventSupported('scroll', true), - "This browser doesn't support the `onScroll` event", - ); - } var isDocumentFragment = rootContainerElement.nodeType === DOC_FRAGMENT_TYPE; var doc = isDocumentFragment ? rootContainerElement diff --git a/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js b/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js index 9a5adddf9a..7a8c73ddc7 100644 --- a/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js +++ b/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js @@ -1233,30 +1233,6 @@ describe('ReactDOMComponent', () => { }); }); - describe('onScroll warning', () => { - it('should warn about the `onScroll` issue when unsupported (IE8)', () => { - // Mock this here so we can mimic IE8 support. We require isEventSupported - // before React so it's pre-mocked before React would require it. - jest.resetModules().mock('isEventSupported'); - var isEventSupported = require('isEventSupported'); - isEventSupported.mockReturnValueOnce(false); - ReactTestUtils = require('ReactTestUtils'); - - spyOn(console, 'error'); - ReactTestUtils.renderIntoDocument(
); - expectDev(console.error.calls.count()).toBe(1); - expectDev(console.error.calls.argsFor(0)[0]).toBe( - "Warning: This browser doesn't support the `onScroll` event", - ); - }); - - it('should not warn when server-side rendering `onScroll`', () => { - spyOn(console, 'error'); - ReactDOMServer.renderToString(