Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Issue 682843005: cc: Consider visible rect for update tile priority early out. (Closed)

Created:
6 years, 1 month ago by vmpstr
Modified:
6 years, 1 month ago
CC:
chromium-reviews, cc-bugs_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2171
Project:
chromium
Visibility:
Public.

Description

cc: Consider visible rect for update tile priority early out. This patch adds a visible rect check for an early out from update tile priorities. The problem is that we can update tile priorities, then update the visible rect in animation, and call update tile priorities again. The second one would normally early out since it's called in the same frame. This ensures the early out doesn't happen in these cases. BUG=423455 TBR=danakj, brianderson, enne Review URL: https://codereview.chromium.org/667053003 Cr-Commit-Position: refs/heads/master@{#300526} (cherry picked from commit cc0e08bd762154a6e31de5a2e3b2aa5b23af0eae) Committed: https://chromium.googlesource.com/chromium/src/+/a383f3fe2e315340b92a25968269b749ee38097a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -7 lines) Patch
M cc/layers/picture_layer_impl.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M cc/resources/picture_layer_tiling.h View 2 chunks +6 lines, -2 lines 0 comments Download
M cc/resources/picture_layer_tiling.cc View 3 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
vmpstr
6 years, 1 month ago (2014-10-27 23:59:25 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
a383f3fe2e315340b92a25968269b749ee38097a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698