Skip to content

Conversation

@sherryzy
Copy link
Contributor

@sherryzy sherryzy commented Nov 4, 2025

By marking resources with 'avoid_reuse', they are freed immediately after use instead of being kept in the resource pool. This ensures tiles are rastered on-demand.

This change significantly reduces the tile_memory usage, ~20M when navigating fast through videos, with no obvious negative performance impact.

Bug: 424591313

@sherryzy sherryzy requested review from y4vor and yell0wd0g November 4, 2025 00:54
@sherryzy sherryzy marked this pull request as ready for review November 4, 2025 00:56
@sherryzy sherryzy requested a review from a team as a code owner November 4, 2025 00:56
Copy link
Contributor

@yell0wd0g yell0wd0g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat! LGTM, ship it.

For main branch - would there be a better way to
instrument this code, perhaps a FeatureFlag?

By marking resources with 'avoid_reuse', they are freed
immediately after use instead of being kept in the resource pool.
This ensures tiles are rastered on-demand.

This change significantly reduces the tile_memory usage, ~20M when
navigating fast through videos, with no obvious negative
performance impact.

Bug: 424591313
@sherryzy sherryzy added the cp-26.android Cherry Pick to the 26.android branch label Nov 4, 2025
@jellefoks jellefoks merged commit 53984e3 into youtube:26.eap Nov 5, 2025
424 of 474 checks passed
cobalt-github-releaser-bot pushed a commit that referenced this pull request Nov 5, 2025
By marking resources with 'avoid_reuse', they are freed immediately
after use instead of being kept in the resource pool. This ensures tiles
are rastered on-demand.

This change significantly reduces the tile_memory usage, ~20M when
navigating fast through videos, with no obvious negative performance
impact.

Bug: 424591313
(cherry picked from commit 53984e3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cp-26.android Cherry Pick to the 26.android branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants