Skip to content

Clean up bounding rect listener on timeout - #1228

Open
dajiaohuang wants to merge 1 commit into
ampproject:mainfrom
dajiaohuang:fix/bounding-rect-timeout-listener
Open

dajiaohuang wants to merge 1 commit into
ampproject:mainfrom
dajiaohuang:fix/bounding-rect-timeout-listener

Conversation

@dajiaohuang

Copy link
Copy Markdown

Summary

  • Remove the global message listener when getBoundingClientRectAsync() falls back after its timeout.
  • Clear the timeout when a matching bounding-rect response arrives.
  • Add a regression test for listener cleanup on the timeout path.

Validation

  • yarn test (1184 tests passed)
  • yarn lint passed
  • yarn size passed (one configured size check warned within 5% of its limit)
  • git diff --check passed

CI was not queried.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant