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

Issue 678403002: Discard atlas after every MultiPictureDraw::draw (Closed)

Created:
6 years, 1 month ago by robertphillips
Modified:
6 years, 1 month ago
Reviewers:
bsalomon
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Project:
skia
Visibility:
Public.

Description

Discard atlas after every MultiPictureDraw::draw This is intended to prevent ghosting on tiled architectures. This CL also defers creation of the atlas (and its texture) until it is actually needed. Committed: https://skia.googlesource.com/skia/+/6d5b5455743414ddb11d2b8c1fe9d7959f2b853d Committed: https://skia.googlesource.com/skia/+/4ab5a9048cf327adfc9ef9757ea956d7003ba047

Patch Set 1 #

Patch Set 2 : Turn off gpu stats #

Patch Set 3 : Discard atlas after every MultiPictureDraw::draw #

Patch Set 4 : Remove creation of atlas in ctor #

Patch Set 5 : Update unit test #

Patch Set 6 : Fix rebase error #

Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -21 lines) Patch
M src/core/SkMultiPictureDraw.cpp View 1 2 3 4 5 1 chunk +3 lines, -0 lines 0 comments Download
M src/gpu/GrLayerCache.h View 1 2 3 4 3 chunks +7 lines, -2 lines 0 comments Download
M src/gpu/GrLayerCache.cpp View 1 2 3 4 11 chunks +28 lines, -15 lines 0 comments Download
M src/gpu/GrLayerHoister.h View 1 2 1 chunk +6 lines, -0 lines 0 comments Download
M src/gpu/GrLayerHoister.cpp View 1 2 3 4 1 chunk +7 lines, -4 lines 0 comments Download
M tests/GpuLayerCacheTest.cpp View 1 2 3 4 3 chunks +10 lines, -0 lines 0 comments Download

Messages

Total messages: 15 (6 generated)
robertphillips
6 years, 1 month ago (2014-10-28 21:17:33 UTC) #2
bsalomon
lgtm
6 years, 1 month ago (2014-10-29 13:53:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/678403002/60001
6 years, 1 month ago (2014-10-29 14:56:15 UTC) #5
commit-bot: I haz the power
Committed patchset #4 (id:60001) as 6d5b5455743414ddb11d2b8c1fe9d7959f2b853d
6 years, 1 month ago (2014-10-29 15:05:26 UTC) #6
robertphillips
A revert of this CL (patchset #4 id:60001) has been created in https://codereview.chromium.org/687233002/ by robertphillips@google.com. ...
6 years, 1 month ago (2014-10-29 15:16:47 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/678403002/80001
6 years, 1 month ago (2014-10-29 16:53:31 UTC) #9
commit-bot: I haz the power
Failed to apply patch for src/core/SkMultiPictureDraw.cpp: While running git apply --index -3 -p1; error: patch ...
6 years, 1 month ago (2014-10-29 19:46:46 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/678403002/100001
6 years, 1 month ago (2014-10-29 20:46:46 UTC) #14
commit-bot: I haz the power
6 years, 1 month ago (2014-10-29 20:56:07 UTC) #15
Message was sent while issue was closed.
Committed patchset #6 (id:100001) as 4ab5a9048cf327adfc9ef9757ea956d7003ba047

Powered by Google App Engine
This is Rietveld 408576698