[Webkit-unassigned] [Bug 118883] New: Cache cssomWrappers for StylePropertySets directly on the MutableStylePropertySet

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=118883

           Summary: Cache cssomWrappers for StylePropertySets directly on
                    the MutableStylePropertySet
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: CSS
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: ggaren at apple.com, koivisto at iki.fi,
                    barraclough at apple.com, akling at apple.com


Consider merging https://chromium.googlesource.com/chromium/blink/+/183bcd51eb0e79cab930cf46695df05dc793630f

In my measurements the mapping is adding more overhead than just having a field
in all MutableStylePropertySet objects. So this saves memory and makes access
faster.

Memory measurements:

gmail:
Wrappers: 276/297 (92%) Overhead: table > 16384 bytes, fields 2376 bytes

facebook:
Wrappers: 9/51 (17%) Overhead: table > 512 bytes, fields 408 bytes

google plus:
Wrappers: 208/229 (90%) Overhead: table > 16384 bytes, fields 1832 bytes

cnn:
Wrappers: 66/106 (62%) Overhead: table > 4096 bytes, fields 848 bytes

youtube:
Wrappers: 9/27 (33%) Overhead: table > 512 bytes, fields 216 bytes

amazon:
Wrappers: 48/90 (53%) Overhead: table > 4096 bytes, fields 720 bytes

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.