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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1274693004: cc: Declare high res tilings to not have low res content. (Closed)

Created:
5 years, 4 months ago by danakj
Modified:
5 years, 4 months ago
Reviewers:
vmpstr, enne (OOO)
CC:
chromium-reviews, Ian Vollick, sievers+watch_chromium.org, jbauman+watch_chromium.org, kalyank, piman+watch_chromium.org, cc-bugs_chromium.org, danakj+watch_chromium.org, piman
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

cc: Declare high res tilings to not have low res content. When a was-low-res tiling is made high res, we reset all the tiles, so we can also reset the flag saying it has low res content in it. That way we don't continually reset all the tiles in it. R=enne, vmpstr BUG=516500 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Committed: https://crrev.com/34fb122ba7915d0d189267e00a5d84d85aca5c98 Cr-Commit-Position: refs/heads/master@{#342670}

Patch Set 1 #

Patch Set 2 : waseverlowres: . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -9 lines) Patch
M cc/layers/picture_layer_impl.cc View 1 chunk +4 lines, -1 line 0 comments Download
M cc/layers/picture_layer_impl_unittest.cc View 3 chunks +15 lines, -4 lines 0 comments Download
M cc/tiles/picture_layer_tiling.h View 2 chunks +8 lines, -3 lines 0 comments Download
M cc/tiles/picture_layer_tiling.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 18 (7 generated)
danakj
5 years, 4 months ago (2015-08-06 21:39:21 UTC) #1
vmpstr
lgtm
5 years, 4 months ago (2015-08-06 21:52:35 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1274693004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1274693004/20001
5 years, 4 months ago (2015-08-06 22:02:18 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: win8_chromium_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/builds/25468)
5 years, 4 months ago (2015-08-06 23:08:25 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1274693004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1274693004/20001
5 years, 4 months ago (2015-08-07 22:02:07 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/74469)
5 years, 4 months ago (2015-08-07 23:36:42 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1274693004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1274693004/20001
5 years, 4 months ago (2015-08-07 23:38:00 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/90839)
5 years, 4 months ago (2015-08-08 01:13:29 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1274693004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1274693004/20001
5 years, 4 months ago (2015-08-10 18:18:01 UTC) #16
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 4 months ago (2015-08-10 19:23:34 UTC) #17
commit-bot: I haz the power
5 years, 4 months ago (2015-08-10 19:24:39 UTC) #18
Message was sent while issue was closed. 
Patchset 2 (id:??) landed as
https://crrev.com/34fb122ba7915d0d189267e00a5d84d85aca5c98
Cr-Commit-Position: refs/heads/master@{#342670}

Powered by Google App Engine
This is Rietveld 408576698