| 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/GrDitherEffect.cpp
|
| 9cdb9920fcad286ecf7875ea19902022b644fbdc |
|
03-Feb-2016 |
robertphillips <robertphillips@google.com> |
Clean up GrGLSLFragmentProcessor-derived classes
In some other patches it was observed that many of the GrGLSLFragmentProcessor-derived classes needlessly pass a parameter to their constructors.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1666773002
Review URL: https://codereview.chromium.org/1666773002
/external/skia/src/gpu/effects/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.cpp
|
| 02141734f2f89fa4e89ba804bc06fa80ed0fc316 |
|
21-Oct-2015 |
bsalomon <bsalomon@google.com> |
Stop creating static GrProcessor instances
R=joshualitt@google.com
BUG=skia:4483
Review URL: https://codereview.chromium.org/1414733007
/external/skia/src/gpu/effects/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.cpp
|
| 38f1f6f9e590fc20b81015ceca9d642cbe02a2d9 |
|
27-Aug-2015 |
mdempsky <mdempsky@chromium.org> |
Remove overly complicated GR_CREATE_STATIC_PROCESSOR macro
This macro was responsible for producing code like:
static SkAlignedStorage<sizeof(Foo)> g_gFoo_Storage;
static Foo* gFoo = new(g_gFoo_Storage.get()) Foo;
static SkAutoTDestroy<Foo> gFoo_ad(gFoo);
which would allocate static storage for an object of type Foo
(g_gFoo_Storage), lazily instantiate the object in that memory (via
gFoo's initializer), and then ensure that at global destruction time
the object is destroyed (via gFoo_Ad's destructor).
However, the exact same effect is achieved by just writing:
static Foo gFoo;
Review URL: https://codereview.chromium.org/1314763009
/external/skia/src/gpu/effects/GrDitherEffect.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/GrDitherEffect.cpp
|
| b1daa86732fe70aa4630c89d75ff0fd619d77c77 |
|
18-Aug-2015 |
wangyix <wangyix@google.com> |
When getGLInstance is called on a frag proc, the resulting GrGLFragmentProcessor will be the root of a tree of GrGLFragmentProcessors that mirrors the GrFragmentProcessor's tree. This allows setData() to be called recursively (removing the responsibility from compose shader) and allows gl instances direct access to their children gl instances so they can emit their code.
BUG=skia:4182
Review URL: https://codereview.chromium.org/1287023009
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 4b3050b410254d0cb38df9a30ae2e209124fa1a2 |
|
04-Aug-2015 |
wangyix <wangyix@google.com> |
Added registerChild; transforms, textures, glKey automatically handled.
BUG=skia:
Review URL: https://codereview.chromium.org/1266633003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 7c157a988845fb00f9024d6db6dda142c3458033 |
|
23-Jul-2015 |
wangyix <wangyix@google.com> |
Added GrGLFragmentProcessor::EmitArgs struct for use with emitCode()
BUG=skia:
Review URL: https://codereview.chromium.org/1251173002
/external/skia/src/gpu/effects/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.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/GrDitherEffect.cpp
|
| 36352bf5e38f45a70ee4f4fc132a38048d38206d |
|
26-Mar-2015 |
mtklein <mtklein@chromium.org> |
C++11 override should now be supported by all of {bots,Chrome,Android,Mozilla}
NOPRESUBMIT=true
BUG=skia:
DOCS_PREVIEW= https://skia.org/?cl=1037793002
Review URL: https://codereview.chromium.org/1037793002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 72c9faab45124e08c85f70ca38536914862d947c |
|
09-Jan-2015 |
mtklein <mtklein@chromium.org> |
Fix up all the easy virtual ... SK_OVERRIDE cases.
This fixes every case where virtual and SK_OVERRIDE were on the same line,
which should be the bulk of cases. We'll have to manually clean up the rest
over time unless I level up in regexes.
for f in (find . -type f); perl -p -i -e 's/virtual (.*)SK_OVERRIDE/\1SK_OVERRIDE/g' $f; end
BUG=skia:
Review URL: https://codereview.chromium.org/806653007
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| eb2a6761654307e8aeeeaabdd63c6bf9ab0411e9 |
|
04-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove backend factories
BUG=skia:
Review URL: https://codereview.chromium.org/778453002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 267ce482b54f46097584e0f9350ec74aa6a2cd44 |
|
25-Nov-2014 |
joshualitt <joshualitt@chromium.org> |
remove proc key
BUG=skia:
Review URL: https://codereview.chromium.org/755363002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 605dd0fbce9dbb2a0d3313e13e161f2bd54870d7 |
|
12-Nov-2014 |
egdaniel <egdaniel@google.com> |
Move GrInvariantOutput out of GrProcessor and into its own class.
This will help with the ability to subclass, add to, etc. GrInvariantOutput. Also it was simply
getting a little too big to be a "supporting" subclass
BUG=skia:
Review URL: https://codereview.chromium.org/699943003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 9e4d6d180fcfbbe2ea242196cc0affd45b7ed7ae |
|
15-Oct-2014 |
egdaniel <egdaniel@google.com> |
Move willUseInputColor check to computeInvariantOutput
BUG=skia:
Review URL: https://codereview.chromium.org/656503002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 98b33ebe6f66ed59b023cf16a09c0a40eb4510ce |
|
15-Oct-2014 |
bsalomon <bsalomon@google.com> |
Some cleanup in processor header files.
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/659803004
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 0e08fc17e4718f7ce4e38f793695896473e96948 |
|
15-Oct-2014 |
bsalomon <bsalomon@google.com> |
Push isEqual/onIsEqual down from GrProcessor to subclasses.
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/654273002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| ccb2e384a036f29d989d3c1468f879324e81a678 |
|
13-Oct-2014 |
egdaniel <egdaniel@google.com> |
Create helper functions to use in computeInvariantOutput calls
BUG=skia:
Review URL: https://codereview.chromium.org/643743003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 1598899975ecc85b003a59740b588d1ddbcedb09 |
|
10-Oct-2014 |
joshualitt <joshualitt@chromium.org> |
FPs now use the correct builder types(just a rename)
BUG=skia:
Review URL: https://codereview.chromium.org/648463003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 1a8ecdfb73a15de600d5779b75d7c4b61863c50b |
|
03-Oct-2014 |
egdaniel <egdaniel@google.com> |
Add isSingleComponent bool to getConstantColorComponent
Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/3b8af078281a5a20f951b9fd84f38d92b8f6217b
Review URL: https://codereview.chromium.org/608253002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| b4b7a4c9ea2e399db550f93e3754c351e5b2079a |
|
02-Oct-2014 |
robertphillips <robertphillips@google.com> |
Revert of Add isSingleComponent bool to getConstantColorComponent (patchset #6 id:100001 of https://codereview.chromium.org/608253002/)
Reason for revert:
Changing some GMs
Original issue's description:
> Add isSingleComponent bool to getConstantColorComponent
>
> Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3b8af078281a5a20f951b9fd84f38d92b8f6217b
TBR=joshualitt@chromium.org,bsalomon@google.com,reed@google.com,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/617853003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 3b8af078281a5a20f951b9fd84f38d92b8f6217b |
|
02-Oct-2014 |
egdaniel <egdaniel@google.com> |
Add isSingleComponent bool to getConstantColorComponent
Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components.
BUG=skia:
Review URL: https://codereview.chromium.org/608253002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| b0a8a377f832c59cee939ad721e1f87d378b7142 |
|
23-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
Patch to create a distinct geometry processor. The vast majority of this patch
is just a rename. The meat is in GrGeometryProcessor, GrProcessor,
GrGL*Processor, GrProcessorStage, Gr*BackendProcessorFactory,
GrProcessUnitTestFactory, and the builders
BUG=skia:
R=bsalomon@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/582963002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 49586bec7383d4ccb81f85f8e2dc4162e2d4f6a8 |
|
16-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
removing GrDrawEffect
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/8ddbe8b9366c8c59c4fb55f01f253de8a0b37d6e
R=bsalomon@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/571163002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 08da4f22d790cfc51bbeb10b4b84dab49cf0eaec |
|
16-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
Revert of removing GrDrawEffect (patchset #4 id:60001 of https://codereview.chromium.org/571163002/)
Reason for revert:
reverting to unblock another revert
Original issue's description:
> removing GrDrawEffect
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/8ddbe8b9366c8c59c4fb55f01f253de8a0b37d6e
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/577593003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 8ddbe8b9366c8c59c4fb55f01f253de8a0b37d6e |
|
16-Sep-2014 |
joshualitt <joshualitt@chromium.org> |
removing GrDrawEffect
BUG=skia:
R=bsalomon@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/571163002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 30ba436f04e61d4505fb854d5fc56079636e0788 |
|
22-Aug-2014 |
joshualitt <joshualitt@chromium.org> |
Initial refactor of shaderbuilder to prepare for geometry shaders
gitignore for eclipse
BUG=skia:
R=bsalomon@google.com, bsalomon@chromium.org
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/491673002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 63e99f7a03b2ac90ae7a00232674fd39c0bdcc68 |
|
21-Jul-2014 |
bsalomon <bsalomon@google.com> |
Allow GrGLEffects to produce variable length keys.
R=robertphillips@google.com, jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/385713005
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 848faf00ec33d39ab3e31e9a11d805cae6ac6562 |
|
11-Jul-2014 |
bsalomon <bsalomon@google.com> |
This moves us towards variable length effect keys. The overall program key now allows for it. After the header it stores an array of offsets to effect keys. This allows us to grab the effect keys to pass to effects when they generate code. It also ensures that we can't get a collision by sets of keys that are different lengths but are the same when appended together.
R=robertphillips@google.com, jvanverth@google.com, egdaniel@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/356513003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 83d081ae1d731b5039e99823620f5e287542ee39 |
|
08-Jul-2014 |
bsalomon <bsalomon@google.com> |
Goodbye GrEffectRef.
Also, reworked some var names and comments around SkShader::asNewEffect.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/374923002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 55fad7af61c21d502acb9891d631e8aa29e3628c |
|
08-Jul-2014 |
bsalomon <bsalomon@google.com> |
Remove GrEffect::CreateEffectRef and GrEffect::AutoEffectRef.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/371103003
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 22900008fb89afe4687a40db8737553ee0e55621 |
|
24-Jun-2014 |
bsalomon <bsalomon@google.com> |
Make GrGLShaderBuilder::fragmentPosition() return a vec4, with 1.0 as the zw components when in the y-flip case. This works around an Adreno driver bug.
Revert "Stop referencing gl_FragCoord z and w components."
This reverts commit 160a52ba217012ed73c240f6d67b3bbf2b4879d6.
R=jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/354663002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| 160a52ba217012ed73c240f6d67b3bbf2b4879d6 |
|
20-Jun-2014 |
bsalomon <bsalomon@google.com> |
Stop referencing gl_FragCoord z and w components.
This fixes an Adreno driver bug where GL programs fail to link if the FS refers to these components of gl_FragCoord.
R=jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/345083002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| aed7007e9e3ee4cdf733bf2f53531fdcd0531b25 |
|
20-Jun-2014 |
krajcevski <krajcevski@google.com> |
Fix shader code
R=bsalomon@google.com, reed@google.com
NOTREECHECKS=true
NOTRYS=true
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/347563006
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|
| f461a8fdf642ba713dcdfb217534652df1eac278 |
|
19-Jun-2014 |
krajcevski <krajcevski@google.com> |
Simple GPU based dithering:
If dithering is turned on, apply an effect that filters the pixel through
the following pipeline:
for each channel c:
1. Compute quantized colors [low, high] that c is between
2. Pick high by flipping a coin weighted by (c - low)
R=bsalomon@google.com, egdaniel@google.com, robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/321253002
/external/skia/src/gpu/effects/GrDitherEffect.cpp
|