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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1412243013: Clean up GrAtlas and rename it GrLayerAtlas (Closed)

Created:
5 years, 1 month ago by robertphillips
Modified:
5 years, 1 month ago
Reviewers:
joshualitt
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Clean up GrAtlas and rename it GrLayerAtlas Committed: https://skia.googlesource.com/skia/+/e51fa4ac31716863b80633419370e7e4174ead61

Patch Set 1 #

Patch Set 2 : clean up #

Unified diffs Side-by-side diffs Delta from patch set Stats (+323 lines, -395 lines) Patch
M gyp/gpu.gypi View 2 chunks +2 lines, -2 lines 0 comments Download
D src/gpu/GrAtlas.h View 1 chunk +0 lines, -137 lines 0 comments Download
D src/gpu/GrAtlas.cpp View 1 chunk +0 lines, -229 lines 0 comments Download
A src/gpu/GrLayerAtlas.h View 1 chunk +145 lines, -0 lines 0 comments Download
A src/gpu/GrLayerAtlas.cpp View 1 chunk +145 lines, -0 lines 0 comments Download
M src/gpu/GrLayerCache.h View 7 chunks +14 lines, -10 lines 0 comments Download
M src/gpu/GrLayerCache.cpp View 9 chunks +17 lines, -17 lines 0 comments Download

Messages

Total messages: 10 (4 generated)
robertphillips
5 years, 1 month ago (2015-11-03 14:39:01 UTC) #2
joshualitt
On 2015/11/03 14:39:01, robertphillips wrote: lgtm
5 years, 1 month ago (2015-11-03 14:49:04 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1412243013/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1412243013/20001
5 years, 1 month ago (2015-11-03 14:50:13 UTC) #5
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 1 month ago (2015-11-03 15:01:20 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1412243013/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1412243013/20001
5 years, 1 month ago (2015-11-03 15:04:10 UTC) #9
commit-bot: I haz the power
5 years, 1 month ago (2015-11-03 15:04:51 UTC) #10
Message was sent while issue was closed. 
Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/e51fa4ac31716863b80633419370e7e4174ead61

Powered by Google App Engine
This is Rietveld 408576698