Chromium Code Reviews
Descriptioncc: Fix tile priority inversion in picture layer tiling.
This patch addresses the case where we can have a tile that is required
for activation but is not in the NOW bin. This can happen since the
check for whether the tile is required uses the tile's content_rect,
which includes border pixels. However, the priority calculation for
the tile uses TileBounds without the border pixels.
This patch also moves the code to calculate required tile state from
all of the different iterator call sites to MakePrioritizedTile, with
an additional check to ensure that we never make a required tile outside
of the now bin.
R=danakj@chromium.org
BUG=668892, 622080
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
Review-Url: https://codereview.chromium.org/2651413004
Cr-Commit-Position: refs/heads/master@{#447293}
(cherry picked from commit a062f034b9c367a0b3481e0390bcc0aca830b707)
Review-Url: https://codereview.chromium.org/2711573002 .
Cr-Commit-Position: refs/branch-heads/2987@{#620}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/050a3b4e60535623b42101cc4e269c077da721c7
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||