From 7585d8619de45f72dc0c980b6282da1e6be945a8 Mon Sep 17 00:00:00 2001 From: Dominic Gannaway Date: Fri, 15 Mar 2019 22:11:51 +0000 Subject: [PATCH] Change passive checker to use defineProperty --- packages/react-dom/src/events/checkPassiveEvents.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/react-dom/src/events/checkPassiveEvents.js b/packages/react-dom/src/events/checkPassiveEvents.js index ad175b3a05b..640caab67c2 100644 --- a/packages/react-dom/src/events/checkPassiveEvents.js +++ b/packages/react-dom/src/events/checkPassiveEvents.js @@ -16,11 +16,13 @@ export let passiveBrowserEventsSupported = false; // https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener#Safely_detecting_option_support if (enableEventAPI && canUseDOM) { try { - const options = { - get passive() { + const options = {}; + // $FlowFixMe: Ignore Flow complaining about needing a value + Object.defineProperty(options, 'passive', { + get: function() { passiveBrowserEventsSupported = true; }, - }; + }); window.addEventListener('test', options, options); window.removeEventListener('test', options, options); } catch (e) {