| 1706f842086c089ebc496dfc7f45c959e5eda01e |
|
10-Nov-2017 |
Ethan Nicholas <ethannicholas@google.com> |
switched GrClipEdge to an enum class
Bug: skia:
Change-Id: Idf41580314a32739c70721530fc3ca48e566b044
Reviewed-on: https://skia-review.googlesource.com/70023
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/gm/beziereffects.cpp
|
| 0f3c73220a25eba9bf11583db3d0f76a2c4f6331 |
|
09-Nov-2017 |
Ethan Nicholas <ethannicholas@google.com> |
renamed GrPrimitiveEdgeType / GrProcessorEdgeType to GrClipEdgeType
Bug: skia:
Change-Id: I4a9af0b9b2cfa47875b2ba098098183e8dca29a7
Reviewed-on: https://skia-review.googlesource.com/69601
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/gm/beziereffects.cpp
|
| 74f623d1617e0ccf3eddf37aeecabd0ac72369fd |
|
07-Nov-2017 |
Cary Clark <caryclark@skia.org> |
make point array methods private
Moved method are not used by chromium, google3,
or android.
SkPoint::setRectIFan isn't used or tested at all.
SkPoint::setRectFan and SkPoint::setRectTriStrip
are only used internally.
These routines pretend that a SkPoint is part
of an array of points. Since that's kind of an
odd contract to make public, and because they
aren't used outside of Skia, relegate them to
a priv file.
R=bsalomon@google.com,reed@google.com
Bug: skia: 6898
Change-Id: I5ec2eb47799f6fd4b2994da962b1fa69ce659931
Reviewed-on: https://skia-review.googlesource.com/68121
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Cary Clark <caryclark@google.com>
/external/skia/gm/beziereffects.cpp
|
| e4442cb0b537720ab32106b3b5353dbb78f11b26 |
|
18-Oct-2017 |
Cary Clark <caryclark@skia.org> |
convert mapHomogeneousPoints to SkPoint3
SkMatrix::mapHomogeneousPoints takes an array of SkScalar,
but expects essentially SkPoint3, so make it so.
R: robertphillips@google.com, reed@google.com
Bug: skia:6898
Change-Id: Ibaf8b05c08b7df16c67d6a77d914667ace9a70da
Reviewed-on: https://skia-review.googlesource.com/58380
Commit-Queue: Cary Clark <caryclark@skia.org>
Commit-Queue: Cary Clark <caryclark@google.com>
Reviewed-by: Mike Reed <reed@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 57caa660c024cf6fda5e1fba8cb21224b51375fe |
|
18-Oct-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans.""
This reverts commit 62563deb6b4dbb0b1db7f29f35e07dcef043af31.
Reason for revert: change that affected similar set of GMs reverted, relanding now that this is more easily triagable.
Original change's description:
> Revert "Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans."
>
> This reverts commit fa2d604a7ded95a3ace905519b476129cd0fffcb.
>
> Reason for revert: <INSERT REASONING HERE>
>
> Original change's description:
> > Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans.
> >
> > Right now when we turn rects into quads we use a vertex order compatible with a tri fan rather than a tri strip.
> >
> > I wanted it to be the case that the same code could be used to generate a non-indexed mesh for a single rect or indexed using the quad index buffer when batching. Triangle fanning is not available in all APIS (e.g. is emulated in ANGLE and not supported in Metal) so it seems better to use a triangle strip over a fan in the single rect case.
> >
> >
> > Change-Id: I31eebd794e7328f4b39e3ec3377bf2ec556360ca
> > Reviewed-on: https://skia-review.googlesource.com/60081
> > Commit-Queue: Brian Salomon <bsalomon@google.com>
> > Reviewed-by: Robert Phillips <robertphillips@google.com>
>
> TBR=bsalomon@google.com,robertphillips@google.com
>
> Change-Id: I7c4c23aa418da09c9708b28cce64ab58e282dd3a
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://skia-review.googlesource.com/60683
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>
TBR=bsalomon@google.com,robertphillips@google.com
Change-Id: Iefcd16676a7617d32e89fc84206cd4e88e9a06e1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/61160
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 62563deb6b4dbb0b1db7f29f35e07dcef043af31 |
|
17-Oct-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans."
This reverts commit fa2d604a7ded95a3ace905519b476129cd0fffcb.
Reason for revert: <INSERT REASONING HERE>
Original change's description:
> Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans.
>
> Right now when we turn rects into quads we use a vertex order compatible with a tri fan rather than a tri strip.
>
> I wanted it to be the case that the same code could be used to generate a non-indexed mesh for a single rect or indexed using the quad index buffer when batching. Triangle fanning is not available in all APIS (e.g. is emulated in ANGLE and not supported in Metal) so it seems better to use a triangle strip over a fan in the single rect case.
>
>
> Change-Id: I31eebd794e7328f4b39e3ec3377bf2ec556360ca
> Reviewed-on: https://skia-review.googlesource.com/60081
> Commit-Queue: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
TBR=bsalomon@google.com,robertphillips@google.com
Change-Id: I7c4c23aa418da09c9708b28cce64ab58e282dd3a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/60683
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| fa2d604a7ded95a3ace905519b476129cd0fffcb |
|
17-Oct-2017 |
Brian Salomon <bsalomon@google.com> |
Make GPU backend triangulate rects such that they are rendered as tri strips rather than tri fans.
Right now when we turn rects into quads we use a vertex order compatible with a tri fan rather than a tri strip.
I wanted it to be the case that the same code could be used to generate a non-indexed mesh for a single rect or indexed using the quad index buffer when batching. Triangle fanning is not available in all APIS (e.g. is emulated in ANGLE and not supported in Metal) so it seems better to use a triangle strip over a fan in the single rect case.
Change-Id: I31eebd794e7328f4b39e3ec3377bf2ec556360ca
Reviewed-on: https://skia-review.googlesource.com/60081
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 9a725dd9485654155e2e4196c32d372360bcdb61 |
|
20-Sep-2017 |
Brian Osman <brianosman@google.com> |
Use shader based blending to clamp kPlus mode w/F16
Bug: skia:6173
Change-Id: I21042d484d9a7b3eee04aa3301d9793d00ad6908
Reviewed-on: https://skia-review.googlesource.com/48183
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/gm/beziereffects.cpp
|
| 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/gm/beziereffects.cpp
|
| b493eebca14aefbd5f22fb5d45ba978b19db4b18 |
|
13-Sep-2017 |
Robert Phillips <robertphillips@google.com> |
Add method to iterate over a GrOp's GrSurfaceProxies
The extra generality of having a std::function is for MDB reordering. In the current MDB reordering world there is one pass through the surfaceProxies at creation time and a second pass after flush to create the usage intervals.
Change-Id: I3f548417eddc1dad7503d919241301e404255ffe
Reviewed-on: https://skia-review.googlesource.com/46200
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| bfd18cdd5421b6fac063bd07a96a5b9b1afa6a88 |
|
09-Aug-2017 |
Brian Salomon <bsalomon@google.com> |
Move GrAppliedClip into GrPipeline
Change-Id: I522c2fd52bea9813baba7cdb3f11b63e7ab96b50
Reviewed-on: https://skia-review.googlesource.com/28861
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 91326c34ee1f1531d62caa153746821b58e6e55d |
|
09-Aug-2017 |
Brian Salomon <bsalomon@google.com> |
Move GrProcessorSet into GrPipeline
Change-Id: Ibfa5e1adda3c32140590aa62a31d35654cef79dd
Reviewed-on: https://skia-review.googlesource.com/28187
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 477d0efcf2d90c70a87c5a126349e76ac57d9649 |
|
14-Jul-2017 |
Brian Salomon <bsalomon@google.com> |
Make ops in bezier_effects GMs be non-Legacy GrMeshDrawOps
Change-Id: I1d56536820a61af293a7bce290f35b724c3a8257
Reviewed-on: https://skia-review.googlesource.com/23480
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| febbffad1c24136f041d7fc2d5ffcd50e47a047f |
|
08-Jun-2017 |
Chris Dalton <csmartdalton@google.com> |
Improve cubic KLM accuracy
Moves cubic root finding logic out of GrPathUtils and
PathOpsCubicIntersectionTest, and unifies it in SkGeometry.
"Normalizes" the homogeneous parameter values of the roots, rather
than the cubic inflection function. Does this normalization by
twiddling the exponents instead of division (which causes a loss of
precision).
Abandons the built-in derivatives in GrCubicEffect. These don't have
high enough precision on many mobile gpus. Instead we pass the KLM
matrix to the vertex shader via uniform, where we can use it to set up
new linear functionals from which the fragment shader can calculate
the gradient of the implicit function.
Bug: skia:4410
Change-Id: Ibd64e999520adc8cdef7803a492d3699995aef5a
Reviewed-on: https://skia-review.googlesource.com/19017
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
/external/skia/gm/beziereffects.cpp
|
| 23e474cb7331c5d2389d97dce2d9e5c93c58f39f |
|
15-May-2017 |
Hal Canary <halcanary@google.com> |
SkCanvas: Helpers for draw{Point,Line,Circle}
Change-Id: Ie9c7d3b8f01aee435563b23b7d27f098f07dd287
Reviewed-on: https://skia-review.googlesource.com/16909
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
/external/skia/gm/beziereffects.cpp
|
| d3ccb0a37f0e62c84fdcd6a77b7b15476b04db7a |
|
03-Apr-2017 |
Brian Salomon <bsalomon@google.com> |
Move pipeline handling out of GrMeshDrawOp.
The monolithic GrPipeline is moved to a subclass GrLegacyDrawMeshOp.
The pipeline used to record a GrMesh draw in a GrMeshDrawOp must now be passed rather than implicitly using the op's pipeline.
Change-Id: I50d77e4dcc8d91a523fa7566ce43a9a291174706
Reviewed-on: https://skia-review.googlesource.com/11002
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| cc26127920069cbd83e92cca3c69bb56cb165bcc |
|
23-Mar-2017 |
csmartdalton <csmartdalton@google.com> |
Find cubic KLM functionals directly
- Updates GrPathUtils to computes the KLM functionals directly instead
of deriving them from their explicit values at the control points.
- Updates the utility to return these functionals as a matrix
rather than an array of scalar values.
- Adds a benchmark for chopCubicAtLoopIntersection.
BUG=skia:
Change-Id: I97a9b5cf610d33e15c9af96b9d9a8eb4a94b1ca7
Reviewed-on: https://skia-review.googlesource.com/9951
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/gm/beziereffects.cpp
|
| 8199d94c0812a05320064f5f864b4208ffb25dba |
|
14-Mar-2017 |
Greg Daniel <egdaniel@google.com> |
Generaly Fixes to gpu bezier code
BUG=skia:
Change-Id: I2246aa0b2bc3327df42ca30d35020a4b878a819b
Reviewed-on: https://skia-review.googlesource.com/9555
Commit-Queue: Greg Daniel <egdaniel@google.com>
Reviewed-by: Chris Dalton <csmartdalton@google.com>
/external/skia/gm/beziereffects.cpp
|
| 649a3411f99a8aea3c46e4ef1f495f61b9801164 |
|
09-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Add a separate draw function to GrRenderTargetContext for GrMeshDrawOp derived classes.
The first phase of deferring GrPipeline creation until flush will apply only to GrDrawOp subclasses that do not derive from GrMeshDrawOp. This change prepares for that by creating separate draw functions on GrRenderTargetContext for GrMeshDrawOp-derived ops. This is temporary and will incrementally be undone as pipeline-creation deferral rolls out to the GrMeshDrawOps in a later phase of this work.
Change-Id: I0f5b71fe913f3273cfe9e965f7d8bbe7f01ad0ef
Reviewed-on: https://skia-review.googlesource.com/9481
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 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/gm/beziereffects.cpp
|
| df85c38163245c7fc3c23cad3a4ad104949f3a62 |
|
14-Feb-2017 |
Mike Reed <reed@google.com> |
stop using SkScalarMul
BUG=skia:
Change-Id: Ie41d8665a1c62ba8eddc93d8cfefaf64ddc52ff8
Reviewed-on: https://skia-review.googlesource.com/8411
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Reed <reed@google.com>
/external/skia/gm/beziereffects.cpp
|
| 82f44319159bb98dcacdbbec7ea643dde5ed024b |
|
11-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Make GrPaints move their GrProcessor ownership into GrPipelineBuilder.
This makes GrPaints usable only once. In some places we must make copies in order to issue draws with the same paint state.
Change-Id: Ie816e5185ce93a064111cad64c6880e1e21184c2
Reviewed-on: https://skia-review.googlesource.com/6844
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| a16339297859f37df69230e64f05624cef511ad3 |
|
09-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Removing ref counting from GrXPFactory.""
This reverts commit 003312a211e65f35e402d6fe80a32e23d4c94ac4.
Change-Id: Ib41065e5c356d1dd99e70fa10611ac6756c2b79d
Reviewed-on: https://skia-review.googlesource.com/6803
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 003312a211e65f35e402d6fe80a32e23d4c94ac4 |
|
09-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Removing ref counting from GrXPFactory."
This reverts commit a8f80de2bc17672b4b6f26d3cf6b38123ac850c9.
Reason for revert: nanobench failing on windows bots, possibly others
Change-Id: Iacb8c650064a28654c165665be057377ffb02ba5
Reviewed-on: https://skia-review.googlesource.com/6802
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| a8f80de2bc17672b4b6f26d3cf6b38123ac850c9 |
|
07-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Removing ref counting from GrXPFactory.
All GrXPFactory instances are static constexpr.
Change-Id: If1086b08534166201e53b3fd9379104e361eb5e6
Reviewed-on: https://skia-review.googlesource.com/6701
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 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/gm/beziereffects.cpp
|
| 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/gm/beziereffects.cpp
|
| 6b316e9e20d67f0f13021ca62f4270105c6151b1 |
|
16-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrTestBatch and subclasses to Op
Change-Id: Ideab66b7ca227057a767be48aba3ea69a0a19115
Reviewed-on: https://skia-review.googlesource.com/6161
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/gm/beziereffects.cpp
|
| 1951f3ddbddcf5e7cdfe151981d13a1bdb6a3baa |
|
09-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename testingOnly_drawBatch to testingOnly_addDrawOp and sk_sp
Change-Id: I35efd4ad2b7132145c1e477f0b1f283276e9fad5
Reviewed-on: https://skia-review.googlesource.com/5704
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 0e8fc8b9e6a138cf4a66b421fb824679df717329 |
|
09-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Relandx2 "Remove antialiasing control from GrPaint."
Fixes a bad merge.
This reverts commit 073285c0595d46205d1482cc19af2d7d891bfeae.
Change-Id: I5e92339d9b33d3a6dc58b9fcd2a1b3a5684e8f8a
Reviewed-on: https://skia-review.googlesource.com/5774
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 073285c0595d46205d1482cc19af2d7d891bfeae |
|
09-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Revert "Reland "Remove antialiasing control from GrPaint.""
This reverts commit 3944484020d98ff8f386378296106c321279482b.
Reason for revert: Merges badly with a recent change. Will rebase and reland.
Original change's description:
> Reland "Remove antialiasing control from GrPaint."
>
> This contains fixes for GLPrograms test and mixed samples rendering.
>
> This reverts commit 419d81eed4a010e6080db199795117cbedf9e6e4.
>
> BUG=skia:
>
> Change-Id: If8f002fbfaaaab6d1607403f2b15ccc7f1e17e87
> Reviewed-on: https://skia-review.googlesource.com/5763
> Commit-Queue: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
>
TBR=bsalomon@google.com,robertphillips@google.com,reviews@skia.org
BUG=skia:
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: Iff9657041e28604a845bc5a9acec7c9b248c53bd
Reviewed-on: https://skia-review.googlesource.com/5772
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 3944484020d98ff8f386378296106c321279482b |
|
09-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Reland "Remove antialiasing control from GrPaint."
This contains fixes for GLPrograms test and mixed samples rendering.
This reverts commit 419d81eed4a010e6080db199795117cbedf9e6e4.
BUG=skia:
Change-Id: If8f002fbfaaaab6d1607403f2b15ccc7f1e17e87
Reviewed-on: https://skia-review.googlesource.com/5763
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 419d81eed4a010e6080db199795117cbedf9e6e4 |
|
08-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Revert "Remove antialiasing control from GrPaint."
This reverts commit 9f549358b3ac9f61e78b194e39d6ac6eb322e35e.
Reason for revert: hitting asserts
Change-Id: I542d34edc05ecf72b7646263f25736a0950c78e7
Reviewed-on: https://skia-review.googlesource.com/5707
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/gm/beziereffects.cpp
|
| 9f549358b3ac9f61e78b194e39d6ac6eb322e35e |
|
08-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Remove antialiasing control from GrPaint.
This adds an additional param (of new enum type GrAA) to draws that can antialias and a new enum GrAAType to indicate the AA technique (none, fragment shader computed coverage, msaa).
Some GMs change due to this:
1) In some places we weren't disabling MSAA when the draw was supposed to be unantialiased.
2) Some bounding rect draws that use GrFragmentProcessors were unnecessarily turning on antialiasing, by disabling it a very small number of pixel LSBs change.
Change-Id: I7d8d8793dda70bcd373d09055beb9949c1a8a4d0
Reviewed-on: https://skia-review.googlesource.com/5608
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 9afd371a8a66f992f98eb2a3fc75ae64bddc730b |
|
01-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrDrawBatch->GrDrawOp
Change-Id: I18f520924b8a2548566fd61dbea4e3e12bd253dd
Reviewed-on: https://skia-review.googlesource.com/5411
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/gm/beziereffects.cpp
|
| 25a880960a9a689a745a01071ecba3fe494b5940 |
|
01-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrBatch to GrOp
Change-Id: I27b6324f8040899fafeda23ca524bc54a4dbf090
Reviewed-on: https://skia-review.googlesource.com/5392
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/gm/beziereffects.cpp
|
| cefc43112c8f6fe3702facb89447bdfcc2715345 |
|
04-Nov-2016 |
Hal Canary <halcanary@google.com> |
gm: s/SkAutoTUnref/sk_sp/
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4434
Change-Id: Ib2bea321617a17012190b33f2c7c439a5b6b3025
Reviewed-on: https://skia-review.googlesource.com/4434
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
/external/skia/gm/beziereffects.cpp
|
| 7d954ad797176afedb9262fdea4507d0fc60eb9d |
|
28-Oct-2016 |
Mike Reed <reed@google.com> |
remove xfermode from public api
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4020
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Change-Id: I19cd056f2af778f10e8c6c2b7b2735593b43dbac
Reviewed-on: https://skia-review.googlesource.com/4020
Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
/external/skia/gm/beziereffects.cpp
|
| 693a540272a771c6b0830094c461397cb77543b1 |
|
27-Oct-2016 |
Brian Osman <brianosman@google.com> |
Rename renderTargetContextPriv to priv, and some more leftover DCs
API change is for non-public API.
TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4060
Change-Id: I5011d753c3c6d1145ff242eab6baff2ae0647ba3
Reviewed-on: https://skia-review.googlesource.com/4060
Reviewed-by: Brian Osman <brianosman@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/gm/beziereffects.cpp
|
| 1105224f9701e57ec5ce0354d6a380b664f5c638 |
|
27-Oct-2016 |
Brian Osman <brianosman@google.com> |
Rename GrDrawContext to GrRenderTargetContext
This is in preparation for GrTextureContext and GrSurfaceContext
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4030
Change-Id: Ie58c93052e68f3f1f5fe8d15d63760de274a6fbd
Reviewed-on: https://skia-review.googlesource.com/4030
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/gm/beziereffects.cpp
|
| 414c4295f951d43068666b6294df15b2fd2ba85c |
|
26-Sep-2016 |
caryclark <caryclark@google.com> |
allow conic chop to fail
Fuzzy values may cause the conic chop to fail.
Check to see if the values are all finite, and
require the caller to do the same.
R=reed@google.com
BUG=650178
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2368993002
Review-Url: https://codereview.chromium.org/2368993002
/external/skia/gm/beziereffects.cpp
|
| dbfd7ab10883f173f5c1b653a233e18dc6142002 |
|
01-Sep-2016 |
mtklein <mtklein@chromium.org> |
Replace a lot of 'static const' with 'constexpr' or 'const'.
'static const' means, there must be at most one of these, and initialize it at
compile time if possible or runtime if necessary. This leads to unexpected
code execution, and TSAN* will complain about races on the guard variables.
Generally 'constexpr' or 'const' are better choices. Neither can cause races:
they're either intialized at compile time (constexpr) or intialized each time
independently (const).
This CL prefers constexpr where possible, and uses const where not. It even
prefers constexpr over const where they don't make a difference... I want to have
lots of examples of constexpr for people to see and mimic.
The scoped-to-class static has nothing to do with any of this, and is not changed.
* Not yet on the bots, which use an older TSAN.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300623005
Review-Url: https://codereview.chromium.org/2300623005
/external/skia/gm/beziereffects.cpp
|
| 28a838e532250fcca9673aca6c4616193a5a139d |
|
23-Jun-2016 |
robertphillips <robertphillips@google.com> |
Move GrPipelineBuilder out of gms & reduce use of GrPipelineBuilder.h
Just another step in reining in the GrPLB
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2086293006
Review-Url: https://codereview.chromium.org/2086293006
/external/skia/gm/beziereffects.cpp
|
| 06ca8ec87cf6fab57cadd043a5ac18c4154a4129 |
|
09-Jun-2016 |
bungeman <bungeman@google.com> |
sk_sp for Ganesh.
Convert use of GrFragmentProcessor, GrGeometryProcessor, and
GrXPFactory to sk_sp. This clarifies ownership and should
reduce reference count churn by moving ownership.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2041113004
Review-Url: https://codereview.chromium.org/2041113004
/external/skia/gm/beziereffects.cpp
|
| 55fdcccf01d49173bc731024b4283d3a3ea9150a |
|
06-Jun-2016 |
robertphillips <robertphillips@google.com> |
Remove GrRenderTarget from GrPipelineBuilder
This used to be "Declassify GrClipMaskManager and Remove GrRenderTarget and GrDrawTarget from GrPipelineBuilder"
but has been split into:
https://codereview.chromium.org/1993263002/ (Make GrAppliedClip friendless)
https://codereview.chromium.org/1997773002/ (Retract GrRenderTarget from GLProgramsTest)
https://codereview.chromium.org/1993403002/ (GrSWMaskHelper and GrSoftwarePathRenderer only need the textureProvider (not GrContext))
https://codereview.chromium.org/2004433002/ (Retract GrRenderTarget from GrTestTarget)
https://codereview.chromium.org/2015333002/ (Change parameters to GrPipelineBuilder's ctor)
https://codereview.chromium.org/2035823002/ (Make GrClipMaskManager stateless and push GrPipelineBuilder construction downstack)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1988923002
Review-Url: https://codereview.chromium.org/1988923002
/external/skia/gm/beziereffects.cpp
|
| 175dd9b5e3d7d749738dac743d2ac360b5340187 |
|
28-Apr-2016 |
robertphillips <robertphillips@google.com> |
Clean up test drawContext usage
The general idea is to provide access to SkGpuDevice's drawContext rather than its GrRenderTarget. That is usually what the testing framework actually wants.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1930623003
Review-Url: https://codereview.chromium.org/1930623003
/external/skia/gm/beziereffects.cpp
|
| 6c7e325236b7fbb9c9b14ebb4715f88a59718695 |
|
27-Apr-2016 |
robertphillips <robertphillips@google.com> |
Bring sk_sp to GrDrawContext
This is split out of: https://codereview.chromium.org/1914883002/ (Refactor drawContext/RenderTarget creation)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1918003003
Review-Url: https://codereview.chromium.org/1918003003
/external/skia/gm/beziereffects.cpp
|
| 5e3815b4d074fe3c47bbf0969446ed9870e5ef0a |
|
20-Apr-2016 |
robertphillips <robertphillips@google.com> |
Rein in the use of SkDevice.h
Nothing interesting.
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1904673003
Review URL: https://codereview.chromium.org/1904673003
/external/skia/gm/beziereffects.cpp
|
| 342bfc25de5b0452b1551bf9db4bf45eac7718b2 |
|
01-Apr-2016 |
bsalomon <bsalomon@google.com> |
Simplify GrDrawBatch uploads and token uage.
GrVertexBatch subclasses no longer need "initDraw".
Simplifies GrTestBatch
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1835283002
Review URL: https://codereview.chromium.org/1835283002
/external/skia/gm/beziereffects.cpp
|
| 9d524f22bfde5dc3dc8f48e1be39bdebd3bb0304 |
|
29-Mar-2016 |
halcanary <halcanary@google.com> |
Style bikeshed - remove extraneous whitespace
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1842753002
Review URL: https://codereview.chromium.org/1842753002
/external/skia/gm/beziereffects.cpp
|
| 391395dcfbff09a83f8f0e9d3e02d38c855ae2e9 |
|
02-Mar-2016 |
robertphillips <robertphillips@google.com> |
Begin weaning GrClipMaskManager off of GrDrawTarget (take 2)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1754563003
Committed: https://skia.googlesource.com/skia/+/86c60758e9e4f9e203d7462cb22b2a245a0f51bd
Review URL: https://codereview.chromium.org/1754563003
/external/skia/gm/beziereffects.cpp
|
| 8b8f36fc49b88ed369523730a4f1f26bb2094a75 |
|
02-Mar-2016 |
robertphillips <robertphillips@google.com> |
Revert of Begin weaning GrClipMaskManager off of GrDrawTarget (take 2) (patchset #7 id:120001 of https://codereview.chromium.org/1754563003/ )
Reason for revert:
fAuditTrail
Original issue's description:
> Begin weaning GrClipMaskManager off of GrDrawTarget (take 2)
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1754563003
>
> Committed: https://skia.googlesource.com/skia/+/86c60758e9e4f9e203d7462cb22b2a245a0f51bd
TBR=joshualitt@chromium.org,bsalomon@google.com,joshualitt@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1754353002
/external/skia/gm/beziereffects.cpp
|
| 86c60758e9e4f9e203d7462cb22b2a245a0f51bd |
|
02-Mar-2016 |
robertphillips <robertphillips@google.com> |
Begin weaning GrClipMaskManager off of GrDrawTarget (take 2)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1754563003
Review URL: https://codereview.chromium.org/1754563003
/external/skia/gm/beziereffects.cpp
|
| f5883a6d75a18c581507214d3057b5c721ed1ffc |
|
13-Jan-2016 |
joshualitt <joshualitt@chromium.org> |
Hide drawBatch on GrDrawTarget from test Methods
TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1574383002
Review URL: https://codereview.chromium.org/1574383002
/external/skia/gm/beziereffects.cpp
|
| 144c3c8b7ff3ebc389b41211f3388fb24a7ff0c2 |
|
30-Nov-2015 |
joshualitt <joshualitt@chromium.org> |
Make onPrepareDraws const
BUG=skia:
Review URL: https://codereview.chromium.org/1483103003
/external/skia/gm/beziereffects.cpp
|
| c4b72720e75313079212e69e46a5ef7c474b2305 |
|
23-Nov-2015 |
egdaniel <egdaniel@google.com> |
Don't create a GXPFactory when blend is SrcOver
BUG=skia:
Review URL: https://codereview.chromium.org/1471053002
/external/skia/gm/beziereffects.cpp
|
| 504ce5dc772e14be2f5697b00ac82fcf82127763 |
|
16-Nov-2015 |
robertphillips <robertphillips@google.com> |
Optionally pass rendertarget to getTestTarget
This shouldn't really make any difference but allocating and holding on to a GrRenderTarget for each test target generates image differences for Mali GPUs. This CL allows an existing render target to be used for the test target.
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/1447113002
/external/skia/gm/beziereffects.cpp
|
| a13e202563979fd5076936606dcc1d660da8c632 |
|
11-Nov-2015 |
robertphillips <robertphillips@google.com> |
Move GrBatchFlushState from GrDrawTarget to GrDrawingManager
This CL:
moves the flushState
disables immediate mode (it was proving difficult to implement)
also moves the program unit test to the drawing manager
BUG=skia:4094
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/1437843002
/external/skia/gm/beziereffects.cpp
|
| 1b55a963a2374a14bb82eb887bb99ee91680f0eb |
|
18-Sep-2015 |
reed <reed@chromium.org> |
Reland of add a ClassID function to GrBatch (patchset #1 id:1 of https://codereview.chromium.org/1353043002/ )
Reason for revert:
DEPS roll seems to have landed w/o this revert, so revert it again
Original issue's description:
> Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )
>
> Reason for revert:
> Speculative revert to unblock DEPS roll
>
> Original issue's description:
> > add a ClassID function to GrBatch
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
> >
> > Committed: https://skia.googlesource.com/skia/+/eb44d53cf96a7eaf103a98d76079ce1f5495e343
>
> TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@google.com,joshualitt@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/48e3a45ade15c52c0c1a10cb00907dd444897745
TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1344373005
/external/skia/gm/beziereffects.cpp
|
| 48e3a45ade15c52c0c1a10cb00907dd444897745 |
|
18-Sep-2015 |
reed <reed@chromium.org> |
Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )
Reason for revert:
Speculative revert to unblock DEPS roll
Original issue's description:
> add a ClassID function to GrBatch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
>
> Committed: https://skia.googlesource.com/skia/+/eb44d53cf96a7eaf103a98d76079ce1f5495e343
TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1353043002
/external/skia/gm/beziereffects.cpp
|
| eb44d53cf96a7eaf103a98d76079ce1f5495e343 |
|
17-Sep-2015 |
joshualitt <joshualitt@chromium.org> |
add a ClassID function to GrBatch
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
Review URL: https://codereview.chromium.org/1352813003
/external/skia/gm/beziereffects.cpp
|
| a7008403dcd03302e88e2df546d8427afe9e0e80 |
|
17-Sep-2015 |
joshualitt <joshualitt@google.com> |
Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )
Reason for revert:
breaks mac bot
Original issue's description:
> add a ClassID function to GrBatch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1345393003
/external/skia/gm/beziereffects.cpp
|
| 4078d529e9e199eea13456db7bf3a63a104ab5b9 |
|
17-Sep-2015 |
joshualitt <joshualitt@chromium.org> |
add a ClassID function to GrBatch
BUG=skia:
Review URL: https://codereview.chromium.org/1352813003
/external/skia/gm/beziereffects.cpp
|
| 2a24338c777462e04a2b26295f9c034155ee8f3e |
|
09-Sep-2015 |
halcanary <halcanary@google.com> |
GM: replace boilerplate with macros
I have verified locally that nothing draws differently.
Motivation:
* SK_SIMPLE_GM makes it easier to write a GM.
* Reducing 1100 lines of code makes maintenance easier.
* Simple GMs are easy to convert to Fiddles.
Review URL: https://codereview.chromium.org/1333553002
/external/skia/gm/beziereffects.cpp
|
| 96fcdcc219d2a0d3579719b84b28bede76efba64 |
|
27-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002
Review URL: https://codereview.chromium.org/1316233002
/external/skia/gm/beziereffects.cpp
|
| 385fe4d4b62d7d1dd76116dd570df3290a2f487b |
|
26-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: SkNEW->new; SkDELETE->delete
DOCS_PREVIEW= https://skia.org/?cl=1316123003
Review URL: https://codereview.chromium.org/1316123003
/external/skia/gm/beziereffects.cpp
|
| 7539856c1b9cbb1886a6a498cc534b77fc83ddb2 |
|
17-Aug-2015 |
bsalomon <bsalomon@google.com> |
Make GrVertexBatch objects hold their own draws during GrDrawTarget flush
NO_MERGE_BUILDS
Review URL: https://codereview.chromium.org/1286043004
/external/skia/gm/beziereffects.cpp
|
| abd30f54b7ff1704a8930c4307ea242d09425d02 |
|
13-Aug-2015 |
bsalomon <bsalomon@google.com> |
Introduce GrBatch subclasses GrDrawBatch and GrVertexBatch to prepare for non-drawing batches
Review URL: https://codereview.chromium.org/1293583002
/external/skia/gm/beziereffects.cpp
|
| 2771b5617c0440de2e8b3a024e367553e8a99d70 |
|
07-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
GrDrawVertices to batches
BUG=skia:
Review URL: https://codereview.chromium.org/1276333004
/external/skia/gm/beziereffects.cpp
|
| fb1141ae1db2ffce479bc46413b008d31cf9d3cc |
|
06-Aug-2015 |
bsalomon <bsalomon@google.com> |
Don't pass pipeline to GrBatch::generateGeometry()
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/1275083002
/external/skia/gm/beziereffects.cpp
|
| dbe1e6f0c32bd07b7669b1b1ac3c7f58c9b8d773 |
|
16-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
Give GrBatch a pointer to GrPipeline
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/1170a12839218f7a23c93487bf95fd83aae0201f
Review URL: https://codereview.chromium.org/1237283007
/external/skia/gm/beziereffects.cpp
|
| 0df62e3d685f6b0b61ee514c68006e013109f5a1 |
|
16-Jul-2015 |
joshualitt <joshualitt@google.com> |
Revert of Give GrBatch a pointer to GrPipeline (patchset #1 id:1 of https://codereview.chromium.org/1237283007/)
Reason for revert:
breaking nanobench
Original issue's description:
> Give GrBatch a pointer to GrPipeline
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/1170a12839218f7a23c93487bf95fd83aae0201f
TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1239073002
/external/skia/gm/beziereffects.cpp
|
| 1170a12839218f7a23c93487bf95fd83aae0201f |
|
16-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
Give GrBatch a pointer to GrPipeline
BUG=skia:
Review URL: https://codereview.chromium.org/1237283007
/external/skia/gm/beziereffects.cpp
|
| 1c735488cbec9abdb25c9a12e0c11af9c302a776 |
|
13-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
Start to propagate constness of GrPipelineBuilder up the stack
BUG=skia:
Review URL: https://codereview.chromium.org/1229143007
/external/skia/gm/beziereffects.cpp
|
| b8c241ad099f3f0c2cbf3e7c10f5f6207175d490 |
|
19-May-2015 |
joshualitt <joshualitt@chromium.org> |
Preliminary attempt to remove batch tracker
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84
Review URL: https://codereview.chromium.org/1139723004
/external/skia/gm/beziereffects.cpp
|
| d3a560fa80bfb3e2d2e989f951bb3b1c52316654 |
|
19-May-2015 |
joshualitt <joshualitt@google.com> |
Revert of Preliminary attempt to remove batch tracker (patchset #3 id:40001 of https://codereview.chromium.org/1139723004/)
Reason for revert:
breaking bots
Original issue's description:
> Preliminary attempt to remove batch tracker
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84
TBR=robertphillips@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1132323003
/external/skia/gm/beziereffects.cpp
|
| cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84 |
|
19-May-2015 |
joshualitt <joshualitt@chromium.org> |
Preliminary attempt to remove batch tracker
BUG=skia:
Review URL: https://codereview.chromium.org/1139723004
/external/skia/gm/beziereffects.cpp
|
| 88c23fc6e8ed0243159fd17d815b4f813805647b |
|
13-May-2015 |
joshualitt <joshualitt@chromium.org> |
remove color from GrGeometryProcessor
BUG=skia:
Review URL: https://codereview.chromium.org/1140983002
/external/skia/gm/beziereffects.cpp
|
| e64eb570a5b9480bc24d0656ccabcff1ab13a229 |
|
07-May-2015 |
bsalomon <bsalomon@google.com> |
Iterate over instanced draws in GrGpu rather than above GrBatchTarget
Review URL: https://codereview.chromium.org/1127273007
/external/skia/gm/beziereffects.cpp
|
| e40d39725908d99d255496025411e5c9ca32663e |
|
07-May-2015 |
robertphillips <robertphillips@google.com> |
Isolate GrBufferAllocPools inside GrBatchTarget
This CL refactors the location of the GrBufferAllocPools so they reside entirely inside the GrBatchTarget. This is in preparation for making them use scratch resources.
Review URL: https://codereview.chromium.org/1131553002
/external/skia/gm/beziereffects.cpp
|
| cb8979d088a66ebaf41f10ba6f5c830615aa0e03 |
|
05-May-2015 |
bsalomon <bsalomon@google.com> |
Move DrawInfo out from GrDrawTarget and rename to GrVertices.
Review URL: https://codereview.chromium.org/1124733004
/external/skia/gm/beziereffects.cpp
|
| b5238a7571c243ba4a154a62575570c3078b3741 |
|
05-May-2015 |
bsalomon <bsalomon@google.com> |
Start on simplifying generateGeometry() overrides
Committed: https://skia.googlesource.com/skia/+/f28381c6866cad92af8ebe5b9d2db074613b1963
Review URL: https://codereview.chromium.org/1122673002
/external/skia/gm/beziereffects.cpp
|
| 8415abe44cf205ac214b7793c076fd8c13272240 |
|
04-May-2015 |
bsalomon <bsalomon@google.com> |
Revert of Start on simplifying generateGeometry() overrides (patchset #10 id:160001 of https://codereview.chromium.org/1122673002/)
Reason for revert:
Breaking bots
Original issue's description:
> Start on simplifying generateGeometry() overrides
>
> Committed: https://skia.googlesource.com/skia/+/f28381c6866cad92af8ebe5b9d2db074613b1963
TBR=joshualitt@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1124633003
/external/skia/gm/beziereffects.cpp
|
| f28381c6866cad92af8ebe5b9d2db074613b1963 |
|
04-May-2015 |
bsalomon <bsalomon@google.com> |
Start on simplifying generateGeometry() overrides
Review URL: https://codereview.chromium.org/1122673002
/external/skia/gm/beziereffects.cpp
|
| ed0bcad9c8147fd37c23bdda00ec27ec9ef8d66b |
|
04-May-2015 |
bsalomon <bsalomon@google.com> |
Move instanced index buffer creation to flush time
Committed: https://skia.googlesource.com/skia/+/ab622c7b8cc8c39f0a594e4392b9e31b7e1ddb26
Review URL: https://codereview.chromium.org/1116943004
/external/skia/gm/beziereffects.cpp
|
| d8ed1b64a6c85a8364c23e825164b7bebb43a3a0 |
|
04-May-2015 |
bsalomon <bsalomon@google.com> |
Revert of Move instanced index buffer creation to flush time (patchset #6 id:100001 of https://codereview.chromium.org/1116943004/)
Reason for revert:
messed up caching, recreating index buffers all the time.
Original issue's description:
> Move instanced index buffer creation to flush time
>
> Committed: https://skia.googlesource.com/skia/+/ab622c7b8cc8c39f0a594e4392b9e31b7e1ddb26
TBR=joshualitt@google.com,robertphillips@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1126613003
/external/skia/gm/beziereffects.cpp
|
| ab622c7b8cc8c39f0a594e4392b9e31b7e1ddb26 |
|
04-May-2015 |
bsalomon <bsalomon@google.com> |
Move instanced index buffer creation to flush time
Review URL: https://codereview.chromium.org/1116943004
/external/skia/gm/beziereffects.cpp
|
| 99c7c07e0f1f7b78980eb21d84bebda8b45a7178 |
|
01-May-2015 |
joshualitt <joshualitt@chromium.org> |
Move bounds to GrBatch
BUG=skia:
Review URL: https://codereview.chromium.org/1121463002
/external/skia/gm/beziereffects.cpp
|
| 36352bf5e38f45a70ee4f4fc132a38048d38206d |
|
26-Mar-2015 |
mtklein <mtklein@chromium.org> |
C++11 override should now be supported by all of {bots,Chrome,Android,Mozilla}
NOPRESUBMIT=true
BUG=skia:
DOCS_PREVIEW= https://skia.org/?cl=1037793002
Review URL: https://codereview.chromium.org/1037793002
/external/skia/gm/beziereffects.cpp
|
| 4b31de8328bbf3ee789157ae1dc6fe7cc74c796a |
|
05-Mar-2015 |
joshualitt <joshualitt@chromium.org> |
handle null vertex or index buffers in batch
BUG=skia:
Review URL: https://codereview.chromium.org/979343002
/external/skia/gm/beziereffects.cpp
|
| 44701df5ce572ac3cccec785cf52103d3d5d14a5 |
|
23-Feb-2015 |
joshualitt <joshualitt@chromium.org> |
Move clip off of draw target
BUG=skia:
Review URL: https://codereview.chromium.org/947443003
/external/skia/gm/beziereffects.cpp
|
| 95964c670b577d5f49c5eab7af277ad987fd0519 |
|
11-Feb-2015 |
joshualitt <joshualitt@chromium.org> |
GMs now use batch
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/3f284d7758d7f35b59d93a22d126f7cd8423be44
Review URL: https://codereview.chromium.org/865313004
/external/skia/gm/beziereffects.cpp
|
| 94dff15404a6c82873f3f5092023426937274cc1 |
|
11-Feb-2015 |
joshualitt <joshualitt@google.com> |
Revert of GMs now use batch (patchset #3 id:40001 of https://codereview.chromium.org/865313004/)
Reason for revert:
missing hairlines on gms
Original issue's description:
> GMs now use batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3f284d7758d7f35b59d93a22d126f7cd8423be44
TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/913153003
/external/skia/gm/beziereffects.cpp
|
| 3f284d7758d7f35b59d93a22d126f7cd8423be44 |
|
11-Feb-2015 |
joshualitt <joshualitt@chromium.org> |
GMs now use batch
BUG=skia:
Review URL: https://codereview.chromium.org/865313004
/external/skia/gm/beziereffects.cpp
|
| b62da80cbf6643c1944d3ebf8df233f286bd3fde |
|
31-Jan-2015 |
bsalomon <bsalomon@google.com> |
Add standard way to indicate GM is GPU-only.
TBR=egdaniel@google.com,mtklein@google.com
Review URL: https://codereview.chromium.org/869393007
/external/skia/gm/beziereffects.cpp
|
| 1c4029296f518a84ef90095243ba210163a1e1f9 |
|
23-Jan-2015 |
mtklein <mtklein@chromium.org> |
remove unused GM flags
Depends on https://codereview.chromium.org/873753002/
Thumbs up to CLion for refactoring this for me.
BUG=skia:
Review URL: https://codereview.chromium.org/867963004
/external/skia/gm/beziereffects.cpp
|
| 8dd688b7569df569a672a8a67b2db86a9d376cfc |
|
22-Jan-2015 |
egdaniel <egdaniel@google.com> |
Rename GrOptDrawState to GrPipeline and GrDrawState to GrPipelineBuilder
BUG=skia:
Review URL: https://codereview.chromium.org/858343002
/external/skia/gm/beziereffects.cpp
|
| 72c9faab45124e08c85f70ca38536914862d947c |
|
09-Jan-2015 |
mtklein <mtklein@chromium.org> |
Fix up all the easy virtual ... SK_OVERRIDE cases.
This fixes every case where virtual and SK_OVERRIDE were on the same line,
which should be the bulk of cases. We'll have to manually clean up the rest
over time unless I level up in regexes.
for f in (find . -type f); perl -p -i -e 's/virtual (.*)SK_OVERRIDE/\1SK_OVERRIDE/g' $f; end
BUG=skia:
Review URL: https://codereview.chromium.org/806653007
/external/skia/gm/beziereffects.cpp
|
| 8059eb9f6e24ed609393fbda4ad71edea03ac258 |
|
30-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Move ViewMatrix off of drawstate
BUG=skia:
Review URL: https://codereview.chromium.org/815553003
/external/skia/gm/beziereffects.cpp
|
| d27f73ef27ff65a6a0a5d00aa8e5b784b1a0b47e |
|
29-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove coordchanges from drawstate
BUG=skia:
Review URL: https://codereview.chromium.org/818233002
/external/skia/gm/beziereffects.cpp
|
| 56995b5cc00c9c83bd5fcf86bca9a67e939a96cb |
|
12-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da
Review URL: https://codereview.chromium.org/791743003
/external/skia/gm/beziereffects.cpp
|
| 8c0f615fdd41b8b8048bf398791bb2138c511826 |
|
10-Dec-2014 |
joshualitt <joshualitt@google.com> |
Revert of Remove GP from drawstate, revision of invariant output for GP (patchset #9 id:160001 of https://codereview.chromium.org/791743003/)
Reason for revert:
breaks mac
Original issue's description:
> Remove GP from drawstate, revision of invariant output for GP
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da
TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/794843002
/external/skia/gm/beziereffects.cpp
|
| c6bc58eded89b0c0a36b8e20e193c200f297a0da |
|
10-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP
BUG=skia:
Review URL: https://codereview.chromium.org/791743003
/external/skia/gm/beziereffects.cpp
|
| 2e3b3e369d79e78f7635d4c20e83a47ab571bdf2 |
|
09-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
This cl moves color and coverage off of drawstate. In an effort to keep this CL manageable, I have left the compute invariant input / output in a bit of a strange state(fixing this will be complicated).
In addition, NVPR makes this very complicated, and I haven't quite figured out a good way to handle it, so for now color and coverage DO live on optstate, but I will figure out some way to refactor that in future CLs.
BUG=skia:
Review URL: https://codereview.chromium.org/783763002
/external/skia/gm/beziereffects.cpp
|
| 2dd1ae016d7f297b433c3ea3a771ef8e01657c1f |
|
03-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
First step to moving vertex attributes to the geometryProcessor
BUG=skia:
Review URL: https://codereview.chromium.org/761563002
/external/skia/gm/beziereffects.cpp
|
| 9853ccef19c200be93a6211f32589fa82a53067c |
|
17-Nov-2014 |
joshualitt <joshualitt@chromium.org> |
Drawstate on stack
BUG=skia:
Review URL: https://codereview.chromium.org/732693002
/external/skia/gm/beziereffects.cpp
|
| 50408adfd23994cdbb5c531ff6366e4e9d29ee6d |
|
03-Nov-2014 |
joshualitt <joshualitt@chromium.org> |
Test factory uses the inorder draw buffer
BUG=skia:
Review URL: https://codereview.chromium.org/696353002
/external/skia/gm/beziereffects.cpp
|
| b0a8a377f832c59cee939ad721e1f87d378b7142 |
|
23-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
Patch to create a distinct geometry processor. The vast majority of this patch
is just a rename. The meat is in GrGeometryProcessor, GrProcessor,
GrGL*Processor, GrProcessorStage, Gr*BackendProcessorFactory,
GrProcessUnitTestFactory, and the builders
BUG=skia:
R=bsalomon@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/582963002
/external/skia/gm/beziereffects.cpp
|
| 249af15fb82833d2274850c589812b6e69df0033 |
|
15-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
BUG=skia:
R=bsalomon@google.com, egdaniel@google.com, jvanverth@google.com, robertphillips@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/543623004
/external/skia/gm/beziereffects.cpp
|
| 49f085dddff10473b6ebf832a974288300224e60 |
|
05-Sep-2014 |
bsalomon <bsalomon@google.com> |
"NULL !=" = NULL
R=reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/544233002
/external/skia/gm/beziereffects.cpp
|
| bd769d0f1c8cf6ccbb2738dfad1624a4c828e4eb |
|
04-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
Initial change to create GeometryProcessor
BUG=skia:
R=bsalomon@google.com, robertphillips@google.com, egdaniel@google.com, jvanverth@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/509153002
/external/skia/gm/beziereffects.cpp
|
| 7b3d5ee72c2238aa239bce4d5b3aea98a437ca7a |
|
28-Aug-2014 |
egdaniel <egdaniel@google.com> |
Make setVertexAttribs in GrDrawState take a stride parameter.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/af013bf8afc4c77ab8ff230f536e2ade973427fa
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/511593004
/external/skia/gm/beziereffects.cpp
|
| ea81ced1e573c98584e6830a92511f80593af0a4 |
|
27-Aug-2014 |
djsollen <djsollen@google.com> |
Revert of Make setVertexAttribs in GrDrawState take a stride parameter. (patchset #5 of https://codereview.chromium.org/511593004/)
Reason for revert:
GM failures on http://108.170.220.120:10115/builders/Test-Ubuntu12-ShuttleA-GTX660-x86-Release/builds/1867/
Original issue's description:
> Make setVertexAttribs in GrDrawState take a stride parameter.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/af013bf8afc4c77ab8ff230f536e2ade973427fa
R=bsalomon@google.com, egdaniel@google.com
TBR=bsalomon@google.com, egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/502533004
/external/skia/gm/beziereffects.cpp
|
| af013bf8afc4c77ab8ff230f536e2ade973427fa |
|
27-Aug-2014 |
egdaniel <egdaniel@google.com> |
Make setVertexAttribs in GrDrawState take a stride parameter.
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/511593004
/external/skia/gm/beziereffects.cpp
|
| 83d081ae1d731b5039e99823620f5e287542ee39 |
|
08-Jul-2014 |
bsalomon <bsalomon@google.com> |
Goodbye GrEffectRef.
Also, reworked some var names and comments around SkShader::asNewEffect.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/374923002
/external/skia/gm/beziereffects.cpp
|
| f539318f0d3dba743ec1886d5d9df0fb1be628a1 |
|
10-Jun-2014 |
tfarina <tfarina@chromium.org> |
Cleanup: Get rid of make_isize() function from gm.h.
This helper function is not necessary.
The same thing can be achieved by using SkISize::Make() provided by
SkTSize API.
BUG=skia:2645
TEST=make dm && out/Debug/dm
R=robertphillips@google.com, reed@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/326523002
/external/skia/gm/beziereffects.cpp
|
| 972f9cd7a063d0544f8c919fd12b9a3adbd12b24 |
|
28-Mar-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
SK_SUPPORT_LEGACY_GRTYPES to hide duplicate types from SkTypes.h
BUG=skia:
R=bsalomon@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/216503004
git-svn-id: http://skia.googlecode.com/svn/trunk@13982 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| 9c135db83d198e7d8200027c7d2cf60f38517ee3 |
|
12-Mar-2014 |
reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
support direct writing to top layer, and hide getTopLayer()
this should remove many of the chrome callers that today call
accessBitmap on the toplayer, so they can read/write those pixels.
The ultimate fix will be to support custom allocation of raster layers
(via GDI/cairo/mac) so we can remove PlatformDevice subclassing in
skia/ext
BUG=skia:
R=bsalomon@google.com, scroggo@google.com
Review URL: https://codereview.chromium.org/197433002
git-svn-id: http://skia.googlecode.com/svn/trunk@13774 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| cabf4b2f3664b98c1084fbb94a999af15ddfb52d |
|
05-Mar-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Unify edge type enums across GrEffect subclasses that clip rendering to a geometry.
BUG=skia:
R=egdaniel@google.com, robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/183893023
git-svn-id: http://skia.googlecode.com/svn/trunk@13674 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| c3fe54975daf6274103bcfefe5ed2e7af8d0170a |
|
30-Jan-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add convex polygon rendering effect and GM to test it.
BUG=skia:2051
R=robertphillips@google.com, jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/149683004
git-svn-id: http://skia.googlecode.com/svn/trunk@13242 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| e0e7cfe44bb9d66d76120a79e5275c294bacaa22 |
|
09-Sep-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Change old PRG to be SkLCGRandom; change new one to SkRandom
The goal here is to get people to start using the new random number
generator, while leaving the old one in place so we don't have to
rebaseline GMs.
R=reed@google.com, bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://chromiumcodereview.appspot.com/23576015
git-svn-id: http://skia.googlecode.com/svn/trunk@11169 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| 1f2f338e23789f3eef168dcbd8171a28820ba6c1 |
|
29-Aug-2013 |
robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Split SkDevice into SkBaseDevice and SkBitmapDevice
https://codereview.chromium.org/22978012/
git-svn-id: http://skia.googlecode.com/svn/trunk@10995 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| 53a0b6cc191639925798c01e9bce853723517419 |
|
23-Aug-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Add GMs for gpu Bezier shaders
BUG=
R=bsalomon@google.com, jvanverth@google.com, robertphillips@google.com
Author: egdaniel@google.com
Review URL: https://chromiumcodereview.appspot.com/23361024
git-svn-id: http://skia.googlecode.com/svn/trunk@10892 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|
| 78a1078f17f4f0ae63415298517262a64f706af6 |
|
21-Aug-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Create new target to hold gpu test code, enable direct testing of GrEffects in GM.
R=robertphillips@google.com, jvanverth@google.com, egdaniel@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/23352003
git-svn-id: http://skia.googlecode.com/svn/trunk@10866 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/gm/beziereffects.cpp
|