5e34167d53c06c3c4512592bd1477fcf2df97172 |
|
18-Oct-2017 |
Brian Osman <brianosman@google.com> |
Convert more code to use color space xform FP Special images, image shaders, and all texture producers, as well as bicubic (which simplifies factory call sites). Bug: skia: Change-Id: I3f7c178060f25db8b659fe66e132f5ea066317b1 Reviewed-on: https://skia-review.googlesource.com/61261 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
aff329b8e9b239bca1d93b13a914fbef45ccf7fe |
|
11-Aug-2017 |
Brian Salomon <bsalomon@google.com> |
Make GrFragmentProcessor be non-refcounted and use std::unique_ptr. Change-Id: I985e54a071338e99292a5aa2f42c92bc115b4008 Reviewed-on: https://skia-review.googlesource.com/32760 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.cpp
|
3f6f965a5a65415c65fe9e64eb41896c66da771d |
|
28-Jul-2017 |
Brian Salomon <bsalomon@google.com> |
Implement clone for 6 additional GrFragmentProcessor subclasses. GrMagnifierEffect GrMorphologyEffect GrBicubicEffect GrGaussianConvolutionFragmentProcessor GrMatrixConvolutionEffect GrTextureDomainEffect Bug: skia: Change-Id: I69721b9b95346b365723e5ee21dff2dee8884466 Reviewed-on: https://skia-review.googlesource.com/27900 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
71603cca8ec779bb3ad5ad18a3dc69e97910fb7c |
|
28-Jul-2017 |
Brian Salomon <bsalomon@google.com> |
Remove the now unused GR_PROXY_MOVE macro Change-Id: I9655a3032aea9c40e87742c76ad9a9ab42a159dc Reviewed-on: https://skia-review.googlesource.com/27841 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
6cd51b51d6603a3100b147c45f38697f2f199fc6 |
|
27-Jul-2017 |
Brian Salomon <bsalomon@google.com> |
Remove GrSingleTextureEffect Change-Id: I510cc0657f9433b206dc2ab643fa557667263294 Reviewed-on: https://skia-review.googlesource.com/27180 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
c686ce39f06d556d55befd290e0eb82851c7d33b |
|
21-Jul-2017 |
Robert Phillips <robertphillips@google.com> |
Smaller fragment from "Reduce dependence on GrSurface's origin field" TBR=bsalomon@google.com Change-Id: I996226ffaf84ca357740a6da9bb80a3b8febb379 Reviewed-on: https://skia-review.googlesource.com/25745 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
96be9df1300b2281641b038ef11064157f6e0b41 |
|
21-Jul-2017 |
Robert Phillips <robertphillips@google.com> |
Revert "Reduce dependence on GrSurface's origin field" This reverts commit 467022b1861033d968195687da15270c208279ff. Reason for revert: GrAHardwareBufferImageGenerator.cpp Original change's description: > Reduce dependence on GrSurface's origin field > > Unfortunately, GrGPU and its ilk are still using the GrSurface's origin a lot. I will clean that up in a second CL. > > Change-Id: Iba729440ce8ea8d24bb7f4e5de55ed576a0f176d > Reviewed-on: https://skia-review.googlesource.com/24700 > Commit-Queue: Robert Phillips <robertphillips@google.com> > Reviewed-by: Brian Salomon <bsalomon@google.com> TBR=bsalomon@google.com,robertphillips@google.com Change-Id: I1b3f5c3b82d250ac164beb1d5c83abb6c3c6ab3b No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/25620 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
467022b1861033d968195687da15270c208279ff |
|
21-Jul-2017 |
Robert Phillips <robertphillips@google.com> |
Reduce dependence on GrSurface's origin field Unfortunately, GrGPU and its ilk are still using the GrSurface's origin a lot. I will clean that up in a second CL. Change-Id: Iba729440ce8ea8d24bb7f4e5de55ed576a0f176d Reviewed-on: https://skia-review.googlesource.com/24700 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.cpp
|
646e4293f06d9de6d44dbfa3c32cdc15a6f5906e |
|
13-Jun-2017 |
Robert Phillips <robertphillips@google.com> |
Retract GrTexture*.h & GrRenderTarget*.h from other headers This does push some additional work (& includes) into the .cpp files. Change-Id: I27c847e371802270d13594dcc22aae44039990bb Reviewed-on: https://skia-review.googlesource.com/19660 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.cpp
|
2983f4022d756def4f93579ee519cd31c8f24d61 |
|
08-May-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Re-land of "eliminated GrGLSLExpr". This change is exactly the same as the last time it was landed; I believe the underlying optimizer bug that was causing this to cause problems has been fixed by a prior CL. Bug: skia: Change-Id: I5436422f094ea758caa3cd69e9338db31b1f93fa Reviewed-on: https://skia-review.googlesource.com/15768 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
421fe443f833c916e6c3616f92570c8920b030f9 |
|
04-May-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "Revert "Revert "eliminated GrGLSLExpr""" This reverts commit 796001c82eca5651bc6a221204f6186918781daf. Reason for revert: looks to be causing problems in Chrome (https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Trusty__dbg_/1553/layout-test-results/results.html) Original change's description: > Revert "Revert "eliminated GrGLSLExpr"" > > This reverts commit 5e550ab57e0204bfadd2cb69c47d2a85e38d6a4c. > > Bug: skia: > Change-Id: I4705e47dbd209aa8f43db3d28c856bd3aa9e49ab > Reviewed-on: https://skia-review.googlesource.com/15187 > Reviewed-by: Ethan Nicholas <ethannicholas@google.com> > Commit-Queue: Ethan Nicholas <ethannicholas@google.com> > TBR=ethannicholas@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I6455a4f16b2dc0d6d1265541f7117e0cfb8dd91c Reviewed-on: https://skia-review.googlesource.com/15309 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
796001c82eca5651bc6a221204f6186918781daf |
|
03-May-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "Revert "eliminated GrGLSLExpr"" This reverts commit 5e550ab57e0204bfadd2cb69c47d2a85e38d6a4c. Bug: skia: Change-Id: I4705e47dbd209aa8f43db3d28c856bd3aa9e49ab Reviewed-on: https://skia-review.googlesource.com/15187 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
5e550ab57e0204bfadd2cb69c47d2a85e38d6a4c |
|
02-May-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "eliminated GrGLSLExpr" This reverts commit 93f20f5629e52eed732d2b9d6dbbb351cc30b2cd. Reason for revert: Mismerge readded deleted files. Original change's description: > eliminated GrGLSLExpr > > Now that skslc performs all of the optimizations (and then some) that > GrGLSLExpr is responsible for, it's just extra work for no benefit. > > Bug: skia: > Change-Id: I40b0629e00a33873ed9fc6c0a9f41d8350221f9a > Reviewed-on: https://skia-review.googlesource.com/14560 > Commit-Queue: Ethan Nicholas <ethannicholas@google.com> > Reviewed-by: Brian Salomon <bsalomon@google.com> > TBR=bsalomon@google.com,ethannicholas@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: Ia8b723594527afe34489fc78a4b49039081b6390 Reviewed-on: https://skia-review.googlesource.com/15154 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
93f20f5629e52eed732d2b9d6dbbb351cc30b2cd |
|
02-May-2017 |
Ethan Nicholas <ethannicholas@google.com> |
eliminated GrGLSLExpr Now that skslc performs all of the optimizations (and then some) that GrGLSLExpr is responsible for, it's just extra work for no benefit. Bug: skia: Change-Id: I40b0629e00a33873ed9fc6c0a9f41d8350221f9a Reviewed-on: https://skia-review.googlesource.com/14560 Commit-Queue: Ethan Nicholas <ethannicholas@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
ab015efc48c462ffdffebb45c02cd19efb254983 |
|
04-Apr-2017 |
Brian Salomon <bsalomon@google.com> |
Move the ability to access textures, buffers, and image storages out from GrProcessor. GrXferProcessor can no longer use this functionality so it is moved to a new intermediate class inherited by GrFragmentProcessor and GrPrimitiveProcessor. Change-Id: I4f30c89bdceb2d77b602bf0646107e0780881c26 Reviewed-on: https://skia-review.googlesource.com/11202 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
3798c86f6885f0b47fb2e659a43b48a4468a97ef |
|
27-Mar-2017 |
Robert Phillips <robertphillips@google.com> |
Remove GrFragmentProcessor-derived class' GrTexture-based ctors Split out into: https://skia-review.googlesource.com/c/8881/ (Switch GrTextureStripAtlas over to GrTextureProxies) https://skia-review.googlesource.com/c/8942/ (Wrap cached GrTextures in GrTextureProxies (e.g., blur profiles, nine-patch blurs, etc.)) https://skia-review.googlesource.com/c/8997/ (Clean up/remove unused GrFragmentProcessor-derived ctors) https://skia-review.googlesource.com/c/9191/ (Switch SkImageGenerator over to generating GrTextureProxies) https://skia-review.googlesource.com/c/9448/ (Switch GrYUVProvider over to GrTextureProxies) https://skia-review.googlesource.com/c/9559/ (Preparatory Proxification) https://skia-review.googlesource.com/c/9626/ (Consolidate Proxy caching code in GrResourceProvider) https://skia-review.googlesource.com/c/9683/ (More pre-emptive proxification) https://skia-review.googlesource.com/c/9917/ (Make experimental Perlin noise shader take texture proxies) https://skia-review.googlesource.com/c/9961/ (rename makeCopyForTextureParams to isACopyNeededForTextureParams) https://skia-review.googlesource.com/c/9945/ (Make SkImageCacherator be deferred) https://skia-review.googlesource.com/c/10180/ (Add new proxy-based DetermineDomainMode w/ test) Change-Id: Ia33389d92360e542a9d2bf395948deb04d017465 Reviewed-on: https://skia-review.googlesource.com/8823 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.cpp
|
c624d9d212c4168fc6c202a8535ddff8a3bfb16a |
|
08-Mar-2017 |
Brian Osman <brianosman@google.com> |
Refactor GrColorSpaceXformHelper Nonlinear blending mode is going to (sometimes) require additional uniforms and shader code for color space transformation. This change just alters the usage of the helper struct so that we can hide any new logic (without having to change all the FPs that use it). BUG=skia: Change-Id: I913478a387973f5bad5aa09a29f85d21daacab94 Reviewed-on: https://skia-review.googlesource.com/9414 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
40fd7c94c24bb30d888c3d85a79cbb96c7fbf800 |
|
30-Jan-2017 |
Robert Phillips <robertphillips@google.com> |
Push GrTextureProxy down to more effects Change-Id: Ie3f32a88f25af082c25bc6daf3fe24e303e80f9e Reviewed-on: https://skia-review.googlesource.com/7616 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
587e08f361ee3e775a6bbc6dca761dbba82e422c |
|
27-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Start of rewrite of GrFragmentProcessor optimizations."" This reverts commit 052fd5158f7f85e478a9f87c45fecaacf7d0f5f3. Disables the test (of unused code) until platform-specific issues are addressed. Change-Id: I7aa23a07954fccf382aa07d28afcbffb0bebcd6d Reviewed-on: https://skia-review.googlesource.com/7656 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
052fd5158f7f85e478a9f87c45fecaacf7d0f5f3 |
|
27-Jan-2017 |
Ethan Nicholas <ethannicholas@google.com> |
Revert "Start of rewrite of GrFragmentProcessor optimizations." This reverts commit 85eb4226a4cd8c10a0e3f3ba2f3a60efbb2dd61b. Reason for revert: test failures on Windows, e.g. https://chromium-swarm.appspot.com/task?id=33f9527484414110&refresh=10 Original change's description: > Start of rewrite of GrFragmentProcessor optimizations. > > This adds a replacement for computeInvariantOutput buts does not use it yet. The replacement allows for three types of optimizations: > > * known input color -> known output color for GrFP elimination > * tracking of whether all color processors modulate their input for the "tweak alpha" optimziation > * opaqueness tracking > > This loses some of the generality of computInvariantOutput. It does not track the known output status of individual color components (other than opaque alpha). It does not track whether GrFragmentProcessors read their input color. It doesn't allow a processor that will receive non-constant output to advertise that it produces a constant output. These could probably be added back in the unlikely case that they prove valuable. > > Unlike computeInvariantOutput the optimizations are decided at instantiation time and constant colors are expressed as GrColor4f rather than GrColor. > > Change-Id: I684d3f9050693dde2d28154fa695e049ed8cf61a > Reviewed-on: https://skia-review.googlesource.com/7481 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> > TBR=egdaniel@google.com,bsalomon@google.com,brianosman@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I2390df257456013fa74137cb5d7b5a93820c291e Reviewed-on: https://skia-review.googlesource.com/7652 Commit-Queue: Ethan Nicholas <ethannicholas@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
85eb4226a4cd8c10a0e3f3ba2f3a60efbb2dd61b |
|
26-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Start of rewrite of GrFragmentProcessor optimizations. This adds a replacement for computeInvariantOutput buts does not use it yet. The replacement allows for three types of optimizations: * known input color -> known output color for GrFP elimination * tracking of whether all color processors modulate their input for the "tweak alpha" optimziation * opaqueness tracking This loses some of the generality of computInvariantOutput. It does not track the known output status of individual color components (other than opaque alpha). It does not track whether GrFragmentProcessors read their input color. It doesn't allow a processor that will receive non-constant output to advertise that it produces a constant output. These could probably be added back in the unlikely case that they prove valuable. Unlike computeInvariantOutput the optimizations are decided at instantiation time and constant colors are expressed as GrColor4f rather than GrColor. Change-Id: I684d3f9050693dde2d28154fa695e049ed8cf61a Reviewed-on: https://skia-review.googlesource.com/7481 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
67c18d6b5188a0497f6912a73d964c763d2f8f84 |
|
20-Jan-2017 |
Robert Phillips <robertphillips@google.com> |
Continue making Ganesh use absolute texture coordinates - take 2 The idea here is that the GrCoordTransform will actually hold a GrTextureProxy (rather than a GrTexture) and then, in GrGLSLPrimitiveProcessor::GetTransformMatrix, use the instantiated width & height (when uploading the transform matrix) Relanding of: https://skia-review.googlesource.com/c/6977/ Change-Id: Ibc9b9e354f7fc23b1a6e6e4fe7c9fe3cef771c02 Reviewed-on: https://skia-review.googlesource.com/7265 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
a7d61a65ee7f7b2a2f6fce59faaca0762cef4c67 |
|
17-Jan-2017 |
Joe Gregorio <jcgregorio@google.com> |
Revert "Continue making Ganesh use absolute texture coordinates" This reverts commit 4493a9dc9df8cf306a974bde7e0e9c5470ebcdc9. Reason for revert: Continuing failures https://luci-milo.appspot.com/swarming/task/33c693c3539a1410 Change-Id: Id371d6df6a875a5f6872f4ebd91b08a354d949de Reviewed-on: https://skia-review.googlesource.com/7125 Commit-Queue: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
4493a9dc9df8cf306a974bde7e0e9c5470ebcdc9 |
|
17-Jan-2017 |
Robert Phillips <robertphillips@google.com> |
Continue making Ganesh use absolute texture coordinates The idea here is that the GrCoordTransform will actually hold a GrTextureProxy (rather than a GrTexture) and then, in GrGLSLPrimitiveProcessor::GetTransformMatrix, use the instantiated width & height (when uploading the transform matrix) Change-Id: Ibac3a540fcb1967ceef571157d41c1937acfadf8 Reviewed-on: https://skia-review.googlesource.com/6977 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
e98234f231d66848e149db683c11b6388e10b233 |
|
09-Jan-2017 |
Robert Phillips <robertphillips@google.com> |
Start making texture coordinates be absolute The idea here is that we will pass GrTextureProxys in (where we're currently passing GrTextures) and defer the normalization until the texture is actually instantiated (and possibly move it to the GPU entirely) This CL does (intentionally) change the texturedomaineffect GM but I believe the new behavior is more correct. Change-Id: I4e0510b3dfb65ff0d0ee5921f9a6f94151e602d3 Reviewed-on: https://skia-review.googlesource.com/6807 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
a3fa2efcfc376b4823fe75d2027a74a54a639880 |
|
06-Jan-2017 |
Brian Osman <brianosman@google.com> |
Revert "Revert "Hard code bicubic coefficients in the shader"" This reverts commit b38db9a48db09b233309dca4f0858c1f0976734d. Reason for revert: Nevermind. The regression was actually a bug fix! Original change's description: > Revert "Hard code bicubic coefficients in the shader" > > This reverts commit 81ad5f744a775826c0d68bf035e891ae178f89d2. > > Reason for revert: Serious regressions on Tegra3, and significant regression on Adreno 530. > > Original change's description: > > Hard code bicubic coefficients in the shader > > > > BUG=skia: > > > > Change-Id: Ie231a9bd2ea2083005a595e9e426590da740b2ed > > Reviewed-on: https://skia-review.googlesource.com/6619 > > Reviewed-by: Robert Phillips <robertphillips@google.com> > > Commit-Queue: Brian Osman <brianosman@google.com> > > > > TBR=bsalomon@google.com,robertphillips@google.com,brianosman@google.com,reviews@skia.org > BUG=skia: > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > > Change-Id: I62b26f2a67e8a245c36bb6098c5cb9fbed21bb8f > Reviewed-on: https://skia-review.googlesource.com/6630 > Commit-Queue: Brian Osman <brianosman@google.com> > Reviewed-by: Brian Osman <brianosman@google.com> > TBR=bsalomon@google.com,robertphillips@google.com,reviews@skia.org,brianosman@google.com BUG=skia: NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I5d6d59f7079aca53b60472e1add75b319068fa67 Reviewed-on: https://skia-review.googlesource.com/6682 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
b38db9a48db09b233309dca4f0858c1f0976734d |
|
05-Jan-2017 |
Brian Osman <brianosman@google.com> |
Revert "Hard code bicubic coefficients in the shader" This reverts commit 81ad5f744a775826c0d68bf035e891ae178f89d2. Reason for revert: Serious regressions on Tegra3, and significant regression on Adreno 530. Original change's description: > Hard code bicubic coefficients in the shader > > BUG=skia: > > Change-Id: Ie231a9bd2ea2083005a595e9e426590da740b2ed > Reviewed-on: https://skia-review.googlesource.com/6619 > Reviewed-by: Robert Phillips <robertphillips@google.com> > Commit-Queue: Brian Osman <brianosman@google.com> > TBR=bsalomon@google.com,robertphillips@google.com,brianosman@google.com,reviews@skia.org BUG=skia: NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I62b26f2a67e8a245c36bb6098c5cb9fbed21bb8f Reviewed-on: https://skia-review.googlesource.com/6630 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
81ad5f744a775826c0d68bf035e891ae178f89d2 |
|
05-Jan-2017 |
Brian Osman <brianosman@google.com> |
Hard code bicubic coefficients in the shader BUG=skia: Change-Id: Ie231a9bd2ea2083005a595e9e426590da740b2ed Reviewed-on: https://skia-review.googlesource.com/6619 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
0857527e8676083b38975b0a5d0e2f4c19fe76f3 |
|
21-Dec-2016 |
Brian Osman <brianosman@google.com> |
Cleanup GrBicubicEffect - The base class already stores a color space xform, so just forward our factory argument and use that one. - We had factories that took arbitrary coefficients, but they were only used in the unit test. Make the assumption of Mitchell's coefficients really hard-coded, which lets us skip transposing, storing, comparing, etc... BUG=skia: Change-Id: Ia1d4f72d32c201cba1da24d61b90250f383336d1 Reviewed-on: https://skia-review.googlesource.com/6394 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
864f6bde37273f1f66a74acf0a19eee1210919e1 |
|
06-Dec-2016 |
Brian Osman <brianosman@google.com> |
Add comment explaining the derivation of our Mitchell coefficients BUG=skia: Change-Id: I8866df425ee9837e75f0b2f76777f7e5d68fb21d Reviewed-on: https://skia-review.googlesource.com/5624 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
1edc5b92fecefb79f01cf0e302646eacf32b06c7 |
|
29-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Rename vars and functions from 'glslcaps'->'shadercaps' Change-Id: I34d226d866dd4ba8e5f33f95cb8cbf01292f9314 Reviewed-on: https://skia-review.googlesource.com/5312 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.cpp
|
99938a8ef24e2dd5b39f78638742e9b50ab6d9bf |
|
21-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Merge GrGLSLShaderVar and GrShaderVar GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5087 Change-Id: Ib8943a1da1ea495554feaf5b0992b94fbb9539ab Reviewed-on: https://skia-review.googlesource.com/5087 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
e2f732f475fbeff9cce2dd4023a183fa2492d2ad |
|
03-Oct-2016 |
Brian Osman <brianosman@google.com> |
Start supplying random color space xforms to FP tests Added helper to create random GrColorSpaceXforms in unit tests, and hooked it up for the FPs that currently accept one. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2873 Change-Id: Iaf93e379e405fbf745f5e0fd23b4daf017355966 Reviewed-on: https://skia-review.googlesource.com/2873 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
1a1aa9303484106a955e5549bf8ae24950f54e7a |
|
12-Sep-2016 |
bsalomon <bsalomon@google.com> |
Remove unneeded GrGLSLTransformedCoordsArray type Rename GrGLSLFragmentBuilder::ensureFSCoords2D to ensureCoords2D and make it take an arbitrary GrShaderVar. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2324663004 Review-Url: https://codereview.chromium.org/2324663004
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
5192475bd8cb98e8e0c1192ab5ece7b8595701d6 |
|
12-Sep-2016 |
brianosman <brianosman@google.com> |
Two changes: 1. Remove special premul handling from gamut xform code Alpha is a constant, so the gamut transformation results remain unchanged (it distributes across the linear matrix multiply). 2. Use SkMatrix44 rather than array of floats Preserves semantic intention, and makes upcoming code (where we transform colors on the CPU by that matrix) simpler. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2329553002 Review-Url: https://codereview.chromium.org/2329553002
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
77320dbabcddf05c0a1489eaf1f496729dc8de0e |
|
07-Sep-2016 |
brianosman <brianosman@google.com> |
Add color gamut xform helpers to GrGLSLShaderBuilder New helper functions inject the necessary shader function. Texture lookup functions can now insert the gamut xform at the appropriate place, too. As written, could be used to transform non-texture colors (e.g. vertex colors) as well. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2180803005 Review-Url: https://codereview.chromium.org/2180803005
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
9f97882322929ee39405acb44cbf155854b38091 |
|
27-Jul-2016 |
brianosman <brianosman@google.com> |
Simplify color space xform storage and usage (float[] vs. SkMatrix44) Public API is really just internal. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2185533005 TBR=bsalomon@google.com Review-Url: https://codereview.chromium.org/2185533005
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
54f30c13fc0a5d89797fc9be5f0fb1050d96b6f4 |
|
18-Jul-2016 |
brianosman <brianosman@google.com> |
Introduce GrColorSpaceXform, for gamut conversion on textures GrTextureAccess optionally includes an instance, computed from the src and dst color spaces. In all common cases (no color space for either src or dst, or same color space for both), no object is allocated. This change is orthogonal to my attempts to get color space attached to render targets - regardless of how we choose to do that, this will give us the source color space at all points where we are connecting src to dst. There are many dangling injection points where I've been inserting nullptr, but I have a record of all of them. Additionally, there are now three places (the most common simple paths for bitmap/image rendering) where things are plumbed enough that I expect to have access to the dst color space (all marked with XFORMTODO). In addition to getting the dst color space, I need to inject shader code and uniform uploading for appendTextureLookup and friends. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2154753003 Review-Url: https://codereview.chromium.org/2154753003
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
4a339529612a43871d021877e58698e067d6c4cd |
|
06-Oct-2015 |
bsalomon <bsalomon@google.com> |
Bye bye processor data manager. Review URL: https://codereview.chromium.org/1388113002
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
5f10b5c1b5744106312e24835d235b72fdba5802 |
|
09-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
More threading of GrProcessorDataManager TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1230813003
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
56995b5cc00c9c83bd5fcf86bca9a67e939a96cb |
|
12-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP BUG=skia: Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da Review URL: https://codereview.chromium.org/791743003
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
8c0f615fdd41b8b8048bf398791bb2138c511826 |
|
10-Dec-2014 |
joshualitt <joshualitt@google.com> |
Revert of Remove GP from drawstate, revision of invariant output for GP (patchset #9 id:160001 of https://codereview.chromium.org/791743003/) Reason for revert: breaks mac Original issue's description: > Remove GP from drawstate, revision of invariant output for GP > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/794843002
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
c6bc58eded89b0c0a36b8e20e193c200f297a0da |
|
10-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP BUG=skia: Review URL: https://codereview.chromium.org/791743003
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
422f56f6e51c2f6a6ab425573b4d790f0157f883 |
|
09-Dec-2014 |
bsalomon <bsalomon@google.com> |
Make addUniform take a precision Review URL: https://codereview.chromium.org/788733003
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
420d7e9a79358908850c74192b4949375563449a |
|
16-Oct-2014 |
bsalomon <bsalomon@google.com> |
Auto-compare GrProcessors' texture accesses in isEqual(). R=joshualitt@google.com Review URL: https://codereview.chromium.org/654313002
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
838f62db0a77487f6732ed13a0972a1e646ae16a |
|
05-Aug-2014 |
bsalomon <bsalomon@google.com> |
Replace op== with CombineIfPossible in GrDrawState. R=egdaniel@google.com, robertphillips@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/439853002
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
7510b224e52b9518a8ddf7418db0e9c258f79539 |
|
30-Jul-2014 |
kkinnunen <kkinnunen@nvidia.com> |
Rename GrGLUniformManager to GrGLProgramDataManager Rename GrGLUniformManager to GrGLProgramDataManager in anticipation that the class would be used to manage shader resources that are not uniforms. This is needed in order to implement NVPR on GLES. R=bsalomon@google.com Author: kkinnunen@nvidia.com Review URL: https://codereview.chromium.org/365853002
/external/skia/src/gpu/effects/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.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/GrBicubicEffect.cpp
|
9927bd355fc1557aa2c98900266ba6f49f806889 |
|
20-May-2014 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Centralize decision about whether to do bicubic filtering, and fallbacks to mip, bilerp, or nearest R=robertphillips@google.com, humper@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/282293004 git-svn-id: http://skia.googlecode.com/svn/trunk@14809 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
7d7f31433b627e62f518e9186d3f2d9bd44662e0 |
|
16-Dec-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Use GrTextureDomain in GrBicubicEffect to perform non-bleeding HQ filter drawBitmap. R=senorblanco@chromium.org, robertphillips@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/99203011 git-svn-id: http://skia.googlecode.com/svn/trunk@12687 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
bc91fd71faa2c5fd14423fecd18e50701c922ced |
|
10-Dec-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Make GrBicubicEffect take tile modes rather than GrTextureParams. GrTextureParams allows the caller to override the filter mode. But no filtering other than nearest neighbor makes sense. Also removes coord set from creation signature since it is unused. R=robertphillips@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/99523008 git-svn-id: http://skia.googlecode.com/svn/trunk@12591 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
dec61503d02862760f3c91203a698636a02c882a |
|
02-Dec-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
HQ filtering for tiled/bleed drawBitmap sample at pix center in bicubic R=senorblanco@chromium.org, jvanverth@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/83153006 git-svn-id: http://skia.googlecode.com/svn/trunk@12440 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
d3baf20dd1de9940717dd50b5c9ff6061561342e |
|
07-Nov-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Added support for Chrome's gpu command buffer extension BindUniformLocation. R=bsalomon@google.com, bsalomon Author: skaslev@chromium.org Review URL: https://codereview.chromium.org/62163004 git-svn-id: http://skia.googlecode.com/svn/trunk@12178 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
759befb72c558202de65a480c44d7b27428ec312 |
|
23-Oct-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
use the supplied matrix instead of the texture w/h div for the bicubic GPU effect BUG= R=bsalomon@google.com, caryclark@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/37003005 git-svn-id: http://skia.googlecode.com/svn/trunk@11921 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
a34995e18b1f0a7d8c9f23451718bb30ff0105b0 |
|
23-Oct-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Implement SkColorFilter as a GrGLEffect Adds GrEffect::willUseInputColor() which indicates whether or not the input color affects the output of the effect. This is needed for certain Xfermodes, such as kSrc_Mode. For these modes the color filter will not use the input color. An effect with GrEffect::willUseInputColor() true will cause all color or coverage effects before it to be discarded, as their computations cannot affect the output. In these cases program is marked as having white input color. This fixes an assert when Skia is compiled in a mode that prefers using uniforms instead of attributes for constants. (Flags GR_GL_USE_NV_PATH_RENDERING or GR_GL_NO_CONSTANT_ATTRIBUTES). Using attributes hides the problem where the fragment shader does not need input color for color filters that ignore DST part of the filter. The assert would be hit when uniform manager tries to bind an uniform which has been optimized away by the shader compiler. Adds specific GrGLSLExpr4 and GrGLSLExpr1 classes. This way the GLSL expressions like "(v - src.a)" can remain somewhat readable in form of "(v - src.a())". The GrGLSLExpr<typename> template implements the generic functionality, GrGLSLExprX is the specialization that exposes the type-safe interface to this functionality. Also adds operators so that GLSL binary operators of the form "(float * vecX)" can be expressed in C++. Before only the equivalent "(vecX * float)" was possible. This reverts the common blending calculations to more conventional order, such as "(1-a) * c" instead of "c * (1-a)". Changes GrGLSLExpr1::OnesStr from 1 to 1.0 in order to preserve the color filter blending formula string the same (with the exception of variable name change). Shaders change in case of input color being needed: - vec4 filteredColor; - filteredColor = (((1.0 - uFilterColor.a) * output_Stage0) + uFilterColor); - fsColorOut = filteredColor; + vec4 output_Stage1; + { // Stage 1: ModeColorFilterEffect + output_Stage1 = (((1.0 - uFilterColor_Stage1.a) * output_Stage0) + uFilterColor_Stage1); + } + fsColorOut = output_Stage1; Shaders change in case of input color being not needed: -uniform vec4 uFilterColor; -in vec4 vColor; +uniform vec4 uFilterColor_Stage0; out vec4 fsColorOut; void main() { - vec4 filteredColor; - filteredColor = uFilterColor; - fsColorOut = filteredColor; + vec4 output_Stage0; + { // Stage 0: ModeColorFilterEffect + output_Stage0 = uFilterColor_Stage0; + } + fsColorOut = output_Stage0; } R=bsalomon@google.com, robertphillips@google.com, jvanverth@google.com Author: kkinnunen@nvidia.com Review URL: https://codereview.chromium.org/25023003 git-svn-id: http://skia.googlecode.com/svn/trunk@11912 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
77af6805e5faea1e2a5c0220098aec9082f3a6e5 |
|
02-Oct-2013 |
bsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Make GPU coord transforms automatic Adds a GrCoordTransform class and updates the framework to handle coord transforms similar to how it handles textures with GrTextureAccess. Renames GrGLEffectMatrix to GrGLCoordTransform and slightly repurposes it to be used by the framework instead of effects. R=bsalomon@google.com, robertphillips@google.com Review URL: https://codereview.chromium.org/24853002 git-svn-id: http://skia.googlecode.com/svn/trunk@11569 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
e0e7cfe44bb9d66d76120a79e5275c294bacaa22 |
|
09-Sep-2013 |
commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> |
Change old PRG to be SkLCGRandom; change new one to SkRandom The goal here is to get people to start using the new random number generator, while leaving the old one in place so we don't have to rebaseline GMs. R=reed@google.com, bsalomon@google.com Author: jvanverth@google.com Review URL: https://chromiumcodereview.appspot.com/23576015 git-svn-id: http://skia.googlecode.com/svn/trunk@11169 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
d1af237d7d52e23bc08ccaf73eddaadddeab6758 |
|
04-Sep-2013 |
humper@google.com <humper@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
fix the onIsEqual method on the new bicubic effect to consider the texture accesses as well as the textures git-svn-id: http://skia.googlecode.com/svn/trunk@11090 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|
3aad3b01afc77993ff051c02e49186294e312980 |
|
04-Sep-2013 |
humper@google.com <humper@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> |
add support for high quality image filtering on the GPU R=bsalomon@google.com, reed@google.com Review URL: https://codereview.chromium.org/23779003 git-svn-id: http://skia.googlecode.com/svn/trunk@11087 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrBicubicEffect.cpp
|