upstream: Merge upstream commits up to 609b7fac79a552f746dc880a28927dee382cd082 #220
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…rweb-io#1468) Setting of the `crossorigin` attribute is not necessary for same-origin images, and causes an immediate image reload (albeit from cache) necessitating the use of a load event listener which subsequently mutates the snapshot. This change allows us to avoid the mutation of the snapshot for the same-origin case. * Modify inlineImages test to remove delay and show that we can inline images without mutation * Add an explicit test for when the `image.crossOrigin = 'anonymous';` method is necessary. Uses a combination of about:blank and our test server to simulate a cross-origin context * Other test changes: there were some spurious rrweb mutations being generated by the addition of the crossorigin attribute that are now elimnated from the rrweb/__snapshots__/integration.test.ts.snap after this PR - this is good
fix for options `recordCanvas: true`: * replace document.createElement with doc.createElement in rrweb-snapshot code * Eoghan: add a regression test to prevent future accidental use of `document` instead of `doc`. This test can be excised if a new feature can only be run in the browser and not in the jsdom environment
* Older versions of Safari 16 don't support lookbehind assertions - https://caniuse.com/js-regexp-lookbehind * Refactor to show the similarity between these two regexes * Apply formatting changes * Create no-neg-lookbehind.md --------- Co-authored-by: eoghanmurray <[email protected]>
* Refactor to preclude the need for a continuous raf loop * Apply formatting changes * Create shadow-dom-unbusify.md --------- Co-authored-by: eoghanmurray <[email protected]>
…ure (rrweb-io#1489) * perf(mutation): add deep tree benchmark * perf(mutation): use iterative procedure * perf(mutation): run formatter * perf(mutation): add changeset
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…merge-609b7fac79a552f746dc880a28927dee382cd082
…2-update-to-609b7fac79a552f746dc880a28927dee382cd082
billyvg
changed the title
c298lee
approved these changes
Sep 18, 2024
billyvg
deleted the
sentry-v2-update-to-609b7fac79a552f746dc880a28927dee382cd082
branch
billyvg
added a commit
to getsentry/sentry-javascript
that referenced
this pull request
Sep 19, 2024
Pulls in upstream rrweb commits, see getsentry/rrweb#220
billyvg
added a commit
to getsentry/sentry-javascript
that referenced
this pull request
Sep 20, 2024
Pulls in upstream rrweb commits, see getsentry/rrweb#220
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merges upstream commits up to 609b7fa