Remove captured scroll feature check (#9283)
* Remove captured scroll feature check IE8 was the only browser that did not support captured scroll. We no longer have that constraint. * Remove WINDOW_HANDLE const
Nathan Hunzaker committed
Mar 30, 2017 at 06:01 UTC
4c292facc9fa176cd0a8a094b38fc1800650325f
2 files changed
+5
-16
src/renderers/dom/shared/ReactBrowserEventEmitter.js
+5
-13
@@ -262,19 +262,11 @@ var ReactBrowserEventEmitter = Object.assign({}, ReactEventEmitterMixin, {
262
);
263
}
264
} else if (dependency === 'topScroll') {
265
- if (isEventSupported('scroll', true)) {
266
- ReactBrowserEventEmitter.ReactEventListener.trapCapturedEvent(
267
- 'topScroll',
268
- 'scroll',
269
- mountAt,
270
- );
271
- } else {
272
- ReactBrowserEventEmitter.ReactEventListener.trapBubbledEvent(
273
- 'topScroll',
274
- 'scroll',
275
- ReactBrowserEventEmitter.ReactEventListener.WINDOW_HANDLE,
276
- );
277
- }
265
+ ReactBrowserEventEmitter.ReactEventListener.trapCapturedEvent(
266
+ 'topScroll',
267
+ 'scroll',
268
+ mountAt,
269
+ );
270
} else if (dependency === 'topFocus' || dependency === 'topBlur') {
271
if (isEventSupported('focus', true)) {
272
ReactBrowserEventEmitter.ReactEventListener.trapCapturedEvent(
src/renderers/dom/shared/ReactEventListener.js
-3
@@ -12,7 +12,6 @@
12
'use strict';
13
14
var EventListener = require('fbjs/lib/EventListener');
15
-var ExecutionEnvironment = require('fbjs/lib/ExecutionEnvironment');
15
var PooledClass = require('PooledClass');
16
var ReactDOMComponentTree = require('ReactDOMComponentTree');
17
var ReactGenericBatching = require('ReactGenericBatching');
@@ -111,8 +110,6 @@ var ReactEventListener = {
110
_enabled: true,
111
_handleTopLevel: null,
112
114
- WINDOW_HANDLE: ExecutionEnvironment.canUseDOM ? window : null,
115
-
113
setHandleTopLevel: function(handleTopLevel) {
114
ReactEventListener._handleTopLevel = handleTopLevel;
115
},