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

Issue 1930013002: (Mostly) Retract GrRenderTarget from SkGpuDevice (Closed)

Created:
4 years, 7 months ago by robertphillips
Modified:
4 years, 7 months ago
Reviewers:
bsalomon
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

(Mostly) Retract GrRenderTarget from SkGpuDevice This gets us most of the way to having SkGpuDevice exclusively use a GrDrawContext instead of a GrRenderTarget. There are a few other refactorings (e.g., rm need for fLegacyBitmap and accessRenderTarget) before the GrRenderTarget can be completely removed. Has calved off: https://codereview.chromium.org/1925313002/ (Tighten up SkSpecialSurface factory functions) https://codereview.chromium.org/1925803004/ (Add sk_sp to SkSurface_Gpu and SkGpuDevice) https://codereview.chromium.org/1956473002/ (Retract GrRenderTarget a bit within SkGpuDevice) https://codereview.chromium.org/1979913002/ (Rename GrDrawingMgr::abandon to wasAbandoned & add a matching entry point to GrDrawingContext) https://codereview.chromium.org/1982583002/ (Add isUnifiedMultisampled entry point to GrDrawContext) GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1930013002 Committed: https://skia.googlesource.com/skia/+/ca6eafc0f7e59ed89f03a9891b9fdf7de64f1f35

Patch Set 1 #

Patch Set 2 : more #

Patch Set 3 : more #

Patch Set 4 : more #

Patch Set 5 : update to ToT #

Patch Set 6 : update to ToT #

Patch Set 7 : update #

Patch Set 8 : update to ToT #

Patch Set 9 : update to ToT #

Patch Set 10 : update to ToT #

Patch Set 11 : clean up #

Patch Set 12 : more clean up #

Patch Set 13 : update #

Patch Set 14 : update to ToT #

Patch Set 15 : clean up #

Unified diffs Side-by-side diffs Delta from patch set Stats (+122 lines, -113 lines) Patch
M include/gpu/GrContext.h View 1 2 3 4 5 6 7 8 9 10 11 12 2 chunks +2 lines, -2 lines 0 comments Download
M include/gpu/GrDrawContext.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 2 chunks +3 lines, -2 lines 0 comments Download
M src/core/SkSpecialSurface.cpp View 1 2 3 4 5 6 7 8 9 10 11 3 chunks +17 lines, -25 lines 0 comments Download
M src/gpu/GrContext.cpp View 1 2 3 4 5 6 7 8 9 10 11 12 13 2 chunks +3 lines, -2 lines 0 comments Download
M src/gpu/SkGpuDevice.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 5 chunks +15 lines, -8 lines 0 comments Download
M src/gpu/SkGpuDevice.cpp View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 9 chunks +80 lines, -72 lines 0 comments Download
M src/image/SkSurface_Gpu.cpp View 1 2 3 4 5 6 7 8 9 10 13 14 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 24 (15 generated)
robertphillips
4 years, 7 months ago (2016-05-16 17:56:06 UTC) #10
robertphillips
ping
4 years, 7 months ago (2016-05-17 12:20:26 UTC) #11
bsalomon
lgtm
4 years, 7 months ago (2016-05-17 13:01:06 UTC) #12
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1930013002/280001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1930013002/280001
4 years, 7 months ago (2016-05-17 13:03:36 UTC) #14
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED, http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_64-Debug-Trybot/builds/8648)
4 years, 7 months ago (2016-05-17 13:44:23 UTC) #16
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1930013002/280001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1930013002/280001
4 years, 7 months ago (2016-05-17 16:19:29 UTC) #18
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 7 months ago (2016-05-17 16:50:58 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1930013002/280001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1930013002/280001
4 years, 7 months ago (2016-05-17 16:56:47 UTC) #22
commit-bot: I haz the power
4 years, 7 months ago (2016-05-17 16:57:50 UTC) #24
Message was sent while issue was closed.
Committed patchset #15 (id:280001) as
https://skia.googlesource.com/skia/+/ca6eafc0f7e59ed89f03a9891b9fdf7de64f1f35

Powered by Google App Engine
This is Rietveld 408576698