@samitouri / QOS-React / commits / e796d77004

Remove ensureScrollValueMonitoring (#9819)

I never actually removed the rest of this when I got rid of ViewportMetrics. We no longer need `ReactDOMEventListener.monitorScrollValue`.

Nathan Hunzaker committed Jun 1, 2017 at 07:18 UTC e796d7700475b15dd1f1be90eae9af5fcc3f023c
1 file changed -11
src/renderers/dom/shared/ReactDOMEventListener.js
-11
@@ -19,7 +19,6 @@ var ReactGenericBatching = require('ReactGenericBatching');
19 var ReactTypeOfWork = require('ReactTypeOfWork');
20
21 var getEventTarget = require('getEventTarget');
22 -var getUnboundedScrollPosition = require('fbjs/lib/getUnboundedScrollPosition');
22
23 var {HostRoot} = ReactTypeOfWork;
24
@@ -102,11 +101,6 @@ function handleTopLevelImpl(bookKeeping) {
101 }
102 }
103
105 -function scrollValueMonitor(cb) {
106 - var scrollPosition = getUnboundedScrollPosition(window);
107 - cb(scrollPosition);
108 -}
109 -
104 var ReactDOMEventListener = {
105 _enabled: true,
106 _handleTopLevel: null,
@@ -165,11 +159,6 @@ var ReactDOMEventListener = {
159 );
160 },
161
168 - monitorScrollValue: function(refresh) {
169 - var callback = scrollValueMonitor.bind(null, refresh);
170 - EventListener.listen(window, 'scroll', callback);
171 - },
172 -
162 dispatchEvent: function(topLevelType, nativeEvent) {
163 if (!ReactDOMEventListener._enabled) {
164 return;