| 4bc7011802e411da74eb2d213d6328e42f7dce1c |
|
01-Mar-2018 |
Robert Phillips <robertphillips@google.com> |
Separate creation time & flush time behavior in GrDrawOpAtlas (take 3)
This CL clarifies what is going on in the GrDrawOpAtlas and GrAtlasGlyphCache.
For the GrDrawOpAtlas:
At creation time all the allowed pages are created (with their backing GrTextureProxies) but they aren't instantiated.
The GrDrawOpAtlas::instantiate call is called in preFlushCB and allocates any pages known to be needed at the start of flush
GrDrawOpAtlas::addToAtlas is called at flush time and, if a new page is activated, will instantiated it at that time.
During compaction, an unused page will be deInstantiated but its Plots and backing GrTextureProxy will remain alive.
The GrAtlasGlyphCache reflects the changes to the GrDrawOpAtlas
It now carries a GrProxyProvider for when it needs to create an atlas
It passes in a GrResourceProvider* at flush time to allow instantiation.
It does not, yet, allocate that GrDrawOpAtlases it might ever require.
TBR=bsalomon@google.com
Change-Id: I276d339d81e7b709140e082a7b58c5584f73ab70
Reviewed-on: https://skia-review.googlesource.com/111100
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 602836138e02935885c77f9dd93dcb51a3ec9a64 |
|
14-Feb-2018 |
Chris Dalton <csmartdalton@google.com> |
Drop unused shader support for sample variables
Bug: skia:
Change-Id: I9e2b7da8c916703027d8dd4303ae67f4e69bcf87
Reviewed-on: https://skia-review.googlesource.com/107356
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7f861927fcd4f9fdef958b5ed9d50b1362ee3232 |
|
30-Jan-2018 |
Robert Phillips <robertphillips@google.com> |
Revert "Drop support for unused MSAA extensions"
This reverts commit 7df27465c4ecc8a4a0cdd95e9785c342903c2de9.
Reason for revert: experimental revert to see if this is the cause of the tree redness
Original change's description:
> Drop support for unused MSAA extensions
>
> Bug: skia:
> Change-Id: I113b80e3f991f195155148625ceb29242ea82776
> Reviewed-on: https://skia-review.googlesource.com/101403
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Chris Dalton <csmartdalton@google.com>
TBR=bsalomon@google.com,csmartdalton@google.com,ethannicholas@google.com
Change-Id: I4fa4123e2d176bef88cd76a09a14053d9ac5809f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/101680
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7df27465c4ecc8a4a0cdd95e9785c342903c2de9 |
|
29-Jan-2018 |
Chris Dalton <csmartdalton@google.com> |
Drop support for unused MSAA extensions
Bug: skia:
Change-Id: I113b80e3f991f195155148625ceb29242ea82776
Reviewed-on: https://skia-review.googlesource.com/101403
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| fc4f768e5aaf8efdd112f38295a35de83a0f9a55 |
|
25-Jan-2018 |
Jim Van Verth <jvanverth@google.com> |
Use int when possible to calculate atlas indices in shaders.
On certain iOS devices half has a mantissa of only 10 bits, which is not
enough to perform the floating point trickery to get the lower bits
out of the "texture coordinates". Instead we use int if available, and
float if not available.
Also re-enables multitexturing for iOS and adds a sample which
stresses the issue, and a version of fontcache that tests multitexturing.
Bug: skia:7285
Change-Id: Ia541b6a418c1860c941071750ceb26459eb846ea
Reviewed-on: https://skia-review.googlesource.com/99800
Reviewed-by: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 09068256d6a12a365f36f14c9fa7e3d6b221abec |
|
03-Jan-2018 |
Brian Osman <brianosman@google.com> |
Remove unused color from all distance field GPs and GrAtlasTextOp
Bug: skia:
Change-Id: I7f2bd68cc4f6ad65660a4415a08777bc71d34ab6
Reviewed-on: https://skia-review.googlesource.com/90522
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e1367b4a064737cbcf78f2297fa89e0a032b2060 |
|
15-Dec-2017 |
Mike Reed <reed@google.com> |
Reland "hide unused var"
This is a reland of 746db42e2c07050d5d0a0db11bfdab441d57e8cd
Original change's description:
> hide unused var
>
> Bug: skia:
> Change-Id: Ia060e0becd8d40817c59e2918678dbdc6dfd6cdd
> Reviewed-on: https://skia-review.googlesource.com/86041
> Reviewed-by: Mike Reed <reed@google.com>
Bug: skia:
Change-Id: I610ccfec1e1eb7888bd4f16f7d83fded10ccf0af
Reviewed-on: https://skia-review.googlesource.com/87720
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5c6ac64516bb56bbdb5d7aedee1a348acc16e29b |
|
19-Dec-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "move homogenous with stride to matrixpriv""
This reverts commit de71572f650005e36d4fc2fe95fb5677a25ae4f6.
Revert "Revert "Transform vertices for distance field glyphs on CPU.""
This reverts commit f226e66d75374e370f3ae2c6895bc689670e9e18.
Change-Id: I2545afae3beb1d6b14bba056853ed826ae7a4679
Reviewed-on: https://skia-review.googlesource.com/86603
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| f226e66d75374e370f3ae2c6895bc689670e9e18 |
|
18-Dec-2017 |
Greg Daniel <egdaniel@google.com> |
Revert "Transform vertices for distance field glyphs on CPU."
This reverts commit 0215e39d7e415d0530231df6ad20d5f215c72152.
Reason for revert: break intel 540 and HD2000 intel release bots on gltestthreading gm dftext_blob_pers
Original change's description:
> Transform vertices for distance field glyphs on CPU.
>
> This allows batching of DF draws with different view matrices.
>
> For perspective matrices this means the transformed position vertex
> attribute must have w values. Currently, non-perspective DF draws still
> use 2 component positions, though this could be changed in the future.
> Consequently, perspective draws can batch with other perspective draws
> but not non-perspective draws.
>
> Adds a GM to test batching and reusing the same blobs with both perspective
> and non-perspective matrices.
>
> Change-Id: I0e42c5449ebf3a5a54025dbcdec824d904d5bd9e
> Reviewed-on: https://skia-review.googlesource.com/79900
> Commit-Queue: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Jim Van Verth <jvanverth@google.com>
TBR=jvanverth@google.com,bsalomon@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.
Change-Id: Idc658d9263976d5b5e00a5026c5d6d3c8f4bdc2d
Reviewed-on: https://skia-review.googlesource.com/86560
Commit-Queue: Greg Daniel <egdaniel@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 1aa6cdad1036f6c9fee02c4425ccc02e6bb3e46d |
|
18-Dec-2017 |
Greg Daniel <egdaniel@google.com> |
Revert "hide unused var"
This reverts commit 746db42e2c07050d5d0a0db11bfdab441d57e8cd.
Reason for revert: revert needed to revert previous cl
Original change's description:
> hide unused var
>
> Bug: skia:
> Change-Id: Ia060e0becd8d40817c59e2918678dbdc6dfd6cdd
> Reviewed-on: https://skia-review.googlesource.com/86041
> Reviewed-by: Mike Reed <reed@google.com>
TBR=reed@google.com,reed@chromium.org
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: skia:
Change-Id: Ia60757c0007199e1c8409a7da15fa452fdb3b16a
Reviewed-on: https://skia-review.googlesource.com/86801
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 746db42e2c07050d5d0a0db11bfdab441d57e8cd |
|
15-Dec-2017 |
Mike Reed <reed@google.com> |
hide unused var
Bug: skia:
Change-Id: Ia060e0becd8d40817c59e2918678dbdc6dfd6cdd
Reviewed-on: https://skia-review.googlesource.com/86041
Reviewed-by: Mike Reed <reed@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0215e39d7e415d0530231df6ad20d5f215c72152 |
|
15-Dec-2017 |
Brian Salomon <bsalomon@google.com> |
Transform vertices for distance field glyphs on CPU.
This allows batching of DF draws with different view matrices.
For perspective matrices this means the transformed position vertex
attribute must have w values. Currently, non-perspective DF draws still
use 2 component positions, though this could be changed in the future.
Consequently, perspective draws can batch with other perspective draws
but not non-perspective draws.
Adds a GM to test batching and reusing the same blobs with both perspective
and non-perspective matrices.
Change-Id: I0e42c5449ebf3a5a54025dbcdec824d904d5bd9e
Reviewed-on: https://skia-review.googlesource.com/79900
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 27372888a5c7698f51069bd5b5beea3e85b66668 |
|
08-Dec-2017 |
Chris Dalton <csmartdalton@google.com> |
Don't force GrGLSLVarying's scope to be compile-time constant
Bug: skia:
Change-Id: I70a191949fd26a39257a8a59a0a4c6448ec1fa37
Reviewed-on: https://skia-review.googlesource.com/82462
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 04460ccee50e03b89420cdaa85882a9da083fa38 |
|
06-Dec-2017 |
Brian Salomon <bsalomon@google.com> |
Make GrGLSLPrimitiveProcessor::emitTransforms take local coords as GrShaderVar.
Also remove unused position variable parameter.
Change-Id: I37f98a03ac1ca750810de13b08e3ffa11e41828c
Reviewed-on: https://skia-review.googlesource.com/81320
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| c02cb8a86ce801f471a8cf7bd46880648c48f089 |
|
20-Nov-2017 |
Jim Van Verth <jvanverth@google.com> |
Revert "Use int when possible to calculate atlas indices in shaders."
This reverts commit 999ec57291dc7cf1d8e3a745627670e6cadc1c12.
Reason for revert: Causing issues with NexusPlayer Vulkan.
Original change's description:
> Use int when possible to calculate atlas indices in shaders.
>
> On certain iOS devices half has a mantissa of only 10 bits, which is not
> enough to perform the floating point trickery to get the lower bits
> out of the "texture coordinates". Instead we use int if available, and
> float if not available.
>
> Also re-enables multitexturing for iOS and adds a sample which
> stresses the issue.
>
> Bug: skia:7285
> Change-Id: I365532c7cbbcca7c7753af209bef46e05be49e11
> Reviewed-on: https://skia-review.googlesource.com/71181
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Jim Van Verth <jvanverth@google.com>
TBR=jvanverth@google.com,bsalomon@google.com,robertphillips@google.com
Change-Id: I82801a73a2a8067588049b213f010ff25f4014f3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:7285
Reviewed-on: https://skia-review.googlesource.com/74001
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 999ec57291dc7cf1d8e3a745627670e6cadc1c12 |
|
20-Nov-2017 |
Jim Van Verth <jvanverth@google.com> |
Use int when possible to calculate atlas indices in shaders.
On certain iOS devices half has a mantissa of only 10 bits, which is not
enough to perform the floating point trickery to get the lower bits
out of the "texture coordinates". Instead we use int if available, and
float if not available.
Also re-enables multitexturing for iOS and adds a sample which
stresses the issue.
Bug: skia:7285
Change-Id: I365532c7cbbcca7c7753af209bef46e05be49e11
Reviewed-on: https://skia-review.googlesource.com/71181
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| c17bf32a1bf2f79a8c4a281c2ad312a3cdb610cd |
|
24-Oct-2017 |
Chris Dalton <csmartdalton@google.com> |
Convert geometry shaders to operate in Skia device space
Defers the transformation to normalized window coordinates until after
the geometry shader. Merges vertex and a geometry shader builders into
a single compilation unit with a common base class. Updates CCPR
geometry shaders accordingly.
Bug: skia:
Change-Id: If93c90e978b1fdc7120febd05cfb05810fd496b5
Reviewed-on: https://skia-review.googlesource.com/62980
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| fdde34eeb2cb5106a6680c49dd30c0b4428cbfbb |
|
16-Oct-2017 |
Chris Dalton <csmartdalton@google.com> |
Remove precision from GrGLSLVaryingHandler
(It's built into the types now.)
Bug: skia:
Change-Id: I44edc6e71098cf74387091903e920b1dd929b103
Reviewed-on: https://skia-review.googlesource.com/59880
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5698c8a8a0dcce3bdabab3551f068bda61d741a3 |
|
12-Oct-2017 |
Jim Van Verth <jvanverth@google.com> |
Batch better in GrSmallPathRenderer and add perspective support.
Minor optimization to text regen also snuck in.
Bug: skia:
Change-Id: I0c5d97defdcf60b9ce663548c056db0e8bf7149b
Reviewed-on: https://skia-review.googlesource.com/57942
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| abff956455637b12eab374fd44b99e1338799113 |
|
09-Oct-2017 |
Ethan Nicholas <ethannicholas@google.com> |
initClassID no longer auto-allocates ids
Auto-allocated IDs mean that the IDs depend upon the order in which
classes happen to get initialized and are therefore not consistent
from run to run. This change paves the way for a persistent shader
cache by fixing the IDs in an enum.
Bug: skia:
Change-Id: I3e923c6c54f41b3b3eb616458abee83e0909c09f
Reviewed-on: https://skia-review.googlesource.com/56401
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| fa7ee2447e6227e7f441d32e570489130c0932bb |
|
25-Sep-2017 |
Ethan Nicholas <ethannicholas@google.com> |
changed vertex attribute precisions to be actual types
Bug: skia:
Change-Id: Ic5555d9f1be7f24655bdea9f2a3677bfb128ef70
Reviewed-on: https://skia-review.googlesource.com/50221
Reviewed-by: Chris Dalton <csmartdalton@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 8aa4569c139a7a7ac38c62b25e3af40309cc2ee2 |
|
20-Sep-2017 |
Ethan Nicholas <ethannicholas@google.com> |
switched SkSL's temporary 'highfloat' type back to 'float'
Bug: skia:
Change-Id: If0debae7318b6b5b4a7cb85d458996a09931127e
Reviewed-on: https://skia-review.googlesource.com/48760
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| f7b8820dc813d1eb0b6b43fe4581dded0da38caf |
|
18-Sep-2017 |
Ethan Nicholas <ethannicholas@google.com> |
re-land of new SkSL precisions
Bug: skia:
Change-Id: Ic1deb3db2cbda6ca45f93dee99832971a36a2119
Reviewed-on: https://skia-review.googlesource.com/47841
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| eafa64b4018a334daae74492fb33bc54bd0bb6aa |
|
18-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Revert "Revert "Allow GrDrawOpAtlas to grow as needed""
Bug: skia:3550
Change-Id: I11a3b39ed7470542447f0e46141a26eef7a9bdaf
Reviewed-on: https://skia-review.googlesource.com/47240
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 27185a9b9756d2f5fcc077c6c2c005259d4ba517 |
|
18-Sep-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "Revert "Revert "Switched highp float to highfloat and mediump float to half."""
This reverts commit 05d5a13fea6246648de7e41358ed338d53c85ea2.
Reason for revert: looks like it broke filterfastbounds
Original change's description:
> Revert "Revert "Switched highp float to highfloat and mediump float to half.""
>
> This reverts commit 1d816b92bb7cf2258007f3f74ffd143b89f25d01.
>
> Bug: skia:
> Change-Id: I388b5e5e9bf619db48297a80c9a80c039f26c9f1
> Reviewed-on: https://skia-review.googlesource.com/46464
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
TBR=bsalomon@google.com,ethannicholas@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: skia:
Change-Id: Iddf6aef2ab084aa73da7ceebdfc303a1d2b80cde
Reviewed-on: https://skia-review.googlesource.com/47441
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 05d5a13fea6246648de7e41358ed338d53c85ea2 |
|
15-Sep-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "Revert "Switched highp float to highfloat and mediump float to half.""
This reverts commit 1d816b92bb7cf2258007f3f74ffd143b89f25d01.
Bug: skia:
Change-Id: I388b5e5e9bf619db48297a80c9a80c039f26c9f1
Reviewed-on: https://skia-review.googlesource.com/46464
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ad70c41f678771f889449c296a8ab48640742050 |
|
15-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Revert "Allow GrDrawOpAtlas to grow as needed"
This reverts commit 03168b8a62a0d3d14b7a0d14642df4d82203b87c.
Reason for revert: Assert in resetting texture proxies.
Original change's description:
> Allow GrDrawOpAtlas to grow as needed
>
> Bug: skia:3550
> Change-Id: Ib5312c8c06ba8549d90545658df6686c45058255
> Reviewed-on: https://skia-review.googlesource.com/45841
> Commit-Queue: Jim Van Verth <jvanverth@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
TBR=jvanverth@google.com,robertphillips@google.com
Change-Id: I05e084b4cefa57e558f296d6087d4f03fa16ac81
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:3550
Reviewed-on: https://skia-review.googlesource.com/47140
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 03168b8a62a0d3d14b7a0d14642df4d82203b87c |
|
15-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Allow GrDrawOpAtlas to grow as needed
Bug: skia:3550
Change-Id: Ib5312c8c06ba8549d90545658df6686c45058255
Reviewed-on: https://skia-review.googlesource.com/45841
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| a950b63b442f1376d022740457b020ada62f6554 |
|
12-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Add support for additional textures in GrDrawOpAtlas
Step two in supporting growable/shrinkable atlases.
Bug: skia:3550
Change-Id: I0cdec2a9f59cc8ced071bfeec2f8ed5a228c4b7a
Reviewed-on: https://skia-review.googlesource.com/43260
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 6a7a704aa4c4e1862f63f42b3e6df997d472dd2f |
|
11-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Add multitexture support to text and path shaders, take 3.
This does not add additional textures to the atlases, just adds the
ability to access those textures in the shaders.
Bug: skia:3550
Change-Id: I4c1da2bc374a76131f5f4ad946543e03e0ab3126
Reviewed-on: https://skia-review.googlesource.com/44841
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| acc8d2e11a7e076cce3a4d0be2f062a2c1f23b54 |
|
09-Sep-2017 |
Mike Klein <mtklein@google.com> |
Revert "Revert "Revert "Add multitexture support to text and path shaders"""
This reverts commit 6c251d24cf90a5c4c091bb7b7373e72705f716d2.
Reason for revert: crashes Fuchsia.
Original change's description:
> Revert "Revert "Add multitexture support to text and path shaders""
>
> This does not add additional textures to the atlases, just adds the
> ability to access those textures in the shaders.
>
> Bug: skia:3550
> Change-Id: I552e2c2e4713a5ea68801fdb8e3bab3867e77988
> Reviewed-on: https://skia-review.googlesource.com/44261
> Reviewed-by: Robert Phillips <robertphillips@google.com>
TBR=jvanverth@google.com,robertphillips@google.com
Change-Id: I25cc5f11a1fa76ca7b2cf24920ae3a2a15aac723
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:3550
Reviewed-on: https://skia-review.googlesource.com/44540
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 6c251d24cf90a5c4c091bb7b7373e72705f716d2 |
|
08-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Revert "Revert "Add multitexture support to text and path shaders""
This does not add additional textures to the atlases, just adds the
ability to access those textures in the shaders.
Bug: skia:3550
Change-Id: I552e2c2e4713a5ea68801fdb8e3bab3867e77988
Reviewed-on: https://skia-review.googlesource.com/44261
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e7492fe49c61915b70bde648b75f9b6fc60b4bef |
|
08-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Revert "Add multitexture support to text and path shaders"
This reverts commit 7f754260f7fc2ae0326a072dd2f0429e584f8ca0.
Reason for revert: Bot failures.
Original change's description:
> Add multitexture support to text and path shaders
>
> This does not add additional textures to the atlases, just adds the
> ability to access those textures in the shaders.
>
> Bug: skia:3550
> Change-Id: I636b329a6f748b6753f5f92a70066fb412623df2
> Reviewed-on: https://skia-review.googlesource.com/43000
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Jim Van Verth <jvanverth@google.com>
TBR=jvanverth@google.com,bsalomon@google.com,robertphillips@google.com
Change-Id: I5c28ea48ed9bdde2cd936ef4f96fc720d5e4b4a5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:3550
Reviewed-on: https://skia-review.googlesource.com/44162
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7f754260f7fc2ae0326a072dd2f0429e584f8ca0 |
|
07-Sep-2017 |
Jim Van Verth <jvanverth@google.com> |
Add multitexture support to text and path shaders
This does not add additional textures to the atlases, just adds the
ability to access those textures in the shaders.
Bug: skia:3550
Change-Id: I636b329a6f748b6753f5f92a70066fb412623df2
Reviewed-on: https://skia-review.googlesource.com/43000
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 2bbdcc44c63974f29f3743bb58d929601a3f65c6 |
|
07-Sep-2017 |
Brian Salomon <bsalomon@google.com> |
Rework GrSamplerParams to be more compact and use its own wrap mode enum.
The main change is to make GrSamplerParams smaller by making its enums have byte-sized underlying types. The rest is cosmetic.
Change-Id: Ib71ea50612d24619a85e463826c6b8dfb9b445e3
Reviewed-on: https://skia-review.googlesource.com/43200
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 964a3b05afabb9594aec2f3c1ba1440404ae91c7 |
|
25-Aug-2017 |
Robert Phillips <robertphillips@google.com> |
Add explicit conversion from uint2 to float2 for Vulkan
Change-Id: I07a28c77f3c9b87d6713ae1d6b7e7d1285098aa8
Reviewed-on: https://skia-review.googlesource.com/38728
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 8296e752fa1803dcb8cf97d6ab10bb4f5f5f51f0 |
|
25-Aug-2017 |
Robert Phillips <robertphillips@google.com> |
Switch atlas clients over to using absolute texture coordinates (take 2)
This is a prerequisite for being able to resize the atlas with impunity.
Change-Id: Iccc9c7ced43f38a7d8483a7bd12a458d59a3453a
Reviewed-on: https://skia-review.googlesource.com/38362
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e253831ee0f3f85c5143e5ac39325400b145106f |
|
24-Aug-2017 |
Robert Phillips <robertphillips@google.com> |
Revert "Switch atlas clients over to using absolute texture coordinates"
This reverts commit e84c087e621978e6d298b8ca950521601a0366cb.
Reason for revert: ANGLE is unhappy
Original change's description:
> Switch atlas clients over to using absolute texture coordinates
>
> This is a prerequisite for being able to resize the atlas with impunity.
>
> Change-Id: I509816c8d6f38fbc92fa39aeab303b42ab09f58b
> Reviewed-on: https://skia-review.googlesource.com/37560
> Reviewed-by: Robert Phillips <robertphillips@google.com>
> Commit-Queue: Robert Phillips <robertphillips@google.com>
TBR=egdaniel@google.com,jvanverth@google.com,robertphillips@google.com,brianosman@google.com
Change-Id: I329efd642c22e11a5c576a4632fc557759b200d5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/38400
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e84c087e621978e6d298b8ca950521601a0366cb |
|
24-Aug-2017 |
Robert Phillips <robertphillips@google.com> |
Switch atlas clients over to using absolute texture coordinates
This is a prerequisite for being able to resize the atlas with impunity.
Change-Id: I509816c8d6f38fbc92fa39aeab303b42ab09f58b
Reviewed-on: https://skia-review.googlesource.com/37560
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 1d816b92bb7cf2258007f3f74ffd143b89f25d01 |
|
17-Aug-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Switched highp float to highfloat and mediump float to half."
This reverts commit 88d99c63878c2d3d340120f0321676f72afcb4f0.
Reason for revert: Believed to be causing unit test failures in Chrome roll:
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/364433
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.android%2Flinux_android_rel_ng%2F364433%2F%2B%2Frecipes%2Fsteps%2Fcontent_browsertests__with_patch__on_Android%2F0%2Flogs%2FWebRtcCaptureFromElementBrowserTest.VerifyCanvasWebGLCaptureColor%2F0
Original change's description:
> Switched highp float to highfloat and mediump float to half.
>
> The ultimate goal is to end up with "float" and "half", but this
> intermediate step uses "highfloat" so that it is clear if I missed a
> "float" somewhere. Once this lands, a subsequent CL will switch all
> "highfloats" back to "floats".
>
> Bug: skia:
> Change-Id: Ia13225c7a0a0a2901e07665891c473d2500ddcca
> Reviewed-on: https://skia-review.googlesource.com/31000
> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
> Reviewed-by: Brian Salomon <bsalomon@google.com>
TBR=bsalomon@google.com,csmartdalton@google.com,ethannicholas@google.com
Change-Id: I8bfa97547ac3920d433665f161d27df3f15c83aa
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/35705
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 88d99c63878c2d3d340120f0321676f72afcb4f0 |
|
16-Aug-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Switched highp float to highfloat and mediump float to half.
The ultimate goal is to end up with "float" and "half", but this
intermediate step uses "highfloat" so that it is clear if I missed a
"float" somewhere. Once this lands, a subsequent CL will switch all
"highfloats" back to "floats".
Bug: skia:
Change-Id: Ia13225c7a0a0a2901e07665891c473d2500ddcca
Reviewed-on: https://skia-review.googlesource.com/31000
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7f23543d1d27452ac1ddc21cc0c9f88479920cde |
|
16-Aug-2017 |
Brian Salomon <bsalomon@google.com> |
Minor geometry processor clarification.
Some renames, comments, and override->final
Change-Id: Iebc7aeee9a64021e958f76bf4278ffff56884a56
Reviewed-on: https://skia-review.googlesource.com/35165
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5af9ea399d5e0344cc4b7da4e97b5dc5b3c74f64 |
|
28-Jul-2017 |
Ethan Nicholas <ethannicholas@google.com> |
renamed SkSL types in preparation for killing precision modifiers
Bug: skia:
Change-Id: Iff0289e25355a89cdc289a0892ed755dd1b1c900
Reviewed-on: https://skia-review.googlesource.com/27703
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 90e89b366a7dc90111c861ba76c492ce1743a0d1 |
|
06-Jul-2017 |
Jim Van Verth <jvanverth@google.com> |
Add proper aliased support for SDF text
Previously, when asked to render aliased text with distance fields,
we would get the aliased glyph from the cache and then try to
anti-alias the edge. This change instead grabs the anti-aliased glyph,
then deliberately aliases the edge.
Bug: chromium:707979
Change-Id: I05766af17d7ae58bca27aaffd9e08e5c586e789c
Reviewed-on: https://skia-review.googlesource.com/21728
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 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/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 18166eeaf2d088d494c4273b88107544166046fd |
|
01-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Omnibus: Push instantiation of GrTextures later (post TextureSampler)
Split into:
https://skia-review.googlesource.com/c/10485/ (More GrSurfaceProxy-clean up)
https://skia-review.googlesource.com/c/15819/ (Expand GrTextureProxy to handle highestFilterMode)
https://skia-review.googlesource.com/c/16714/ (Switch ImageStorageAccess over to GrTextureProxies)
https://skia-review.googlesource.com/c/16908/ (Convert DstTexture to DstProxy)
Change-Id: I6cf3ba0f3bf0e1908d36749bc83571c066ddd568
Reviewed-on: https://skia-review.googlesource.com/10484
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 9bee2e5894bb8dd374392f238bc429e16f239583 |
|
29-May-2017 |
Robert Phillips <robertphillips@google.com> |
Revise system for checking for uninstantiated proxies
The new pattern is:
we will "instantiate" pipelines at flush time
at flush time we will only access the backing GrSurface by peeking
If instantiation fails we should never try to access the GrSurfaces
Change-Id: I87f7ff41bd0e84d9ca3dbdd61d3361d3d4ceefd6
Reviewed-on: https://skia-review.googlesource.com/17932
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5592382a7ad13668f6b70be842948766f0c3a199 |
|
22-May-2017 |
Greg Daniel <egdaniel@google.com> |
Remove setting of alpha coverage in text ops when in LCD mode
Bug: skia:
Change-Id: I0e320497fe72a0edad7bda7ea1c34dc2f713fc56
Reviewed-on: https://skia-review.googlesource.com/17530
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ab8ce1a5ae0a0c87cb127634b9bdb94f3a17cec8 |
|
19-Apr-2017 |
Brian Osman <brianosman@google.com> |
Fixed distance field interpolation for gamma correct rendering
Bug: skia:
Change-Id: I9037decfeb9b2ab673434bb3c4b7f55aca186d68
Reviewed-on: https://skia-review.googlesource.com/13872
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| a91e0b7cc2537dc57ccf67914638e13b4612ffd1 |
|
01-May-2017 |
Robert Phillips <robertphillips@google.com> |
Allow TextureSamplers to have null GrTexture pointer
Bug: 715488
Change-Id: I69775cbb50d334d81872e236e59368fe65e698ff
Reviewed-on: https://skia-review.googlesource.com/14605
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 296b1ccf9b8e9c8b945645efcbaa9c71c7135f58 |
|
15-Mar-2017 |
Robert Phillips <robertphillips@google.com> |
Retract GrContext from src/gpu/effects
Change-Id: Iceb7263098286bafb2605ef17d1fe6bb25d71e97
Reviewed-on: https://skia-review.googlesource.com/9693
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 32342f032e1dfd133040324f851f0365f9d4cb51 |
|
04-Mar-2017 |
Brian Osman <brianosman@google.com> |
Rebase and fix chromium
Combine texture provider and resource provider
Largely mechanical. Only three places that were calling createApprox
via texture provider (ie without flags), so that was simple.
BUG=skia:
Change-Id: I876367bcdc6a8db736deedab1028de1972015509
Reviewed-on: https://skia-review.googlesource.com/9176
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 32f2818c9d10090efeea62ccc211d48a33322dfb |
|
28-Feb-2017 |
Robert Phillips <robertphillips@google.com> |
Clean up/remove unused GrFragmentProcessor-derived ctors
This is the simple (i.e., non-TextureAdjuster) portion of: https://skia-review.googlesource.com/c/8823/ (Remove GrFragmentProcessor-derived class' GrTexture-based ctors)
Change-Id: I8f673ebe922e03c69473c18c166bcf818507c662
Reviewed-on: https://skia-review.googlesource.com/8997
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 33632d8edad59b80eaffd1b32f1edde5e4a22f79 |
|
28-Feb-2017 |
Jim Van Verth <jvanverth@google.com> |
Reland: Allow DFPathRenderer to store bitmaps at low resolutions
BUG=chromium:682918
Change-Id: Ieadb41229227a20d41b8e932ba0770fe72479898
Reviewed-on: https://skia-review.googlesource.com/9068
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0755bb1db7c4076c68fac344484f7652356c2e1e |
|
28-Feb-2017 |
Cary Clark <caryclark@google.com> |
Revert "Allow distance field path renderer to store bitmaps at low resolutions"
This reverts commit c0bc1bb8690e5ce489394112b0cf4fe4601c1f2c.
Reason for revert: broke build with SkTDynamicHash error
Original change's description:
> Allow distance field path renderer to store bitmaps at low resolutions
>
> BUG=chromium:682918
>
> Change-Id: I1a0608f7e6394ab05eebc4b78fb7087ca718f617
> Reviewed-on: https://skia-review.googlesource.com/8971
> Commit-Queue: Jim Van Verth <jvanverth@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
>
TBR=egdaniel@google.com,jvanverth@google.com,bsalomon@google.com,robertphillips@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:682918
Change-Id: I4a3c370a248915fe7c7e77dd0346d6ab6f0d10c6
Reviewed-on: https://skia-review.googlesource.com/9063
Reviewed-by: Cary Clark <caryclark@google.com>
Commit-Queue: Cary Clark <caryclark@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| c0bc1bb8690e5ce489394112b0cf4fe4601c1f2c |
|
28-Feb-2017 |
Jim Van Verth <jvanverth@google.com> |
Allow distance field path renderer to store bitmaps at low resolutions
BUG=chromium:682918
Change-Id: I1a0608f7e6394ab05eebc4b78fb7087ca718f617
Reviewed-on: https://skia-review.googlesource.com/8971
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| dbc8eeb592123619d9c5bb4b6c6225b9fd45d03b |
|
21-Feb-2017 |
Robert Phillips <robertphillips@google.com> |
Remove GrProcessorTestData's GrTextures
(No longer) Blocked on: https://skia-review.googlesource.com/c/8450/ (Remove asTextureRef from SkSpecialImage & update effects accordingly (take 2))
This also (unfortunately) picks up a few straggling effects that didn't have a sk_sp<GrTextureProxy> factory.
Change-Id: I5ce583a084aa8fe00e866eec1db90e2ec9dd2ab0
Reviewed-on: https://skia-review.googlesource.com/8500
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| cb30bb2cb727e276792812c6390547dba474c831 |
|
12-Feb-2017 |
Brian Salomon <bsalomon@google.com> |
Remove GrFragmentProcessor::computeInvariantOutput
Change-Id: If475730103052c6097eb91be06808fb723b70bf8
Reviewed-on: https://skia-review.googlesource.com/8330
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 6f6961ebad65c582318564b3688e78e5c99f3935 |
|
31-Jan-2017 |
Hal Canary <halcanary@google.com> |
make GR_TEST_UTILS=0 work
Change-Id: Icce35140ca08b65217b32e6b1ffc6ad2f38ab37f
Reviewed-on: https://skia-review.googlesource.com/7840
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| bfd5183b9e039b50fb33441d1f90130b8eced80a |
|
04-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Removes tracking of whether color is ignored by XP.
Removes the feedback to GrDrawOp via GrPipelineOptimizations.
Change-Id: I3cb17cad41779af292a92385fcd5ac23ae5a1ffd
Reviewed-on: https://skia-review.googlesource.com/6561
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 94efbf51f5a88d9e8aa961d3fbe38c5e335d6108 |
|
29-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Merge GrGLSLCaps into GrShaderCaps
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5121
Change-Id: If8d13638f80f42161cbc766a2666c5789e5772c8
Reviewed-on: https://skia-review.googlesource.com/5121
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 1fc83b131afa7c29e971dd3ccc7906006bde6c09 |
|
22-Nov-2016 |
Ethan Nicholas <ethannicholas@google.com> |
removed appendPrecisionModifier
skslc always provides support for precision modifiers (ignoring them if
unsupported on the target platform), so the Skia appendPrecisionModifier
function was redundant.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5117
Change-Id: I4bcbd47ef8d1f4f1b055086d4c9050254584b48c
Reviewed-on: https://skia-review.googlesource.com/5117
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 514baff8be7f71111aa7bfb9b099a096b31e16ec |
|
17-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrTextureParams to GrSamplerParams
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4965
Change-Id: I7d52e81c670e92ca96117284f44b274ce3cc3671
Reviewed-on: https://skia-review.googlesource.com/4965
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| db4183d227b764b219e9e314dd5387ded4c38797 |
|
17-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
In GrProcessor::TextureSampler drop the "get", it's cleaner
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4962
Change-Id: I55e7f8d1b6e1097fdbe411e9989dd42a03dd5f33
Reviewed-on: https://skia-review.googlesource.com/4962
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0bbecb21ab82b3d742c491780bcc2e74be03efed |
|
17-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrTextureAccess to GrProcessor::TextureSampler.
Renames vars and methods that used the work "access" to refer to this type.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4931
Change-Id: Ibcf488fbd445c5119fc13d190544cd98981bdbee
Reviewed-on: https://skia-review.googlesource.com/4931
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ddb37d67ba4db42fa5c6012b58d0f4985b454dc0 |
|
20-Oct-2016 |
ethannicholas <ethannicholas@google.com> |
re-re-land of skslc now automatically turns on derivatives support
Only change from last attempt is putting the call to shaderDerivativeExtensionString behind a check for shaderDerivativeSupport to avoid a spurious assertion failure.
TBR=benjaminwagner@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2437063002
Review-Url: https://chromiumcodereview.appspot.com/2437063002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ac66ae63eade5347864c5d0bd025756c5ea23fda |
|
18-Oct-2016 |
Mike Klein <mtklein@google.com> |
Revert "re-land of skslc now automatically turns on derivatives support"
This reverts commit d7d1997643db3b9b5caaaac235c0e2cd0fa23ec2.
This breaks the Chromium roll (it did last time it landed too).
https://codereview.chromium.org/2429853002
Please gate your next attempt on at least one of linux_android_rel_ng, linux_chromium_rel_ng, or mac_chromium_rel_ng.
TBR=benjaminwagner@google.com,ethannicholas@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I999d4af70a5704b8990f3cd7d145eabff57646f5
Reviewed-on: https://skia-review.googlesource.com/3590
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| d7d1997643db3b9b5caaaac235c0e2cd0fa23ec2 |
|
18-Oct-2016 |
Ethan Nicholas <ethannicholas@google.com> |
re-land of skslc now automatically turns on derivatives support
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3602
Change-Id: I822fc23f7d4eac76d5059412706117818f3738f0
Reviewed-on: https://skia-review.googlesource.com/3602
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 37269ff2592dfb3abff987a3507b18e8073e96d8 |
|
18-Oct-2016 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "skslc now automatically turns on derivatives support"
This reverts commit 311742bd0eac6fad150bf8f0a4e2809679464e47.
Reason for revert: Android try failure
Change-Id: I6e92b49df1c8f692da947b277067b9003a9367d0
Reviewed-on: https://skia-review.googlesource.com/3600
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 311742bd0eac6fad150bf8f0a4e2809679464e47 |
|
18-Oct-2016 |
Ethan Nicholas <ethannicholas@google.com> |
skslc now automatically turns on derivatives support
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3569
Change-Id: I211f4a80ced951a0d2f29763f85fe75a5daccff7
Reviewed-on: https://skia-review.googlesource.com/3569
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| a624bf3d1cb454c1959c5bbbf23a3afdfa3481f3 |
|
20-Sep-2016 |
bsalomon <bsalomon@google.com> |
Stop flattening GrCoordTransforms in parent GrFragmentProcessors.
This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002
Committed: https://skia.googlesource.com/skia/+/d91237ee051523f439238042674ade99207fe4a6
Review-Url: https://codereview.chromium.org/2339203002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 56b7dc476ba5e4a53ab24b5830b5ed03e404006d |
|
19-Sep-2016 |
bsalomon <bsalomon@google.com> |
Revert of Stop flattening GrCoordTransforms in parent GrFragmentProcessors. (patchset #3 id:40001 of https://codereview.chromium.org/2339203002/ )
Reason for revert:
Crashing blink
https://codereview.chromium.org/2351743002/
Original issue's description:
> Stop flattening GrCoordTransforms in parent GrFragmentProcessors.
>
> This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors.
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002
>
> Committed: https://skia.googlesource.com/skia/+/d91237ee051523f439238042674ade99207fe4a6
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/2351753002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| d91237ee051523f439238042674ade99207fe4a6 |
|
19-Sep-2016 |
bsalomon <bsalomon@google.com> |
Stop flattening GrCoordTransforms in parent GrFragmentProcessors.
This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002
Review-Url: https://codereview.chromium.org/2339203002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 32d1e95ca593adfaa12bc48d4ac0d5501a4b2046 |
|
15-Sep-2016 |
bsalomon <bsalomon@google.com> |
Remove "f" suffix from literal in LCD DF text shader.
Also improve DF GP names
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2340603003
Review-Url: https://codereview.chromium.org/2340603003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 6cb807bf99ac0f8f166e1790f91bcb3afbfb5458 |
|
17-Aug-2016 |
bsalomon <bsalomon@google.com> |
Simplify adding attributes to GrGeometryProcessor
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2248323003
Review-Url: https://codereview.chromium.org/2248323003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e499adf328bd7fc15a755325749f54c2b7e71f54 |
|
20-Jul-2016 |
jvanverth <jvanverth@google.com> |
Use dFdx in Vulkan to address distance field issues.
On Nexus Player and occasionally Nexus 5x we get transparent boxes around
paths. This appears to be because the dFdy call is not as accurate as
dFdx, which is the opposite of Mali 400. As Mali 400 is not supported with
Vulkan, we can go back to using dFdx in this case.
BUG=skia:5523
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2163213004
Review-Url: https://codereview.chromium.org/2163213004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0e3c55431b463b5575983c0c875909e08a3562bf |
|
13-Apr-2016 |
brianosman <brianosman@google.com> |
Tweak distance field path renderer behavior in gamma-correct mode to match
recent changes to text rendering.
Uses linear coverage falloff. Produces results that are perceptually more
similar to L32 (raster and gpu). Smoothstep + sRGB was too soft.
Plumb the gamma-correctness via DrawPathArgs, which also paves the way for
other path rendering implementations to easily make decisions about rendering
technique based on that flag.
Fix a few typos and formatting issues from my most recent change.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1889453002
Review URL: https://codereview.chromium.org/1889453002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| b461d3457567cf79a3efac3123c00f7afc37fecb |
|
13-Apr-2016 |
brianosman <brianosman@google.com> |
Rename lots of things from 'sRGB' to 'GammaCorrect', where appropriate
Trying to be much more explicit about where we really mean sRGB as a format,
and where we mean gamma-correct, as in: "not legacy behavior". Most of the
changes to rendering behavior are dependent on the latter, so let's be precise.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1884873006
Review URL: https://codereview.chromium.org/1884873006
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0586f5cc9713268238394411a5daa2c7758b092b |
|
12-Apr-2016 |
brianosman <brianosman@google.com> |
sRGB support for distance field text.
Add a second distance field adjust table that only applies contrast,
not fake-gamma correction. Store a flag in the batch at creation time,
using the same logic we apply elsewhere (render target format, plus
paint flags).
That gets us close, but not as good as bitmap text. The final step is
to use a linear step function (rather than smoothstep) to map distance
to coverage, when we have sRGB output. Smoothstep's nonlinear response
is actually doing some fake-gamma, so it ends up over-correcting when
the output is already gamma-correct.
Results are now very close between L32 (old table, smoothstep) and S32
(contrast-only table, linstep).
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1885613002
Review URL: https://codereview.chromium.org/1885613002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 3f6f76f98b6b37d17d1492791ff0feb1b7586bd6 |
|
11-Apr-2016 |
cdalton <cdalton@nvidia.com> |
Rename EmitArgs::fSamplers to fTexSamplers
Renames fSamplers to fTexSamplers in GrProcessor EmitArgs, and renames
GrGLSLTextureSampler to GrGLSLSampler. This will allow us to add a
second array of buffer samplers.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1862373003
Review URL: https://codereview.chromium.org/1862373003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 1f50acff0dc09e50cc207846508d8e6459334b2a |
|
11-Apr-2016 |
cdalton <cdalton@nvidia.com> |
Add appendPrecisionModifier method to GrGLSLShaderBuilder
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1881513002
Review URL: https://codereview.chromium.org/1881513002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| cf371bb41b4a0591347a076ce7d83336d4bbc6fb |
|
10-Mar-2016 |
jvanverth <jvanverth@google.com> |
Use correct gradient calculation for rotated distance fields.
For all uniform scaled matrices, we were using the y coordinate of the
y-direction gradient of the v texel space coordinate to scale the
distance values. The problem is, if you rotate by 90 degrees this value
becomes 0. The correct answer is to take the length of the y-direction
gradient.
BUG=skia:4864
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1776383002
Review URL: https://codereview.chromium.org/1776383002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7023a00c35d904e4ccff09c377e9ba26abba6181 |
|
22-Feb-2016 |
jvanverth <jvanverth@google.com> |
Use unorm shorts for texture coordinates when rendering text.
There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com
Committed: https://skia.googlesource.com/skia/+/e507ff0460f4f878214b9454fb5b9ab8d64d8063
Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| d106c2c64012688f8d1439543d942f8f66a142e0 |
|
19-Feb-2016 |
jvanverth <jvanverth@google.com> |
Revert of Use unorm shorts for texture coordinates when rendering text. (patchset #3 id:40001 of https://codereview.chromium.org/1713693002/ )
Reason for revert:
Causing issues with text on Mali 400s. Examples: largeglyphblur, imageblurtiled. It appears that there are precision problems.
Original issue's description:
> Use unorm shorts for texture coordinates when rendering text.
>
> There are a couple of reasons for this:
> - Vulkan does not guarantee conversions from integral vertex attributes
> to floating point shader variables
> - This may be faster and more precise on some platforms, as it avoids
> the aforementioned conversion and changes a multiply by a very small
> value to a multiply by a medium-sized value.
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
> TBR=bsalomon@google.com
>
> Committed: https://skia.googlesource.com/skia/+/e507ff0460f4f878214b9454fb5b9ab8d64d8063
TBR=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/1709133003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e507ff0460f4f878214b9454fb5b9ab8d64d8063 |
|
19-Feb-2016 |
jvanverth <jvanverth@google.com> |
Use unorm shorts for texture coordinates when rendering text.
There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 8528541dd7f09f5bd76f3f1ce5f45d08ac7347c7 |
|
18-Feb-2016 |
cdalton <cdalton@nvidia.com> |
Add more specialized fragment builders
Adds specialized fragment builders for primitive and fragment
processors. Removes fragment-specific functionality from the Xfer
fragment builder.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1709153002
Review URL: https://codereview.chromium.org/1709153002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5e58ceea8569f0d90ff7e3daf5de2def50407212 |
|
11-Feb-2016 |
cdalton <cdalton@nvidia.com> |
Add GrShaderFlags enum
Replaces GrGLSLUniformHandler::ShaderVisibility with GrShaderFlags.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1684063006
Review URL: https://codereview.chromium.org/1684063006
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 53f26aa045d58feb9f2114f53629024bffe52dea |
|
10-Dec-2015 |
joshualitt <joshualitt@chromium.org> |
Remove uniform color for GrDistanceFieldGeoProc
BUG=skia:
Review URL: https://codereview.chromium.org/1512533003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| d9d30f7b577711a200395f7b62acc0d82a19f9c3 |
|
08-Dec-2015 |
joshualitt <joshualitt@chromium.org> |
Allow LCD text to batch across colorchanges. This will always use
color vertices, even when we can't batch across color changes
BUG=skia:
Review URL: https://codereview.chromium.org/1502253003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7ea439b2203855db97330b25945b87dd4b170b8b |
|
03-Dec-2015 |
egdaniel <egdaniel@google.com> |
Create GLSLUniformHandler class for gpu backend
BUG=skia:
Review URL: https://codereview.chromium.org/1490283004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0eafe79f42e3c675f3c504aed4a41abf511df2b7 |
|
20-Nov-2015 |
egdaniel <egdaniel@google.com> |
Create GrGLSLVaryingHandler class for program building
BUG=skia:
Review URL: https://codereview.chromium.org/1462123003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| a2e3e0f7f8ceed2ab152428d7ee2812ad8c842c3 |
|
19-Nov-2015 |
egdaniel <egdaniel@google.com> |
Move glsl onto EmitArgs struct for emitCode
BUG=skia:
Review URL: https://codereview.chromium.org/1453623003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 4ca2e6034365ad280ec64473f7f1d72ebd8335e4 |
|
18-Nov-2015 |
egdaniel <egdaniel@google.com> |
Add ShaderBuilders to EmitArgs and remove gettings from ProgBuilder.
BUG=skia:
Review URL: https://codereview.chromium.org/1457543003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 57d3b039c635945e1dc2fcbac3462ed8bfedb068 |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Rename some processor functions from GL to GLSL
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1443743002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e659a581f63fdccb64dce2dc8a478cf56831feea |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Move GrGLPrimitive/GeometryProc to GLSL
BUG=skia:
Review URL: https://codereview.chromium.org/1441683008
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 64c4728c70001ed074fecf5c4e083781987b12e9 |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Make all GrFragmentProcessors GL independent.
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1434313002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 2d721d33aad192cc8a7a1321504b39bdca2a57ce |
|
11-Nov-2015 |
egdaniel <egdaniel@google.com> |
Move all ShaderBuilder files to GLSL
BUG=skia:
Review URL: https://codereview.chromium.org/1438003003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 8dcdedc4a087ea46ce1e2458d335d60918e56310 |
|
11-Nov-2015 |
egdaniel <egdaniel@google.com> |
Make GrGLSLProgramBuilder base class for GrGLProgramBuilder.
This CL still keeps the weird diamond shape we have for all our ProgramBuilders.
However, the GrGLSL base class will allow us to pull multiple other parts
of our program setup away from GL which will eventually allow us to break up
the diamond.
As part of this all ShaderBuilder subclass have been made gl independent,
however I will move them to GLSL files/class names in a follow on CL.
BUG=skia:
Review URL: https://codereview.chromium.org/1416423003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 018fb62d12d1febf121fe265da5b6117b86a6541 |
|
28-Oct-2015 |
egdaniel <egdaniel@google.com> |
Create GLSL base class for ProgramDataManager
BUG=skia:
Review URL: https://codereview.chromium.org/1428543003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0d3f061262a53b775f0a92b0abf8a4a846290d65 |
|
21-Oct-2015 |
egdaniel <egdaniel@google.com> |
Move GrGLShaderVar to GrGLSL
BUG=skia:
Review URL: https://codereview.chromium.org/1417123002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| f529439fea003851d986a0573a7e0465754b2a48 |
|
21-Oct-2015 |
egdaniel <egdaniel@google.com> |
Move shader precision modifier check onto GLSLCaps
BUG=skia:
Review URL: https://codereview.chromium.org/1414373002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 27b6335c71bccb96206ff0495793433bfa982777 |
|
15-Sep-2015 |
egdaniel <egdaniel@google.com> |
On gpu, use max(r,g,b) for coverage alpha in LCD and update lcd blend gm.
BUG=skia:
Review URL: https://codereview.chromium.org/1339213004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 465283cdf98ed9ab5285ca7b9814e430fca1d452 |
|
11-Sep-2015 |
joshualitt <joshualitt@chromium.org> |
Remove batchtracker
BUG=skia:
Review URL: https://codereview.chromium.org/1332923003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| c21b09eec91c9e263cb0b88467ea44e348ed4962 |
|
29-Aug-2015 |
bsalomon <bsalomon@google.com> |
Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
Committed: https://skia.googlesource.com/skia/+/ecfdc251be71f3d634e76afdd6375bf55fc061aa
Review URL: https://codereview.chromium.org/1316513002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| a511e6ad1042093bea5b015c22a920313fc57c0b |
|
29-Aug-2015 |
rmistry <rmistry@google.com> |
Revert of Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor* (patchset #8 id:140001 of https://codereview.chromium.org/1316513002/ )
Reason for revert:
Primary suspect in failing DEPS rolls:
* https://codereview.chromium.org/1315753006
* https://codereview.chromium.org/1308323006
* https://codereview.chromium.org/1320903004
Primary suspect because the failing win bots did not fail in https://codereview.chromium.org/1315753005
Original issue's description:
> Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
>
> Committed: https://skia.googlesource.com/skia/+/ecfdc251be71f3d634e76afdd6375bf55fc061aa
TBR=joshualitt@google.com,wangyix@google.com,robertphillips@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1313573005
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ecfdc251be71f3d634e76afdd6375bf55fc061aa |
|
28-Aug-2015 |
bsalomon <bsalomon@google.com> |
Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
Review URL: https://codereview.chromium.org/1316513002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| da04e0e80a0113e1d838f7406cd8a8d545b8c02b |
|
19-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Allow setting of GrBatchFontCache atlas sizes
BUG=skia:
Review URL: https://codereview.chromium.org/1255943006
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 7375d6bab2ee8b02da276597ed4d60f22f54eb89 |
|
07-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
fixup precision with configurable atlas
BUG=skia:
Review URL: https://codereview.chromium.org/1276383003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 922c8b13c512c3287509936795735c1b31bedba9 |
|
07-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Break LCD and Bitmap text dependency on hardcoded atlas values
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
Review URL: https://codereview.chromium.org/1271873002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 9162ea9444ac3127c3a93a880dc77efc839391b3 |
|
05-Aug-2015 |
joshualitt <joshualitt@google.com> |
Revert of Break LCD and Bitmap text dependency on hardcoded atlas values (patchset #2 id:20001 of https://codereview.chromium.org/1271873002/ )
Reason for revert:
Breaks S3
Original issue's description:
> Break LCD and Bitmap text dependency on hardcoded atlas values
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
TBR=jvanverth@google.com,bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1271163003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 846b022f6b469cfde285372f26e0d5c593d122ac |
|
04-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Break LCD and Bitmap text dependency on hardcoded atlas values
BUG=skia:
Review URL: https://codereview.chromium.org/1271873002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 6af0c9394301b9f976813cc6dbdbbfe779030fe1 |
|
22-Jul-2015 |
wangyix <wangyix@google.com> |
Moved GrGLFragmentProcessor definition to its own file
BUG=skia:
Review URL: https://codereview.chromium.org/1246193002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 0067ff5e0f85084dd2b5ad9886b526482b89a116 |
|
08-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
fix up test create functions
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1213623022
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| cba99b88fd5fb39def7a094dc32c0745c7a1cfea |
|
24-Jun-2015 |
jvanverth <jvanverth@google.com> |
Move GLSL-specific routines/classes to separate glsl directory
The purpose is to begin separating our GLSL-specific code from
our GL-specific code, so it can be used with the GL45 platform
Review URL: https://codereview.chromium.org/1202293002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 5559ca2a18687ef16b2fc92d8fcacbc1c3e93d54 |
|
22-May-2015 |
joshualitt <joshualitt@chromium.org> |
Remove uniform view matrix from GrGLPrimitiveProcessor base class
BUG=skia:
Review URL: https://codereview.chromium.org/1151693005
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| f2539d50f911914af0f80f0092ff8c654869e650 |
|
20-May-2015 |
senorblanco <senorblanco@chromium.org> |
Use high precision vertex attributes.
Specifying vertex attributes and the view matrix as mediump causes bad artifacts on Mali T6XX.
Covered by the bigmatrix GM in msaa4 mode.
BUG:skia=3545,3771
Review URL: https://codereview.chromium.org/1145273002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 4b91f768b348aa1cebeb54f3ff9331938734c242 |
|
19-May-2015 |
bsalomon <bsalomon@google.com> |
rename GrDrawTargetCaps to GrCaps
Review URL: https://codereview.chromium.org/1133123009
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.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/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e3ababe44315452cd33b96a18ce316ede09ff3c3 |
|
15-May-2015 |
joshualitt <joshualitt@chromium.org> |
remove localmatrix from GrGeometryProcessor base class
BUG=skia:
Review URL: https://codereview.chromium.org/1131513005
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| e578a95d3ab16544794b94da4e7ae13fc2ce6244 |
|
14-May-2015 |
joshualitt <joshualitt@chromium.org> |
Remove viewmatrix from GrGeometryProcessor base class
BUG=skia:
Review URL: https://codereview.chromium.org/1127953003
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 88c23fc6e8ed0243159fd17d815b4f813805647b |
|
13-May-2015 |
joshualitt <joshualitt@chromium.org> |
remove color from GrGeometryProcessor
BUG=skia:
Review URL: https://codereview.chromium.org/1140983002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 1ba8cc9193f13a812401e14b8037fdcf3f3acbac |
|
13-May-2015 |
joshualitt <joshualitt@chromium.org> |
removing equality / compute invariant loops from GrGeometryProcessors
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/ef292a0901205b9785a30daae2c036aa34a970ca
Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 29bee0fe657fabf7c396502b69c9167fba13eaaa |
|
29-Apr-2015 |
egdaniel <egdaniel@google.com> |
Make XPFragmentBuilder only Builder with access to DstCopy.
Plus a bunch of renaming.
BUG=skia:
Review URL: https://codereview.chromium.org/1110033004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 4eaf9cef5a76098f78efac30beb966ac833d32c2 |
|
28-Apr-2015 |
joshualitt <joshualitt@chromium.org> |
create GrTestUtils.h, move some common functions into it
BUG=skia:
Review URL: https://codereview.chromium.org/1117443002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 50cb76b2bb0ffa607a1409f77be0ae7d64e31436 |
|
28-Apr-2015 |
joshualitt <joshualitt@google.com> |
Revert of removing equality / compute invariant loops from GrGeometryProcessors (patchset #2 id:20001 of https://codereview.chromium.org/1111603004/)
Reason for revert:
breaks gl programs
Original issue's description:
> removing equality / compute invariant loops from GrGeometryProcessors
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/ef292a0901205b9785a30daae2c036aa34a970ca
TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1110993002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| ef292a0901205b9785a30daae2c036aa34a970ca |
|
28-Apr-2015 |
joshualitt <joshualitt@chromium.org> |
removing equality / compute invariant loops from GrGeometryProcessors
BUG=skia:
Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| cfc18867d982119d9dc2888bf09f1093012daadd |
|
28-Apr-2015 |
jvanverth <jvanverth@google.com> |
Use GLSLCaps for creating processor keys and GLSL-specific programs
Effectively all this does is future-proof any GLSL-specific code, as
GLSLCaps is just a typedef of GLCaps.
BUG=skia:
Review URL: https://codereview.chromium.org/1109863004
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| bc0273524b039c45dcea2c1ab5ab379c75486c07 |
|
15-Apr-2015 |
jvanverth <jvanverth@google.com> |
Fix LCD text and add path fix for Mali.
dFdy(st.y) produces negative values, so we need to take
abs() to get the right inverse scale.
BUG=skia:3528
Review URL: https://codereview.chromium.org/1086183002
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 221360a514fb4bfff5b461e83262306b2a0f36af |
|
15-Apr-2015 |
jvanverth <jvanverth@google.com> |
Real fix for the Mali bug.
The dFdy(st.y) computation seems to be better than the dFdx(st.x) computation
on the Mali, so using that. The end result should be about the same.
BUG=skia:3528
Review URL: https://codereview.chromium.org/1082913006
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|
| 8ed3b9a386374d7996dfbe0c9de13b42f3dd245d |
|
09-Apr-2015 |
jvanverth <jvanverth@google.com> |
Rename DistanceFieldTextureEffect.{cpp,h}
BUG=skia:
Review URL: https://codereview.chromium.org/1073473005
/external/skia/src/gpu/effects/GrDistanceFieldGeoProc.cpp
|