Skip to content

Speed up snapshotting of many new dom nodes #903

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Jun 6, 2022
Merged

Speed up snapshotting of many new dom nodes #903

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
ed86eca
Speed up snapshotting of many new dom nodes
Juice10 May 25, 2022
30e4f7d
Improve newlyAddedElement docs
Juice10 May 25, 2022
39a70d0
Optimize needMaskingText by using el.closest and less recursion
Juice10 May 26, 2022
778471e
Serve all rrweb dist files
Juice10 May 26, 2022
9eaf4ce
Split serializeNode into smaller functions
Juice10 May 26, 2022
5d77c7b
Slow down cpu enhance tracing on fast machines
Juice10 May 26, 2022
909c127
Increase timeout
Juice10 May 26, 2022
d289cde
Perf: only loop through ancestors when they have something to compare to
Juice10 May 27, 2022
9d666d3
Perf: `hasNode` is cheaper than `getMeta`
Juice10 May 27, 2022
ff2b225
Perf: If parents where already checked, no need to do it again
Juice10 May 27, 2022
b179941
Perf: reverse for loops are faster
Juice10 May 27, 2022
b2ae33e
Clean up code
Juice10 May 27, 2022
288c275
Perf: check ancestors once with isBlocked
Juice10 May 28, 2022
64c50a9
guessing this might fixes canvas test
Juice10 May 28, 2022
038f40e
Update packages/rrweb/src/record/observers/canvas/webgl.ts
Juice10 Jun 5, 2022
0a63f4c
Fix #904 (#906)
Juice10 May 31, 2022
4b4e343
Bump minimist from 1.2.5 to 1.2.6 (#902)
dependabot[bot] Jun 5, 2022
100c12f
Merge branch 'master' into serialize-perf
Juice10 Jun 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix #904 (#906) 
Properly remove crossorigin attribute
  • Loading branch information
@Juice10
Juice10 committed Jun 6, 2022
commit 0a63f4c27225a78faf0161c882690967c7f8e63b
2 changes: 1 addition & 1 deletion packages/rrweb-snapshot/src/snapshot.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -710,7 +710,7 @@ function serializeElementNode(
}
oldValue
? (attributes.crossOrigin = oldValue)
: delete attributes.crossOrigin;
: image.removeAttribute('crossorigin');
};
// The image content may not have finished loading yet.
if (image.complete && image.naturalWidth !== 0) recordInlineImage();
Expand Down