-
Notifications
You must be signed in to change notification settings - Fork 276
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Retain tiles just outside the viewport a bit longer. #3467
Conversation
This includes #3465, which is being reviewed separately. We can either wait until that's reviewed or close that and land both commits in this PR. Pending picture caching try: |
Expand the size of the needed region that active tiles exist over. This means that as tiles get scrolled just in / out of the screen we retain them for a bit longer. Since we only support a single dirty rect (for now), this is helpful for avoiding the case of creating a large dirty rect when something in the middle of the screen is animating while scrolling. In future, once multiple dirty rects are supported, it will be far less important to retain tiles outside the viewport. Also fix a subtle bug with the descriptor dependencies where clipped out primitives would invalidate the tile due to changing the length of the descriptor needed_rects array.
@bors-servo r=bholley (via email) |
📌 Commit 63a331d has been approved by |
Retain tiles just outside the viewport a bit longer. Expand the size of the needed region that active tiles exist over. This means that as tiles get scrolled just in / out of the screen we retain them for a bit longer. Since we only support a single dirty rect (for now), this is helpful for avoiding the case of creating a large dirty rect when something in the middle of the screen is animating while scrolling. In future, once multiple dirty rects are supported, it will be far less important to retain tiles outside the viewport. Also fix a subtle bug with the descriptor dependencies where clipped out primitives would invalidate the tile due to changing the length of the descriptor needed_rects array. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/3467) <!-- Reviewable:end -->
☀️ Test successful - status-appveyor, status-taskcluster |
…3e4d157d4478 (WR PR #3467). r=kats servo/webrender#3467 Differential Revision: https://phabricator.services.mozilla.com/D15722 --HG-- extra : moz-landing-system : lando
…3e4d157d4478 (WR PR #3467). r=kats servo/webrender#3467 Differential Revision: https://phabricator.services.mozilla.com/D15722
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @gw3583)
webrender/src/picture.rs, line 506 at r1 (raw file):
let needed_device_rect = needed_device_rect.inflate( 0.0, 3.0 * TILE_SIZE_HEIGHT as f32,
would be good to move it out as a const, unless this is strictly temporary
…3e4d157d4478 (WR PR #3467). r=kats servo/webrender#3467 Differential Revision: https://phabricator.services.mozilla.com/D15722 UltraBlame original commit: 1b01f47f32c02a3335ee107c53ebf85306026875
…3e4d157d4478 (WR PR #3467). r=kats servo/webrender#3467 Differential Revision: https://phabricator.services.mozilla.com/D15722 UltraBlame original commit: 1b01f47f32c02a3335ee107c53ebf85306026875
…3e4d157d4478 (WR PR #3467). r=kats servo/webrender#3467 Differential Revision: https://phabricator.services.mozilla.com/D15722 UltraBlame original commit: 1b01f47f32c02a3335ee107c53ebf85306026875
Expand the size of the needed region that active tiles exist
over. This means that as tiles get scrolled just in / out of
the screen we retain them for a bit longer.
Since we only support a single dirty rect (for now), this is
helpful for avoiding the case of creating a large dirty rect
when something in the middle of the screen is animating while
scrolling. In future, once multiple dirty rects are supported,
it will be far less important to retain tiles outside the
viewport.
Also fix a subtle bug with the descriptor dependencies where
clipped out primitives would invalidate the tile due to changing
the length of the descriptor needed_rects array.
This change is