fix: cancel requestAnimationFrame on unmount in ContentOverlayController

[eunwoo-levi]

Description

• The useEffect in ContentOverlayController schedules a requestAnimationFrame but does not cancel it in the cleanup function. If the component unmounts before the next frame fires (e.g. rapidly opening and closing an overlay), the callback still executes and dispatches a stale OPEN action.

Changes

• Added cancelAnimationFrame cleanup in the useEffect of ContentOverlayController to cancel the pending frame on unmount.

Motivation and Context

• Without cleanup, a scheduled requestAnimationFrame can fire after the component has already unmounted.
• This is the standard React cleanup pattern for async scheduling inside useEffect, as described in the https://react.dev/learn/synchronizing-with-effects#how-to-handle-the-effect-firing-twice-in-development.
• It also prevents duplicate OPEN dispatches under React Strict Mode, which runs effects twice in development.

How Has This Been Tested?

• Verified existing unit tests pass locally.
• Manually confirmed that rapid open/close sequences no longer produce stale dispatches.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update

Checklist

• I have performed a self-review of my own code.
• My code is commented, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.
• Any dependent changes have been merged and published in downstream modules.

Further Comments

[changeset-bot]

⚠️ No Changeset found

Latest commit: dfa6a1d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

@eunwoo-levi is attempting to deploy a commit to the Toss Team on Vercel.

A member of the Team first needs to authorize it.