| 47c315eeb288003291bdb98ea04b039f83a4cc83 |
|
08-Nov-2017 |
Robert Phillips <robertphillips@google.com> |
Remove destination pointer from GrCopySurfaceOp
This appears to be surplus to requirements (and the extra call in visitProxies was causing grief for explicit allocation)
Bug: skia:
Change-Id: Icc0b7f04fed96883408d54986b02d0f3756d56c7
Reviewed-on: https://skia-review.googlesource.com/68862
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| f1748f5a9238a0d3e189d50fc5e57ae8b8ec087c |
|
14-Sep-2017 |
Robert Phillips <robertphillips@google.com> |
Change visitProxies signature
There is a perf regression (mainly on the Nexus5) for the https://skia-review.googlesource.com/c/skia/+/46200 (Add method to iterate over a GrOp's GrSurfaceProxies)
This is one candidate.
Change-Id: I995d3a88bad2a914f24b49512abbf01aeaf579c8
Reviewed-on: https://skia-review.googlesource.com/46586
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 9d6c64f29a27496c9b778ec55aff0fdc851fd066 |
|
14-Sep-2017 |
Robert Phillips <robertphillips@google.com> |
Make GrCopySurfaceOp participate in the GrOplist DAG
Change-Id: Ic72884c8447ad950a91a2afe9566dd5a92dd6f11
Reviewed-on: https://skia-review.googlesource.com/46582
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 500d58b2a6e6fd03239622da42b67b2c9843b7be |
|
24-Aug-2017 |
Greg Daniel <egdaniel@google.com> |
Make Copy Ops to go through GpuCommandBuffer instead of straigt to GPU.
Bug: skia:
Change-Id: I4eae4507e07278997e26419e94586eef0780c423
Reviewed-on: https://skia-review.googlesource.com/38361
Commit-Queue: Greg Daniel <egdaniel@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 646e4293f06d9de6d44dbfa3c32cdc15a6f5906e |
|
13-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Retract GrTexture*.h & GrRenderTarget*.h from other headers
This does push some additional work (& includes) into the .cpp files.
Change-Id: I27c847e371802270d13594dcc22aae44039990bb
Reviewed-on: https://skia-review.googlesource.com/19660
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| eee4d6e4e8cc5c4c79f065abcc3ce609f71238f9 |
|
05-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Make instantiate return a Boolean
From an off-line conversation:
The longer term idea will be to create a helper class isolates the
ability to instantiate proxies until flush time. The peek* methods
could then be moved to GrSurfaceProxy.
Change-Id: I8e8c02c098475b77d515791c0d6b81f7e4a327dd
Reviewed-on: https://skia-review.googlesource.com/18076
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| a16f6cb40a7d6d186c86c2689c2be827b2093ae4 |
|
01-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Update copyOp to split-opList world
Change-Id: Ib18fc0a589185b11b21241e50acb7b506c44bfac
Reviewed-on: https://skia-review.googlesource.com/17325
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| f5442bb4c152e7c8138c83d27140e55d846f7ea5 |
|
17-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Split cleanup changes out of bigger CL (take 2)
This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)
This is a reland of: https://skia-review.googlesource.com/c/13585/ (Split cleanup changes out of bigger CL)
TBR=bsalomon@google.com
Change-Id: Id0d5c764953230412e78ae6aa654f8dfae998c60
Reviewed-on: https://skia-review.googlesource.com/13643
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| e186cfd15f2df43415ae024185bdcf5baa955ec9 |
|
17-Apr-2017 |
Mike Klein <mtklein@google.com> |
Revert "Split cleanup changes out of bigger CL"
This reverts commit cd11c809f206af0da3ce1779dee3c91193baa7b0.
Reason for revert: assert(rtp->getLastOpList())
Original change's description:
> Split cleanup changes out of bigger CL
>
> This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)
>
> TBR=bsalomon@google.com
>
> Change-Id: I1239e806a30a93d3c17ad933f118b98a5077dd3a
> Reviewed-on: https://skia-review.googlesource.com/13585
> Commit-Queue: Robert Phillips <robertphillips@google.com>
> Reviewed-by: Greg Daniel <egdaniel@google.com>
>
TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: Ia3ef7322fe830a6a4dc29226ce931d35d181c6e0
Reviewed-on: https://skia-review.googlesource.com/13641
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| cd11c809f206af0da3ce1779dee3c91193baa7b0 |
|
17-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Split cleanup changes out of bigger CL
This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)
TBR=bsalomon@google.com
Change-Id: I1239e806a30a93d3c17ad933f118b98a5077dd3a
Reviewed-on: https://skia-review.googlesource.com/13585
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 178ce3eddd326feee608af65cdeedb08e1af91c3 |
|
13-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Make all GrRenderTargetOpList ops store a GrRenderTarget pointer
Change-Id: I39652b4b8b2ba96b5206304dfc51395283c3fe16
Reviewed-on: https://skia-review.googlesource.com/13332
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| bf25d433f48bb336f1f7f8bcb0a4a357ac08bb16 |
|
07-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Make GrCopySurfaceOp friendlier to GrSurfaceProxy (take 2)
Reland of: https://skia-review.googlesource.com/c/11325/ (Make GrCopySurfaceOp friendlier to GrSurfaceProxy)
Change-Id: Ibe7312cebe7793af5803353a1499fbe7874c558f
Reviewed-on: https://skia-review.googlesource.com/11780
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 9a121cc6ad746c37611229dc0ec1805545c4d2e0 |
|
06-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Revert "Make GrCopySurfaceOp friendlier to GrSurfaceProxy"
This reverts commit d7cda9a6f050c30e65b0744b3cfb45cecdc40834.
Reason for revert: Tegra3!!!!
Original change's description:
> Make GrCopySurfaceOp friendlier to GrSurfaceProxy
>
> Change-Id: Iba8c068bd96ab154d466e485a9db31dd552138de
> Reviewed-on: https://skia-review.googlesource.com/11325
> Commit-Queue: Robert Phillips <robertphillips@google.com>
> Reviewed-by: Brian Salomon <bsalomon@google.com>
>
TBR=bsalomon@google.com,robertphillips@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: Idc31d469d94e6e2772ee5714987b3a05bb902d88
Reviewed-on: https://skia-review.googlesource.com/11580
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| d7cda9a6f050c30e65b0744b3cfb45cecdc40834 |
|
06-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Make GrCopySurfaceOp friendlier to GrSurfaceProxy
Change-Id: Iba8c068bd96ab154d466e485a9db31dd552138de
Reviewed-on: https://skia-review.googlesource.com/11325
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 9e50f7b11e9a9d3dc23c924bbb0f0b85c6f1d6cb |
|
06-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Store clipped bounds on GrOp rather than in parallel in GrRenderTargetOpList.
When the op is recorded we update its bounds to incorporate both clipping and aa bloat.
Change-Id: Ia205e058ebeda5fcdb2cd5e9b8a30a4972672b14
Reviewed-on: https://skia-review.googlesource.com/9233
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| f8334781914363caf537f22f012fcd5c03c60dad |
|
03-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Remove ref counting from GrOp.
Instead use std::unique_ptr to manage GrOp lifetime.
Change-Id: Ic1dc1e0ffd7254c3994221f498677af5bbf66a71
Reviewed-on: https://skia-review.googlesource.com/6479
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 69868af68403bd12aee040187347426affe41acc |
|
22-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Remove render target unique ID virtual from GrOp.
GrRenderTargetOpList now stores the IDs along side each op.
This should put us closer to using proxy IDs and not forcing early render target instantiation as many comments point towards.
Change-Id: I1ee82b01a0818a80d2bcac39fdf3a4ee7dccecc9
Reviewed-on: https://skia-review.googlesource.com/6403
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| bde428513ff3bd98a3d4f1c66ac779411233f817 |
|
21-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrOp::draw to GrOp::execute
Change-Id: I36b8532c57d0b6004a5fd283e30a506df89a4fa6
Reviewed-on: https://skia-review.googlesource.com/6387
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|
| 895274391db8df7357334aec260edca2e1735626 |
|
16-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
move src/gpu/batches -> src/gpu/ops
Change-Id: I6410eae41f051ce38bef6f38d670924c3483c325
Reviewed-on: https://skia-review.googlesource.com/6163
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/ops/GrCopySurfaceOp.h
|