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

Issue 800883002: cc: Optimise eviction tile priority queue (Closed)

Created:
6 years ago by USE eero AT chromium.org
Modified:
6 years ago
Reviewers:
vmpstr
CC:
chromium-reviews, cc-bugs_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

cc: Optimise eviction tile priority queue Do not process the tiling with required for activation tiles if it is certain that that tiling cannot have required for activation tiles having the current priority bin. Committed: https://crrev.com/335a9099bb1da584e87f228d3f0a70015d350e0a Cr-Commit-Position: refs/heads/master@{#308337}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M cc/resources/tiling_set_eviction_queue.cc View 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 8 (2 generated)
USE eero AT chromium.org
This optimizes tiling set eviction queue not to look for RFA tiles in pending tree ...
6 years ago (2014-12-12 15:00:43 UTC) #2
vmpstr
lgtm. What's the performance impact on cc_perftests, if any?
6 years ago (2014-12-12 20:04:09 UTC) #3
USE eero AT chromium.org
On 2014/12/12 20:04:09, vmpstr wrote: > lgtm. What's the performance impact on cc_perftests, if any? ...
6 years ago (2014-12-15 10:48:29 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/800883002/1
6 years ago (2014-12-15 10:49:15 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years ago (2014-12-15 12:00:49 UTC) #7
commit-bot: I haz the power
6 years ago (2014-12-15 12:01:42 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/335a9099bb1da584e87f228d3f0a70015d350e0a
Cr-Commit-Position: refs/heads/master@{#308337}

Powered by Google App Engine
This is Rietveld 408576698