Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Allow drawing raster cache results whose device rect is one pixel larger than the cached image #17278

Merged

Conversation

jason-simmons
Copy link
Member

RasterCacheResult::draw constructs the device target rectangle by
calling SkRect::roundOut, which rounds down the left/top coordinates
and rounds up the right/bottom coordinates. The rounding can produce
a device rect whose width and/or height differs from the cache result
image's width/height by one pixel.

@auto-assign auto-assign bot requested a review from gaaclarke March 23, 2020 23:07
@jason-simmons jason-simmons requested a review from liyuqian March 23, 2020 23:07
Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

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

LGTM!

@jason-simmons jason-simmons force-pushed the nexus7_raster_cache_assert branch 2 times, most recently from 29f06bd to f29a5b9 Compare March 23, 2020 23:16
…ger than the cached image

RasterCacheResult::draw constructs the device target rectangle by
calling SkRect::roundOut, which rounds down the left/top coordinates
and rounds up the right/bottom coordinates.  The rounding can produce
a device rect whose width and/or height differs from the cache result
image's width/height by one pixel.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants