Bug 250640
| Summary: | Entity storage is not very compact | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, bfulgham, cdumez, darin, rniwa, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 253358, 253360 | ||
Ahmad Saleem
Hi Team,
While going through Blink's commit, I came across another potential optimization:
Blink Commit - https://chromium.googlesource.com/chromium/blink/+/0c7555da9c7b6059b8f55820b5b21685469a42c1
Benefits:
The binary will shrink by about 20-35 KB (depending on architecture),
the memory usage should shrink by similar numbers for every
renderer (since the memory now can be shared between renderers) and
there will be a tiny startup time improvement due to
fewer relocation blocks and less to read from disk.
Just wanted to raise to see if we can do 1-1 merge or if we can come up with similar.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/104525785>
Ahmad Saleem
Just unscientific test to see whether it regress Speedometer 2.1 by compiling with trunk (260964@main) and few other test patches and applying this and before and after the scored remain same on M1 Pro 16 GB using macOS Ventura 13.2.1.
I am not sure on memory saving part but just wanted to share update on Speedometer side. If you want me to run any particular benchmark etc., happy to run and happy to do PR as well, if anyone want.
EWS
Committed 261245@main (6c9eb0e31985): <https://commits.webkit.org/261245@main>
Reviewed commits have been landed. Closing PR #10888 and removing active labels.
Darin Adler
*** Bug 253360 has been marked as a duplicate of this bug. ***