feat: Tooltip - Performance - avoid calling clearTimeout on unmount if timeout id is 0

[carloseustaquio]

Description

Context:

We have a product list page with products with multiple color variants. These color variants have the Tooltip component to display the color name on hover.

Issue:

We have noticed a significant amount of clearTimeout calls when the component with the Tooltips unmounts. That's interesting because it happens even when we don't trigger any of the Tooltips in the page.

After some investigation, we noticed this is caused by a window.clearTimeout that's happening in the component unmount.

Fix:

The fix proposed is quite simple. I'm checking openTimerRef.current and only calling this if it's not 0. I'm also resetting openTimerRef.current to 0 after each clearTimeout(). We have noticed a reduction in the number of clearTimeout calls in the unmount of this component. This means a significant performance improvement in our scenario given we have dozens of Tooltips in some of our pages.