[ResizeSensor] try/catch findDOMNode to handle possible error state

packages/core/src/components/resize-sensor/resizeSensor.tsx

@@ -59,26 +59,24 @@ export class ResizeSensor extends React.PureComponent<IResizeSensorProps> {
     }
 
     public componentDidMount() {
-        // using findDOMNode for two reasons:
-        // 1. cloning to insert a ref is unwieldy and not performant.
-        // 2. ensure that we get an actual DOM node for observing.
-        this.observeElement(findDOMNode(this));
+        this.observeElement();
     }
 
     public componentDidUpdate(prevProps: IResizeSensorProps) {
-        this.observeElement(findDOMNode(this), this.props.observeParents !== prevProps.observeParents);
+        this.observeElement(this.props.observeParents !== prevProps.observeParents);
     }
 
     public componentWillUnmount() {
         this.observer.disconnect();
     }
 
     /**
-     * Observe the given element, if defined and different from the currently
+     * Observe the DOM element, if defined and different from the currently
      * observed element. Pass `force` argument to skip element checks and always
      * re-observe.
      */
-    private observeElement(element: Element | Text | null, force = false) {
+    private observeElement(force = false) {
+        const element = this.getElement();
         if (!(element instanceof Element)) {
             // stop everything if not defined
             this.observer.disconnect();
@@ -106,4 +104,16 @@ export class ResizeSensor extends React.PureComponent<IResizeSensorProps> {
             }
         }
     }
+
+    private getElement() {
+        try {
+            // using findDOMNode for two reasons:
+            // 1. cloning to insert a ref is unwieldy and not performant.
+            // 2. ensure that we resolve to an actual DOM node (instead of any JSX ref instance).
+            return findDOMNode(this);
+        } catch {
+            // swallow error if findDOMNode is run on unmounted component.
+            return null;
+        }
+    }
 }