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.
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
Use of many shadow trees with unique style triggers significant CPU usage and a rise in process size #18889
Merged
webkit-commit-queue
merged 1 commit into
WebKit:main
from
anttijk:style-resolver-sharing-limit
Oct 10, 2023
Merged
Use of many shadow trees with unique style triggers significant CPU usage and a rise in process size #18889
webkit-commit-queue
merged 1 commit into
WebKit:main
from
anttijk:style-resolver-sharing-limit
Oct 10, 2023
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
EWS run on previous version of this PR (hash 2cb9ccf) |
anttijk
force-pushed
the
style-resolver-sharing-limit
branch
from
October 10, 2023 10:31
2cb9ccf
to
6d8dc58
Compare
EWS run on previous version of this PR (hash 6d8dc58) |
anttijk
force-pushed
the
style-resolver-sharing-limit
branch
from
October 10, 2023 11:47
6d8dc58
to
14d1276
Compare
EWS run on previous version of this PR (hash 14d1276) |
anttijk
force-pushed
the
style-resolver-sharing-limit
branch
from
October 10, 2023 11:51
14d1276
to
fbb033f
Compare
EWS run on current version of this PR (hash fbb033f) |
alanbaradlay
approved these changes
Oct 10, 2023
β¦sage and a rise in process size https://bugs.webkit.org/show_bug.cgi?id=260925 rdar://114731415 Reviewed by Alan Baradlay. Style resolver sharing cache in Style::Scope has unlimited size and the test case keeps adding more items there. Low memory handler clears the cache causing memory use to eventually drop. * Source/WebCore/dom/InlineStyleSheetOwner.cpp: (WebCore::InlineStyleSheetOwner::createSheet): Increase the size of the inline stylesheet sharing cache (50 -> 256 items). Content that uses many unique shadow tree style (like this test) could overrun this smallish cache easily rendering it ineffective. Style resolver sharing in Style::Scope depends on inline stylesheet getting shared so this also made that cache work poorly. * Source/WebCore/style/StyleScope.cpp: (WebCore::Style::Scope::createOrFindSharedShadowTreeResolver): Add a limit to the resolver sharing cache size. Canonical link: https://commits.webkit.org/269142@main
webkit-commit-queue
force-pushed
the
style-resolver-sharing-limit
branch
from
October 10, 2023 15:23
fbb033f
to
b3c5d42
Compare
Committed 269142@main (b3c5d42): https://commits.webkit.org/269142@main Reviewed commits have been landed. Closing PR #18889 and removing active labels. |
webkit-commit-queue
removed
the
merge-queue
Applied to send a pull request to merge-queue
label
Oct 10, 2023
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.
b3c5d42
fbb033f
π iosπ ios-simπ§ͺ wpe-wk2π§ͺ ios-wk2π§ͺ ios-wk2-wptπ§ͺ api-ios