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

Issue 1141353002: cc: Update transform tree after compositor thread scroll (Closed)

Created:
5 years, 7 months ago by enne (OOO)
Modified:
5 years, 7 months ago
Reviewers:
Ian Vollick, ajuma
CC:
cc-bugs_chromium.org, chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@compproptree_2_add_verification_code
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

cc: Update transform tree after compositor thread scroll When LayerImpls scroll, their associated transform trees need to be updated so that they are correct. This fixes 17 of the 79 failing tests, when verification is turned on. R=ajuma@chromium.org,vollick@chromium.org BUG=481585 Committed: https://crrev.com/04192cf91fe892f516854317886ae2227af0ccb5 Cr-Commit-Position: refs/heads/master@{#330662}

Patch Set 1 #

Patch Set 2 : Remove unnecessary TODO #

Patch Set 3 : Update to scroll offset #

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

Messages

Total messages: 7 (2 generated)
enne (OOO)
5 years, 7 months ago (2015-05-18 23:44:14 UTC) #1
ajuma
lgtm
5 years, 7 months ago (2015-05-19 13:24:46 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1141353002/40001
5 years, 7 months ago (2015-05-19 22:44:24 UTC) #5
commit-bot: I haz the power
Committed patchset #3 (id:40001)
5 years, 7 months ago (2015-05-20 00:32:33 UTC) #6
commit-bot: I haz the power
5 years, 7 months ago (2015-05-20 00:33:43 UTC) #7
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/04192cf91fe892f516854317886ae2227af0ccb5
Cr-Commit-Position: refs/heads/master@{#330662}

Powered by Google App Engine
This is Rietveld 408576698