fbcef6eb8abad142daf45418516550f7635b4a52 |
|
15-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Clean up GrResourceProvider usage The only substantive changes are the removal of GrProxy instantiation in: SkGpuBlurUtils::GaussianBlur GrSimpleTextureEffect::Make* Change-Id: I10970609693bd6ff5b3a3c21b41d82642bb277bc Reviewed-on: https://skia-review.googlesource.com/19965 Reviewed-by: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
e3a0be73a61147379ab0ce33a0e773c072c47908 |
|
12-Jun-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator: fix two NaN issues. If a point in the path rounds to +inf/-inf, the intersection code can produce NaN, which is unsortable. Fix: ignore non-finite intersections. Quadratic interpolation can sometimes produce NaN, which will never satisfy the flatness criterion. Abort if any of the interpolated points are non-finite. Bug:732023 Change-Id: If5881796e589c75b8f74459f42d00918619713a2 Reviewed-on: https://skia-review.googlesource.com/19467 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
019f6c0bcf611fec93913315708eba1633857efa |
|
09-Jun-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator: fix logging; fix test comment. Change-Id: Iaded9025a1518d8c69dbe366deb1035e0bd4295b Reviewed-on: https://skia-review.googlesource.com/19289 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
0cb31675f3aa9cc878aa3a9e1a7ad09efdb350fc |
|
08-Jun-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator: fix intersection above the first vertex. Handle the case where the an intersection falls not only above both edge endpoints, but above the first vertex in the mesh. This requires passing the mesh into check_for_intersection(), in order to modify the head. We also need to rewind the mesh after insertion, since we need to rewind to the newly-inserted vertex. This also cleans up vertex ID computation a little (for logging), so that vertices before the first vertex or after the last have a reasonable ID. It also cleans up the intersection-on-endpoint special cases by refactoring the calls to split_edge(). BUG=730687 Change-Id: Idea736eca7b7c3c5d8a470b1373a16ad8e649e80 Reviewed-on: https://skia-review.googlesource.com/19069 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
6eca90fea6548bfb2f5c30f391cc0caa6b1de04f |
|
25-May-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator: fix for three edges becoming collinear after splitting. In rare cases, a single edge can become collinear with two adjacent neighbour edges after it's split. The solution is to continue to merge until no collinear edges are found. BUG=722000 Change-Id: Ia5dd212b7acfb40ed1d6c74ebfa9e4a4746fe40a Reviewed-on: https://skia-review.googlesource.com/17963 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
4aed13889bd9085337e0d4c20df28686687b833b |
|
25-May-2017 |
Florin Malita <fmalita@chromium.org> |
Reland of SkShaderBase Introduce a private base class (SkShaderBase), to hide implementation details from the public interface (SkShader). Change-Id: Ib1d76cde880bd51868b97408710f8bb38128e536 Reviewed-on: https://skia-review.googlesource.com/17925 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
d93e11ceb427ff433d219081b74d10b8757d0b03 |
|
24-May-2017 |
Florin Malita <fmalita@chromium.org> |
Revert "SkShaderBase" This reverts commit 58a756435ca3700d9766a4580bb0771a9774f603. Reason for revert: g3, Android borkage. Original change's description: > SkShaderBase > > Introduce a private base class (SkShaderBase), to hide > implementation details from the public interface (SkShader). > > Change-Id: If3ec26ca6abc9da20e3f139c11fdc023bdd85176 > Reviewed-on: https://skia-review.googlesource.com/17241 > Commit-Queue: Florin Malita <fmalita@chromium.org> > Reviewed-by: Mike Reed <reed@google.com> > TBR=mtklein@google.com,fmalita@chromium.org,reed@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I32b012ee466dd006c074593f211f43ed602f1078 Reviewed-on: https://skia-review.googlesource.com/17845 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
58a756435ca3700d9766a4580bb0771a9774f603 |
|
24-May-2017 |
Florin Malita <fmalita@chromium.org> |
SkShaderBase Introduce a private base class (SkShaderBase), to hide implementation details from the public interface (SkShader). Change-Id: If3ec26ca6abc9da20e3f139c11fdc023bdd85176 Reviewed-on: https://skia-review.googlesource.com/17241 Commit-Queue: Florin Malita <fmalita@chromium.org> Reviewed-by: Mike Reed <reed@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
64dbb89efc9daba9f62b4a385354a77aaa54861e |
|
03-May-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator: fix for vertex coincident with enclosing edge. If a previously-enclosing edge coincides exactly with the current vertex, there are no two adjacent edges which enclose the vertex. Since find_enclosing_edges() ensures that the left enclosing edge is to the left of the vertex, the fix is to split the right enclosing edge on the current vertex and restart intersection tests. Bug: 716720 Change-Id: Id26c5b92a6d6139f348e99554638cded37e81a8e Reviewed-on: https://skia-review.googlesource.com/15261 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
dd3b3f41829d32d7eaf3eb4903570d49c2ba9ff8 |
|
24-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Rm makeRenderTargetContext in favor of deferred version (take 3) This is a reland of: https://skia-review.googlesource.com/c/13001/ (Rm makeRenderTargetContext in favor of deferred version) TBR=bsalomon@google.com Change-Id: If81f4d9fb889c091cd37ffde133d906fb3e37773 Reviewed-on: https://skia-review.googlesource.com/14027 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
2af746c1e74af30d0fb02d5c35a178e61d152965 |
|
21-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Revert "Rm makeRenderTargetContext in favor of deferred version (take 2)" This reverts commit 02242e82e4bf94a8f6862dadb5bf347ea5e31eb5. Reason for revert: Maybe breaking Chrome DEPS roll Original change's description: > Rm makeRenderTargetContext in favor of deferred version (take 2) > > This is a reland of: https://skia-review.googlesource.com/c/13001/ (Rm makeRenderTargetContext in favor of deferred version) > > Change-Id: Ife77b012d09c46895884a168fc5045bd92a4b919 > Reviewed-on: https://skia-review.googlesource.com/13196 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Robert Phillips <robertphillips@google.com> > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I2607116ed743f5d313da4a7b7f056776ed907702 Reviewed-on: https://skia-review.googlesource.com/14024 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
02242e82e4bf94a8f6862dadb5bf347ea5e31eb5 |
|
20-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Rm makeRenderTargetContext in favor of deferred version (take 2) This is a reland of: https://skia-review.googlesource.com/c/13001/ (Rm makeRenderTargetContext in favor of deferred version) Change-Id: Ife77b012d09c46895884a168fc5045bd92a4b919 Reviewed-on: https://skia-review.googlesource.com/13196 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
934292170bade335f972a050a2f64d2cca9ca299 |
|
11-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Revert "Rm makeRenderTargetContext in favor of deferred version" This reverts commit d83ec0441279b79551b4ecb4598696249ea55145. Reason for revert: various bot failures Original change's description: > Rm makeRenderTargetContext in favor of deferred version > > Change-Id: Ifdf67453607256ba7cdb0837263377bd83907212 > Reviewed-on: https://skia-review.googlesource.com/13001 > Reviewed-by: Brian Salomon <bsalomon@google.com> > Commit-Queue: Robert Phillips <robertphillips@google.com> > TBR=bsalomon@google.com,robertphillips@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I4e67a20cbe658b24725dcf386ac8bfdd127647ea Reviewed-on: https://skia-review.googlesource.com/13131 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
d83ec0441279b79551b4ecb4598696249ea55145 |
|
10-Apr-2017 |
Robert Phillips <robertphillips@google.com> |
Rm makeRenderTargetContext in favor of deferred version Change-Id: Ifdf67453607256ba7cdb0837263377bd83907212 Reviewed-on: https://skia-review.googlesource.com/13001 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
c65aec97619682b2c0191554f44ddf35f618a94d |
|
09-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Make GrClip.h private BUG=skia: Change-Id: I4aa3c2707811ece3a63d161035e316c3bbc1cd15 Reviewed-on: https://skia-review.googlesource.com/9482 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
256c37bc9ea2a0420b8ac1084f6d645aaeb919f0 |
|
01-Mar-2017 |
Robert Phillips <robertphillips@google.com> |
Remove atlas creation from GrResourceProvider This is pulled out of: https://skia-review.googlesource.com/c/6680/ (Make SkImage_Gpu be deferred) and is only tangentially related to the goal of that CL. Change-Id: I6b6db4869597070f85ab3b9fea178fc88c104f87 Reviewed-on: https://skia-review.googlesource.com/9106 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
cc70083fbf76fd86b901fffdcce40d0f53787fc8 |
|
15-Feb-2017 |
Stephen White <senorblanco@chromium.org> |
GrTessellator (AA): null GrGeometryProcessor crash fix. Add a null-check on geometry processor creation. Add a test which exercises it: AA tessellator, with a non-invertible matrix and a fragment processor which needs local coords (e.g., linear gradient). BUG=691902 Change-Id: I005b893aed58d3ad2500c41501045ac94b0b4b95 Reviewed-on: https://skia-review.googlesource.com/8462 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
77505da16093695f717ad34ff100ec11ffb2835b |
|
20-Jan-2017 |
Robert Phillips <robertphillips@google.com> |
Remove GrTestTarget Change-Id: I4d2e1aa54732ff75b44a197255035c0e04548219 Reviewed-on: https://skia-review.googlesource.com/7327 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.cpp
|
777b5633f599f2a99e2035fdb7ab600779ab95ac |
|
14-Oct-2016 |
Brian Osman <brianosman@google.com> |
Remove many uses of kSkia8888 and kSkiaGamma8888 BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3405 Change-Id: Ie920f4bcbb3dd28114a66f2363643bb10009b840 Reviewed-on: https://skia-review.googlesource.com/3405 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/tests/TessellatingPathRendererTests.cpp
|
70f5251cc59191040a14cd0f1567aa2129e1f7c6 |
|
17-Aug-2016 |
senorblanco <senorblanco@chromium.org> |
Fix assert caused by floating point error in tessellating path renderer. In rare cases, floating point error causes the tesselator to add the same Vertex to more than one Poly on the same side. This was not a big problem when we were allocating new vertices when constructing Polys, but after https://codereview.chromium.org/2029243002 it causes more serious issues, since each Edge can only belong to two Polys, and violating this condition messes up the linked list of Edges used for left & right Polys and the associated estimated vertex count. The fix is to simply let the first Poly win, and skip that vertex for subsequent Polys. Since this only occurs in cases where vertices are very close to each other, it should have little visual effect. This is also exercised by Nebraska-StateSeal.svg. BUG=skia:5636 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2259493002 Review-Url: https://codereview.chromium.org/2259493002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
6738c702423226619ee0172c12a3a007c5f68e57 |
|
27-Jul-2016 |
robertphillips <robertphillips@google.com> |
Rename GrContext's newDrawContext & drawContext to makeDrawContext These both return sk_sp. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2186073002 Review-Url: https://codereview.chromium.org/2186073002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
dfe4f2e4fe5b162d4adb4486fe751f1e3b30bea7 |
|
21-Jul-2016 |
brianosman <brianosman@google.com> |
Add SkColorSpace to GrDrawContext BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2164363002 Review-Url: https://codereview.chromium.org/2164363002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
3950f0d7239c05687ac96c211e49ceab9ab9ff4d |
|
07-Jul-2016 |
robertphillips <robertphillips@google.com> |
Remove fColor from PathRenderer DrawPathArgs struct This was getting in the way of other refactorings. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2126083002 Review-Url: https://codereview.chromium.org/2126083002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
8acedde5970ce70de6d9791ffeda87a65af4ed07 |
|
24-Jun-2016 |
bsalomon <bsalomon@google.com> |
Use GrShape in GrPathRenderer. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2081383006 Review-Url: https://codereview.chromium.org/2081383006
/external/skia/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.cpp
|
976f5f0dc5e907d1ca50685fad117bd15d7fc87b |
|
03-Jun-2016 |
robertphillips <robertphillips@google.com> |
Make GrClipMaskManager stateless and push GrPipelineBuilder construction downstack This will be followed up with a CL to remove the GrRenderTarget from the GrPipelineBuilder. Split out of: https://codereview.chromium.org/1988923002/ (Declassify GrClipMaskManager and Remove GrRenderTarget and GrDrawTarget from GrPipelineBuilder) GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2035823002 Review-Url: https://codereview.chromium.org/2035823002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
87f15c8ff11ad2bfb0c6711d7837020a50fe7fb8 |
|
20-May-2016 |
robertphillips <robertphillips@google.com> |
Retract GrRenderTarget from GrTestTarget Split out of: https://codereview.chromium.org/1988923002/ (Declassify GrClipMaskManager and Remove GrRenderTarget and GrDrawTarget from GrPipelineBuilder) GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=2004433002 Review-Url: https://codereview.chromium.org/2004433002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
846c051a4800b3cea341a0195db24297d6d9047f |
|
13-May-2016 |
cdalton <cdalton@nvidia.com> |
Convert GrClip to an abstract base class Converts GrClip to an abstract base class and adds a "GrFixedClip" implementation. GrFixedClip denotes a clip implemented with fixed- function hardware. GrFixedClip allows us to remove the stateful "fClipMode" member from GrClipMaskManager, and in the future will be able to nicely encapsulate window rectangles. After this change GrClipMaskManager is just a wrapper around GrDrawTarget. We may want to consider removing it altogether. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1971343002 Review-Url: https://codereview.chromium.org/1971343002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
862cff30eaa16206d76d7de7594c9167375ca87e |
|
13-May-2016 |
cdalton <cdalton@nvidia.com> |
Remove clip from GrPipelineBuilder This eliminates a copy and will allow us to make the GrClip class virutal. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1966903004 Review-Url: https://codereview.chromium.org/1966903004
/external/skia/tests/TessellatingPathRendererTests.cpp
|
8b7451aaf6b1c71e9d343a4df107893db277b6aa |
|
11-May-2016 |
bsalomon <bsalomon@google.com> |
Turn ContextInfos returned by GrContextFactory into structs. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1966013002 Review-Url: https://codereview.chromium.org/1966013002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
6663acff010ce752e4bf778da81fa97448c9db31 |
|
10-May-2016 |
bsalomon <bsalomon@google.com> |
Replace GrStrokeInfo with GrStyle. A side effect is that arbitrary path effects can no be pushed deeper into the Ganesh flow for paths. They may be applied by path renderers. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1957363002 Committed: https://skia.googlesource.com/skia/+/33595bdf4b64a745f6340338d307e806e96c587f Review-Url: https://codereview.chromium.org/1957363002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
85d9667f59d4138438427bb2cdf67992d100e1a0 |
|
10-May-2016 |
bsalomon <bsalomon@google.com> |
Revert of Replace GrStrokeInfo with GrStyle. (patchset #4 id:160001 of https://codereview.chromium.org/1957363002/ ) Reason for revert: Breaking some bots Original issue's description: > Replace GrStrokeInfo with GrStyle. > > A side effect is that arbitrary path effects can no be pushed deeper into the Ganesh flow for paths. They may be applied by path renderers. > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1957363002 > > Committed: https://skia.googlesource.com/skia/+/33595bdf4b64a745f6340338d307e806e96c587f TBR=egdaniel@google.com,robertphillips@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/1967513002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
33595bdf4b64a745f6340338d307e806e96c587f |
|
10-May-2016 |
bsalomon <bsalomon@google.com> |
Replace GrStrokeInfo with GrStyle. A side effect is that arbitrary path effects can no be pushed deeper into the Ganesh flow for paths. They may be applied by path renderers. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1957363002 Review-Url: https://codereview.chromium.org/1957363002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
fda880710c28a1b0ff0c8c88f90a7659a7ce553f |
|
11-Apr-2016 |
bsalomon <bsalomon@google.com> |
Convert some unit tests from running on GL contexts to running on all contexts GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1873313002 Review URL: https://codereview.chromium.org/1873313002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
758586c7f11a6b3529bd4a1c9b4e982a0d0b0582 |
|
06-Apr-2016 |
bsalomon <bsalomon@google.com> |
Make existing unit tests only run on GL contexts BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1869503002 Review URL: https://codereview.chromium.org/1869503002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
f2f1c17e331fe1e0ce695969970ecebc81e12ceb |
|
05-Apr-2016 |
bsalomon <bsalomon@google.com> |
One signature for creating unit tests that run on premade GrContexts GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1860593002 Review URL: https://codereview.chromium.org/1860593002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
546eb5c57aebee47172f9d2a3eae6b49945dba08 |
|
11-Dec-2015 |
kkinnunen <kkinnunen@nvidia.com> |
Initialize glGen* call id args to zero Initialize glGenTexture, glGenBuffer call id buffer items to zero before calls. Otherwise it's not easy to check if the call succeeded or not. Assert the rule in debug gl context. Make TesselatingPathRendererTests use debug gl context. It exercises some of the GenBuffers call sites. Review URL: https://codereview.chromium.org/1514033002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
1530283c483cb88aa725bce50a6d193dd00ee570 |
|
01-Dec-2015 |
kkinnunen <kkinnunen@nvidia.com> |
Generate list of GPU contexts outside tests Use DEF_GPUTEST_FOR_*_CONTEXT macros to obtain the test GPU context. Makes changing the context -related classes easier, since not all tests need to be changed. BUG=skia:2992 Review URL: https://codereview.chromium.org/1448873002
/external/skia/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.cpp
|
6335a729769b06f0d6a1450c7c02e03f95bbb049 |
|
01-Sep-2015 |
joshualitt <joshualitt@chromium.org> |
Move PathRenderers to batches folder BUG=skia: Review URL: https://codereview.chromium.org/1306143005
/external/skia/tests/TessellatingPathRendererTests.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/tests/TessellatingPathRendererTests.cpp
|
eae6200acbec2255ac00ab363ffbe16758ec9076 |
|
31-Jul-2015 |
bsalomon <bsalomon@google.com> |
Some cleanup in GrTextureProvider and GrResourceProvider. Review URL: https://codereview.chromium.org/1261643004
/external/skia/tests/TessellatingPathRendererTests.cpp
|
0aff2fa82a5fa9f99aa77327dac3e9e803b4ed07 |
|
31-Jul-2015 |
bsalomon <bsalomon@google.com> |
Also provides the ResourceProvider to onDrawPath which allows the DF PR to no longer require access to GrContext. Review URL: https://codereview.chromium.org/1265763002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
d309e7aa0efa2d5dd7e7b1af97026fcd3a047e98 |
|
30-Apr-2015 |
bsalomon <bsalomon@google.com> |
This replaces the texture creation/caching functions on GrContext with a GrTextureProvider interface. The goal is to pass this narrowly focused object in places that currently take a GrContext but don't need and shouldn't use its other methods. It also has an extended private interface for interacting with non-texture resource types. Review URL: https://codereview.chromium.org/1107973004
/external/skia/tests/TessellatingPathRendererTests.cpp
|
1899651ffc459f5462aa989cd6d08507947b67e4 |
|
27-Apr-2015 |
kkinnunen <kkinnunen@nvidia.com> |
Extract gpu line dashing to GrDashLinePathRenderer Move line dashing logic from GrContext::drawPath to GrDashLinePathRenderer. This makes it possible to let path renderers render arbitrary dashed paths. End goal is to implement dashing in GrStencilAndCoverPathRenderer. Review URL: https://codereview.chromium.org/1100073003
/external/skia/tests/TessellatingPathRendererTests.cpp
|
c8e934007a48465e584cdeed9ef02077d3cd5451 |
|
21-Apr-2015 |
senorblanco <senorblanco@chromium.org> |
Yet more ASAN fixes. More nullptr checks for factories I have added. Other checks more Yoda-like I have made. (Skia style this is.) BUG=skia: Review URL: https://codereview.chromium.org/1086393004
/external/skia/tests/TessellatingPathRendererTests.cpp
|
0e95b116fcc7bec1bbdf5f0459ebd464c458f714 |
|
21-Apr-2015 |
senorblanco <senorblanco@chromium.org> |
Speculative fix for ASAN build. Check for NULL GrContext in TessellatingPathRendererTests. Also fix an upload nag (no #includes before GPU #ifdef). TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1097943004
/external/skia/tests/TessellatingPathRendererTests.cpp
|
a2b6d28755916cbb4817cd9d1cd1b0e237de9a50 |
|
02-Mar-2015 |
senorblanco <senorblanco@chromium.org> |
Fix for out-of-bounds intersection (found by fuzzer). Sometimes, the intersection returned by check_intersection() is out-of-bounds for both edges (above both tops or below both bottoms) due to floating-point inaccuracy. This causes split_edge() to create a tiny negative-length edge on one side (which would then assert). Although we could safely remove this assert and allow the negative length edge to be removed, it's faster/safer to simply avoid its creation in the first place by adjusting one edge to the other edge's endpoint. Added a new unit test to exercise this case. Review URL: https://codereview.chromium.org/968993002
/external/skia/tests/TessellatingPathRendererTests.cpp
|
d6ed19cc751463285491a538bc7bf154cc7e6d8c |
|
26-Feb-2015 |
senorblanco <senorblanco@chromium.org> |
Tessellating GPU path renderer. This path renderer converts paths to linear contours, resolves intersections via Bentley-Ottman, implements a trapezoidal decomposition a la Fournier and Montuno to produce triangles, and renders those with a single draw call. It does not currently do antialiasing, so it must be used in conjunction with multisampling. A fair amount of the code is to handle floating point edge cases in intersections. Rather than perform exact computations (which would require arbitrary precision arithmetic), we reconnect the mesh to reflect the intersection points. For example, intersections can occur above the current vertex, and force edges to be merged into the current vertex, requiring a restart of the intersections. Splitting edges for intersections can also force them to merge with formerly-distinct edges in the same polygon, or to violate the ordering of the active edge list, or the active edge state of split edges. BUG=skia: Review URL: https://codereview.chromium.org/855513004
/external/skia/tests/TessellatingPathRendererTests.cpp
|