New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 824003 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 732399



Sign in to add a comment

hit-test aggregator should know to shrink hit-test data buffer size

Project Member Reported by riajiang@chromium.org, Mar 21 2018

Issue description

right now hit-test aggregator knows how to grow the buffer size when we have too many hit-test regions [1]. it is possible that after a while, user closes a lot of tabs/iframes that we now only need a small buffer size. hit-test aggregator should know to shrink the buffer size in that case.

[1] https://cs.chromium.org/chromium/src/components/viz/service/hit_test/hit_test_aggregator.h?type=cs&q=hit_test_agg&sq=package:chromium&l=56

 
riajiang: Is this still an issue?
afaik: this isn't done. It should still be done. Do we think that this would be a good task for an intern?
I think this becomes an issue only when/if we use shared-memory for the aggregated data. Right now, we always send a copy of the data back to the host*. So the task would involve actually switching back to using shared-memory for the transport.

[*] If we don't plan on using shared-memory soon-ish, perhaps we could simplify the existing code a bit.


Yea it was used to know when to allocate new shared-memory block on the aggregator side, now it's being used to decide when to increase the vector size cuz we don't know children size yet when we start aggregating.

Sign in to add a comment