Skip to content

fix: Fix checking for patchTarget in initAdoptedStyleSheetObserver#110

Merged
mydea merged 2 commits intosentry-v2from
fn/fix-guard-stylesheet
Oct 13, 2023
Merged

fix: Fix checking for patchTarget in initAdoptedStyleSheetObserver#110
mydea merged 2 commits intosentry-v2from
fn/fix-guard-stylesheet

Conversation

@mydea
Copy link
Member

@mydea mydea commented Oct 12, 2023

Calling Object.getOwnPropertyDescriptor(undefined, 'xx') actually errors out, so let's probably guard there.

@mydea mydea requested a review from billyvg October 12, 2023 14:43
@mydea mydea self-assigned this Oct 12, 2023
@mydea mydea merged commit cb834b8 into sentry-v2 Oct 13, 2023
@mydea mydea deleted the fn/fix-guard-stylesheet branch October 13, 2023 06:56
@mydea
Copy link
Member Author

mydea commented Oct 13, 2023

Opened PR upstream as well: rrweb-io#1327

mydea added a commit to getsentry/sentry-javascript that referenced this pull request Oct 13, 2023
mydea added a commit to getsentry/sentry-javascript that referenced this pull request Oct 13, 2023
@mydea mydea added the is-upstreamed This PR has been merged upstream as well. label Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is-upstreamed This PR has been merged upstream as well.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants