New issue
Advanced search Search tips

Issue 606947 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 578211
Owner: ----
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Scroll customization map should use RareData instead

Project Member Reported by esprehn@chromium.org, Apr 26 2016

Issue description

I'm not sure I follow the discussion in here:

https://codereview.chromium.org/1236913004

but we should definitely not have a Map for scroll customization callbacks. I don't know what the comment about not affording the size increase is about, we've added stuff to RareData lots since then, and we add RareData for all kinds of reasons. If ElementRareData is too big we can do things like move the PseudoElements into a sub object, or other space saving things. We shouldn't keep separate tracking maps.
 
Mergedinto: 578211
Status: Duplicate (was: Untriaged)
Last time I thought about this, it seemed like traceWrapper would likely change these bindings a bunch, so I figured I'd hold off until then before fixing this.

Comment 2 by tkent@chromium.org, Jun 23 2016

Components: -Blink>Architecture Blink>Internals
Renaming Blink>Architecture to Blink>Internals

Sign in to add a comment