History log of /external/skia/src/gpu/GrProcessor.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
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/GrProcessor.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/GrProcessor.cpp
5efd5ea4e146c47394d16c8b6f876264822a0bee 30-May-2017 Robert Phillips <robertphillips@google.com> Update clearOp for split-OpList world (take 3)

It would reduce a lot of noise if the GrRenderTargetOpList kept a pointer to the GrCaps but, for now, I'm trying to shrink the GrRTOpList, not expand it.

Reland of: https://skia-review.googlesource.com/c/17323/ (Update clearOp for split-OpList world)

Change-Id: Ia61ce4d6ce245380fc9651928a46c22039fc12cf
Reviewed-on: https://skia-review.googlesource.com/18026
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
f2ae2b2dbcbdcc31b72fa86ed79f1cc02220528d 30-May-2017 Mike Reed <reed@google.com> move perlinenoise2 into effects

Bug: skia:
Change-Id: I5c178bdc5901d15c6924b3fb1f29119ab3cc701d
Reviewed-on: https://skia-review.googlesource.com/18131
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
/external/skia/src/gpu/GrProcessor.cpp
bb581ce30f55360fd3a12e7f5aa1fe324b16d085 29-May-2017 Robert Phillips <robertphillips@google.com> Convert DstTexture to DstProxy (take 2)

The last GrTexture-based TextureSampler::reset call must be removed before the TextureSamplers can become purely GrTextureProxy-backed

Reland of: https://skia-review.googlesource.com/c/16908/ (Convert DstTexture to DstProxy)

Split out of: https://skia-review.googlesource.com/c/10484/ (Omnibus: Push instantiation of GrTextures later (post TextureSampler))

Change-Id: I3a497b6a950fad899f23882c0a9552894ef640f8
Reviewed-on: https://skia-review.googlesource.com/17205
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
bb5711a5e4b9c83f0fc49f2d4ee19ca1e4592e14 17-May-2017 Brian Salomon <bsalomon@google.com> Remove GrSurfaceDesc member from GrSurfaceProxy.

Stores the config, origin, and dimensions in GrSurfaceProxy, sample count in GrRenderTargetProxy, and "was constructed with mip maps" in GrTextureProxy.

Change-Id: Iee058674dce49107a991cca9d083cd33e3572809
Reviewed-on: https://skia-review.googlesource.com/17209
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
c375704a20a811b9b8d294533f166bed44b7618d 17-May-2017 Robert Phillips <robertphillips@google.com> Revert "Convert DstTexture to DstProxy"

This reverts commit 87f7f1c3ce519115141b40f1d8faede437c8f357.

Reason for revert: grumble, grumble

Original change's description:
> Convert DstTexture to DstProxy
>
> The last GrTexture-based TextureSampler::reset call must be removed before the TextureSamplers can become purely GrTextureProxy-backed
>
> Split out of: https://skia-review.googlesource.com/c/10484/ (Omnibus: Push instantiation of GrTextures later (post TextureSampler))
>
> Change-Id: Ic1435177d8b5d9bd3fc38b4903c9baae8205cfb0
> Reviewed-on: https://skia-review.googlesource.com/16908
> Reviewed-by: Greg Daniel <egdaniel@google.com>
> Commit-Queue: Robert Phillips <robertphillips@google.com>
>

TBR=egdaniel@google.com,robertphillips@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Change-Id: I9af52bb222bd2d8cc696250a9efb62afb80edba1
Reviewed-on: https://skia-review.googlesource.com/17203
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
87f7f1c3ce519115141b40f1d8faede437c8f357 15-May-2017 Robert Phillips <robertphillips@google.com> Convert DstTexture to DstProxy

The last GrTexture-based TextureSampler::reset call must be removed before the TextureSamplers can become purely GrTextureProxy-backed

Split out of: https://skia-review.googlesource.com/c/10484/ (Omnibus: Push instantiation of GrTextures later (post TextureSampler))

Change-Id: Ic1435177d8b5d9bd3fc38b4903c9baae8205cfb0
Reviewed-on: https://skia-review.googlesource.com/16908
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
8a02f65c5cc16d010f188c34861b03d96cb8ec10 12-May-2017 Robert Phillips <robertphillips@google.com> Switch ImageStorageAccess over to GrTextureProxies

Split out of: https://skia-review.googlesource.com/c/10484/ (Omnibus: Push instantiation of GrTextures later (post TextureSampler))

Change-Id: I341de6ae121620d30e50bff21450878a18bdf4f2
Reviewed-on: https://skia-review.googlesource.com/16714
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
49081d13bacbea0631351dc5031d98e3fbb3ec45 08-May-2017 Robert Phillips <robertphillips@google.com> Expand GrTextureProxy to handle highestFilterMode


Once TextureProxies aren't instantiated in the TextureSamplers, the they will need to be able to supply this information.

split out of: https://skia-review.googlesource.com/c/10484/ (Omnibus: Push instantiation of GrTextures later (post TextureSampler))
Change-Id: I66555c0746131f565862f7a30d54ff1d458d2062
Reviewed-on: https://skia-review.googlesource.com/15819
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
f5442bb4c152e7c8138c83d27140e55d846f7ea5 17-Apr-2017 Robert Phillips <robertphillips@google.com> Split cleanup changes out of bigger CL (take 2)

This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)

This is a reland of: https://skia-review.googlesource.com/c/13585/ (Split cleanup changes out of bigger CL)

TBR=bsalomon@google.com

Change-Id: Id0d5c764953230412e78ae6aa654f8dfae998c60
Reviewed-on: https://skia-review.googlesource.com/13643
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
e186cfd15f2df43415ae024185bdcf5baa955ec9 17-Apr-2017 Mike Klein <mtklein@google.com> Revert "Split cleanup changes out of bigger CL"

This reverts commit cd11c809f206af0da3ce1779dee3c91193baa7b0.

Reason for revert: assert(rtp->getLastOpList())

Original change's description:
> Split cleanup changes out of bigger CL
>
> This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)
>
> TBR=bsalomon@google.com
>
> Change-Id: I1239e806a30a93d3c17ad933f118b98a5077dd3a
> Reviewed-on: https://skia-review.googlesource.com/13585
> Commit-Queue: Robert Phillips <robertphillips@google.com>
> Reviewed-by: Greg Daniel <egdaniel@google.com>
>

TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Change-Id: Ia3ef7322fe830a6a4dc29226ce931d35d181c6e0
Reviewed-on: https://skia-review.googlesource.com/13641
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
/external/skia/src/gpu/GrProcessor.cpp
cd11c809f206af0da3ce1779dee3c91193baa7b0 17-Apr-2017 Robert Phillips <robertphillips@google.com> Split cleanup changes out of bigger CL

This are split out of: https://skia-review.googlesource.com/c/11581/ (Split up opLists)

TBR=bsalomon@google.com

Change-Id: I1239e806a30a93d3c17ad933f118b98a5077dd3a
Reviewed-on: https://skia-review.googlesource.com/13585
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
63954c9944a5eed4527e0ed368f4e501faf2c625 14-Mar-2017 Brian Osman <brianosman@google.com> GPU version of onMakeColorSpace

New fragment processor that implements end-to-end
color space conversion, with nonlinear blending.

BUG=skia:6242

Change-Id: Ied86170fc28537a2bc209d57530d3ded48b467a9
Reviewed-on: https://skia-review.googlesource.com/9543
Reviewed-by: Brian Salomon <bsalomon@google.com>
Reviewed-by: Matt Sarett <msarett@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
30f9bc69cfc506075e1fce8e7934f941c0203023 22-Feb-2017 Robert Phillips <robertphillips@google.com> Switch GrTextureStripAtlas over to GrTextureProxies

This is split out of: https://skia-review.googlesource.com/c/8823/ (Remove GrFragmentProcessor-derived class' GrTexture-based ctors)

Change-Id: I9f602985b6010fc58b595e2be6d4e67e50179747
Reviewed-on: https://skia-review.googlesource.com/8881
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
901f29ad3e38b7072a2abef5ff1665cd755d21a2 24-Jan-2017 Robert Phillips <robertphillips@google.com> Allow GrSingleTextureEffect to take GrTextureProxies

Change-Id: I1dd441a5838f665c6815a5c629f5763f43f66e09
Reviewed-on: https://skia-review.googlesource.com/7429
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
bc7a4fb06780f9829b4b21470fe6f0503d2297cd 23-Jan-2017 Robert Phillips <robertphillips@google.com> Make GrYUVEffect take GrTextureProxies

This opens the door for swapping all the effects over to taking GrTextureProxies.

Change-Id: I3b03ba93a68f9945c9a8fee008fd170ed57616eb
Reviewed-on: https://skia-review.googlesource.com/7344
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
e57194f74d125c7b0ba767ae5357b97d102792ef 09-Jan-2017 Brian Salomon <bsalomon@google.com> Move resource ref/io manipulation from GrProgramElement to GrProcessor

Change-Id: I783e74107f7d34b57d80b843fe23ce7126b77424
Reviewed-on: https://skia-review.googlesource.com/6816
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
a16339297859f37df69230e64f05624cef511ad3 09-Jan-2017 Brian Salomon <bsalomon@google.com> Revert "Revert "Removing ref counting from GrXPFactory.""

This reverts commit 003312a211e65f35e402d6fe80a32e23d4c94ac4.

Change-Id: Ib41065e5c356d1dd99e70fa10611ac6756c2b79d
Reviewed-on: https://skia-review.googlesource.com/6803
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
003312a211e65f35e402d6fe80a32e23d4c94ac4 09-Jan-2017 Brian Salomon <bsalomon@google.com> Revert "Removing ref counting from GrXPFactory."

This reverts commit a8f80de2bc17672b4b6f26d3cf6b38123ac850c9.

Reason for revert: nanobench failing on windows bots, possibly others

Change-Id: Iacb8c650064a28654c165665be057377ffb02ba5
Reviewed-on: https://skia-review.googlesource.com/6802
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
a8f80de2bc17672b4b6f26d3cf6b38123ac850c9 07-Jan-2017 Brian Salomon <bsalomon@google.com> Removing ref counting from GrXPFactory.

All GrXPFactory instances are static constexpr.

Change-Id: If1086b08534166201e53b3fd9379104e361eb5e6
Reviewed-on: https://skia-review.googlesource.com/6701
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
1c4717b54b21bc6c640864caf600ef16496803ec 06-Jan-2017 Brian Salomon <bsalomon@google.com> Remove arithmetic mode GrXP code.

The arithmetic mode xfermode is only used as an implementation detail of SkXfermodeImageFilter which always uses the arithmetic FP.

Change-Id: I5f9607aa9731a21e6666d9c749dfa705d40d5775
Reviewed-on: https://skia-review.googlesource.com/6688
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/GrProcessor.cpp
f9f451213a3951d8a61568998de2ddbd643f6693 29-Nov-2016 Brian Salomon <bsalomon@google.com> Reland image storage with fixes.

Revert "Revert "Initial OpenGL Image support.""

This reverts commit 59dc41175d99d0a31c046aec0c26c4d82a3a3574.

BUG=skia:

Change-Id: Ibe3c87ce7f746f065fdbcc5a518388cc291112f5
Reviewed-on: https://skia-review.googlesource.com/5131
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
59dc41175d99d0a31c046aec0c26c4d82a3a3574 23-Nov-2016 Brian Salomon <bsalomon@google.com> Revert "Initial OpenGL Image support."

This reverts commit be34882042048db096baca32ddf4a8b472529804.

Reason for revert: Test failures and shader compilation issues on Android.

Original change's description:
> Initial OpenGL Image support.
>
> This change along with recently landed changes is enough to make the new unit test work and not much else. imageLoad is support but not stores or any other image functions (atomics). Barriers in the shading language or the GL API are not yet hooked up.
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4182
>
> Change-Id: I5958b7c89e40ae5ee05f7bbaca3b3738162fe5ce
> Reviewed-on: https://skia-review.googlesource.com/4182
> Reviewed-by: Greg Daniel <egdaniel@google.com>
> Reviewed-by: Chris Dalton <csmartdalton@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>
>

TBR=egdaniel@google.com,bsalomon@google.com,csmartdalton@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Change-Id: I29a2502c296ae39ed9a999957800414ae46e6f0f
Reviewed-on: https://skia-review.googlesource.com/5129
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
be34882042048db096baca32ddf4a8b472529804 22-Nov-2016 Brian Salomon <bsalomon@google.com> Initial OpenGL Image support.

This change along with recently landed changes is enough to make the new unit test work and not much else. imageLoad is support but not stores or any other image functions (atomics). Barriers in the shading language or the GL API are not yet hooked up.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4182

Change-Id: I5958b7c89e40ae5ee05f7bbaca3b3738162fe5ce
Reviewed-on: https://skia-review.googlesource.com/4182
Reviewed-by: Greg Daniel <egdaniel@google.com>
Reviewed-by: Chris Dalton <csmartdalton@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.cpp
b014cca49d684886ec9813d4b47d126bfa48cf4e 18-Nov-2016 Brian Salomon <bsalomon@google.com> Make GrBufferAccess a nested class of GrProcessor

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4978

Change-Id: I08c24e9183108f4dd6068216488fd3ac9b5f3ec2
Reviewed-on: https://skia-review.googlesource.com/4978
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
c03e1c55a79f00d02ab528945425ff50cb700402 17-Oct-2016 Ben Wagner <bungeman@google.com> Re-enable overdraw mode in debugger.

Debugger is the last user of the deprecated SkPaintFilterCanvas
constructor. Stop using it and remove the constructor.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3268

Change-Id: I3e9180d48abdf86cb2c05bd8d95acabcdaa70427
Reviewed-on: https://skia-review.googlesource.com/3268
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Mike Reed <reed@google.com>
/external/skia/src/gpu/GrProcessor.cpp
2ebd0c80a2a9d90a2c2c653f40a2a7205bd2d31b 03-Oct-2016 Brian Salomon <bsalomon@google.com> Remove option to make GrCoordTransforms apply to device positions.

Adds a device space texture decal effect to use for clipping.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2870

Change-Id: Ifcc7617ea87f5a86e301995cba9dfc30a4b0e2c5
Reviewed-on: https://skia-review.googlesource.com/2870
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/GrProcessor.cpp
68440f8ce90a48e14e79b6fe5a6dd47c8c09bfb4 29-Aug-2016 msarett <msarett@google.com> Skip spinlock overhead on Android

Using the spinlock is only necessary when we multiple threads
might use a GrContext. Android uses the GrContext from a
single thread, so these locks are not needed.

This is a temporary fix until we can refactor to avoid
creating GrContexts in a global memory pool.

BUG=skia:5696
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2293633003

Review-Url: https://codereview.chromium.org/2293633003
/external/skia/src/gpu/GrProcessor.cpp
2d1ee7936e3536e45c963db004e3b512bb415fd8 05-May-2016 brianosman <brianosman@google.com> Added --deepColor option to SampleApp, triggers creation of a ten-bit/channel
buffer. (Only on Windows at the moment). Uses new effect to do the final
gamma adjustment

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1919993002

Review-Url: https://codereview.chromium.org/1919993002
/external/skia/src/gpu/GrProcessor.cpp
74b8d323323c8533e3e5cc7719e0bd127aacd829 11-Apr-2016 cdalton <cdalton@nvidia.com> Implement texel buffers

Adds a mechanism for processors to add buffer accesses and implements
them in the GL backend.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1870893002

Review URL: https://codereview.chromium.org/1870893002
/external/skia/src/gpu/GrProcessor.cpp
59dc5424240bf5974d15624c32cfbe0ece54fddc 05-Apr-2016 reed <reed@google.com> remove avoid and pixelxor xfermodes

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1853103005

Review URL: https://codereview.chromium.org/1853103005
/external/skia/src/gpu/GrProcessor.cpp
15923c9e475894d89028b7a6a0b38aeeb9f9e645 29-Feb-2016 mtklein <mtklein@chromium.org> Modernize SkSpinlock.

- Use std::atomic directly.
- No more need for SkPODSpinlock or SK_DECLARE_STATIC_SPINLOCK.

Now simple code like this works as you'd hope:
static SkSpinlock gLock;

That is, it starts unlocked and there's no static initializer.

std::atomic_flag would make this terser and standard-guaranteed,
but ATOMIC_FLAG_INIT caused not-yet-implemented errors on MSVC 2013.
The generated code for this approach is identical.

It appears the implicit constructor is constexpr when all the member
initializers are. I'm hoping this way of producing constexpr constructors
without typing "constexpr" gives us a way to eliminate more SkFoo / SkBaseFoo
distinctions and SK_DECLARE_STATIC_FOO. This was certainly the easiest.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1734383002

Review URL: https://codereview.chromium.org/1734383002
/external/skia/src/gpu/GrProcessor.cpp
afb188de27d047c8327ccc7b099203e8fc2a4129 03-Feb-2016 robertphillips <robertphillips@google.com> Add gpu implementation of SkAvoidXfermode
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658623002

TBR=bsalomon@google.com

Committed: https://skia.googlesource.com/skia/+/15691a055db9b68c9b48f589e48d8a85888cf83f

Review URL: https://codereview.chromium.org/1658623002
/external/skia/src/gpu/GrProcessor.cpp
f16e2a2e92df1eabd04e9e1260ef0b763ecfa426 03-Feb-2016 mtklein <mtklein@google.com> Revert of Add gpu implementation of SkAvoidXfermode (patchset #10 id:170001 of https://codereview.chromium.org/1658623002/ )

Reason for revert:
shader compilation failures

error C1008: undefined variable "null"

https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Debug/builds/4064/steps/dm/logs/stdio

Original issue's description:
> Add gpu implementation of SkAvoidXfermode
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658623002
>
> TBR=bsalomon@google.com
>
> Committed: https://skia.googlesource.com/skia/+/15691a055db9b68c9b48f589e48d8a85888cf83f

TBR=egdaniel@google.com,bsalomon@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/1663093002
/external/skia/src/gpu/GrProcessor.cpp
15691a055db9b68c9b48f589e48d8a85888cf83f 03-Feb-2016 robertphillips <robertphillips@google.com> Add gpu implementation of SkAvoidXfermode
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658623002

TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/1658623002
/external/skia/src/gpu/GrProcessor.cpp
f42fca4027ed4f235ef927786813a5ed0e6652b9 27-Jan-2016 robertphillips <robertphillips@google.com> Add gpu implementation of OverdrawXfermode
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1607253002

Committed: https://skia.googlesource.com/skia/+/8bc3cf88bbf5e5d5724356f076931bb70a6117ba

Review URL: https://codereview.chromium.org/1607253002
/external/skia/src/gpu/GrProcessor.cpp
727b7d27afd01ab76771ba0b3279208cff728d1a 26-Jan-2016 robertphillips <robertphillips@google.com> Add gpu backend for SkPixelXorXfermode
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1631873002

Review URL: https://codereview.chromium.org/1631873002
/external/skia/src/gpu/GrProcessor.cpp
e429c88a811dfa05a26582a06fd53752768aa9f4 22-Jan-2016 robertphillips <robertphillips@google.com> Revert of Add gpu implementation of OverdrawXfermode (patchset #6 id:100001 of https://codereview.chromium.org/1607253002/ )

Reason for revert:
Overdraw Xfer mode using new fangled glsl features

Original issue's description:
> Add gpu implementation of OverdrawXfermode
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1607253002
>
> Committed: https://skia.googlesource.com/skia/+/8bc3cf88bbf5e5d5724356f076931bb70a6117ba

TBR=egdaniel@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.

Review URL: https://codereview.chromium.org/1626443002
/external/skia/src/gpu/GrProcessor.cpp
8bc3cf88bbf5e5d5724356f076931bb70a6117ba 20-Jan-2016 robertphillips <robertphillips@google.com> Add gpu implementation of OverdrawXfermode
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1607253002

Review URL: https://codereview.chromium.org/1607253002
/external/skia/src/gpu/GrProcessor.cpp
f234272cbdcbcd46be462317e75cd138999c9fe9 21-Nov-2015 egdaniel <egdaniel@google.com> Revert of Make specialized SrcOver XPFactory (patchset #3 id:40001 of https://codereview.chromium.org/1455273006/ )

Reason for revert:
breaking some builds

Original issue's description:
> Make specialized SrcOver XPFactory
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/a7006d45217d128a94fa53cb4b827cca79bc7049

TBR=bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1464933003
/external/skia/src/gpu/GrProcessor.cpp
a7006d45217d128a94fa53cb4b827cca79bc7049 20-Nov-2015 egdaniel <egdaniel@google.com> Make specialized SrcOver XPFactory

BUG=skia:

Review URL: https://codereview.chromium.org/1455273006
/external/skia/src/gpu/GrProcessor.cpp
bf87730c7da80616f6ea17765290dbd2c147a669 22-Sep-2015 bsalomon <bsalomon@google.com> Move GrFragmentProcessor implementation to its own cpp file

TBR=joshualitt@google.com

Review URL: https://codereview.chromium.org/1350523004
/external/skia/src/gpu/GrProcessor.cpp
3905c379e8da7c413f6f47398e6f605de84dc8a4 22-Sep-2015 Brian Salomon <bsalomon@google.com> fix factory count

TBR=egdaniel@google.com

Review URL: https://codereview.chromium.org/1357773003 .
/external/skia/src/gpu/GrProcessor.cpp
87ba62e67c9ed0c453754a5341255829075c8e22 22-Sep-2015 bsalomon <bsalomon@google.com> Replace GrExtractAlphaFragmentProcessor with DstIn compose processor

Review URL: https://codereview.chromium.org/1347943003
/external/skia/src/gpu/GrProcessor.cpp
ae4738f677c70f4ec7687422e1510ee3d80d810e 16-Sep-2015 bsalomon <bsalomon@google.com> Create fragment processor for performing input color blend with child processor

The new FP is used to implement SkXM::Mode color filters and SkXM::Mode image filters. Also, these now support all advanced SkXM::Mode xfermodes.

Review URL: https://codereview.chromium.org/1334293003
/external/skia/src/gpu/GrProcessor.cpp
30c4cae7d3a26252e7e45adf6e5722b34adf6848 15-Sep-2015 robertphillips <robertphillips@google.com> Add special case circle blur for Ganesh

This makes the blurcircles bench go from ~33us to ~8us on Windows desktop.

It will require layout test suppressions

Review URL: https://codereview.chromium.org/1311583005
/external/skia/src/gpu/GrProcessor.cpp
506c802a3dd3b6dc8a172621ff2a1f9bda202a13 14-Sep-2015 bsalomon <bsalomon@google.com> Add helper for creating leaf FPs inside GrFP::TestCreate functions

Review URL: https://codereview.chromium.org/1334273003
/external/skia/src/gpu/GrProcessor.cpp
af2533a165ae99f79416b730910731768b0a4ddf 09-Sep-2015 joshualitt <joshualitt@chromium.org> Make GrProcessorDataManager a noop

TBR=bsalomon@google.com
BUG=skia:

Review URL: https://codereview.chromium.org/1323963003
/external/skia/src/gpu/GrProcessor.cpp
036fd8e6f66b53cf87a5f91083cae82f0aeb3635 09-Sep-2015 wangyix <wangyix@google.com> Added TestCreate for SkComposeShader; will pick two random child procs that don't have children of their own. This prevents creating an arbitrarily large tree of procs. Also, it will choose a random coefficient mode for the xfermode.

BUG=skia:4182

Review URL: https://codereview.chromium.org/1306163002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
4204800cd8f311f11491cf2eb7e32ca681f05489 28-Aug-2015 bsalomon <bsalomon@google.com> Convert child FPs from refs to pending executions when parent converts

Review URL: https://codereview.chromium.org/1315923004
/external/skia/src/gpu/GrProcessor.cpp
ac856c97acc84dcb54d9cdb068ec8a02b8869647 27-Aug-2015 bsalomon <bsalomon@google.com> Remove GrStagedProcessor, remove the word Stage as it applies to FPs

Committed: https://skia.googlesource.com/skia/+/24243446cdf7b7e4e132c2a0c387c7723777e0c7

Review URL: https://codereview.chromium.org/1307223004
/external/skia/src/gpu/GrProcessor.cpp
4bf69f7bbc073c1dc257f93f8c5b2a530123cb3c 26-Aug-2015 rmistry <rmistry@google.com> Revert of Remove GrStagedProcessor, remove the word Stage as it applies to FPs (patchset #6 id:90001 of https://codereview.chromium.org/1307223004/ )

Reason for revert:
Causes bot failures:

https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Debug/builds/1639

https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Debug/builds/1702

https://uberchromegw.corp.google.com/i/client.skia.android/builders/Test-Android-GCC-NexusPlayer-GPU-PowerVR-x86-Debug/builds/1223

Original issue's description:
> Remove GrStagedProcessor, remove the word Stage as it applies to FPs
>
> Committed: https://skia.googlesource.com/skia/+/24243446cdf7b7e4e132c2a0c387c7723777e0c7

TBR=joshualitt@google.com,egdaniel@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1306803003
/external/skia/src/gpu/GrProcessor.cpp
24243446cdf7b7e4e132c2a0c387c7723777e0c7 26-Aug-2015 bsalomon <bsalomon@google.com> Remove GrStagedProcessor, remove the word Stage as it applies to FPs

Review URL: https://codereview.chromium.org/1307223004
/external/skia/src/gpu/GrProcessor.cpp
a71170cce8831b4d1e06bd673560f8b6fea9fbfe 21-Aug-2015 wangyix <wangyix@google.com> This code should've been part of the CL that added registerChildProcessor(); without updating fUsesLocalCoords in the parent when a child proc is registered, batching will not work properly.

BUG=skia:4182

Review URL: https://codereview.chromium.org/1308513004
/external/skia/src/gpu/GrProcessor.cpp
93ab254b7e4ce82074e88e219a8e32b31086ca86 19-Aug-2015 wangyix <wangyix@google.com> This change is in preparation for updating how processor keys and meta keys are generated for frag procs.

BUG=skia:4182

Review URL: https://codereview.chromium.org/1298233002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
54017d7e5b09e7ca58346ea72ff065677fcd7be1 18-Aug-2015 wangyix <wangyix@google.com> Made isEqual in GrFragmentProcessor recursive

Added comment about how computeInvariantOutput() is non-recursive in GrFragmentProcessor

Made isEqual() recursive in GrFragmentProcessor

BUG=skia:4182

Review URL: https://codereview.chromium.org/1287343005
/external/skia/src/gpu/GrProcessor.cpp
7ef45a1aeb2c764d41838323f5bcff149c5de756 13-Aug-2015 wangyix <wangyix@google.com> Added mangleString member and onBefore*, onAfter* functions to GrGLFragmentShaderBuilder

BUILDS! Added mangleString, onBefore, and onAfterChildProcEmitCode() to GrGLFragmentShaderBuilder.cpp

BUG=skia:4182

Review URL: https://codereview.chromium.org/1288723002
/external/skia/src/gpu/GrProcessor.cpp
58d890bd45a201477ec6c4ba0ae61c574b4e5aef 12-Aug-2015 wangyix <wangyix@google.com> All child GrFragmentProcs' transforms and textures will be stored in the root GrFragmentProc in preorder

Changed childProcessor(i) to return const referencd

Fixed rootProc/parentProc offset issues; renamed a few things.

added nonempty check to gatherTransforms to avoid segfault

removed recursive append_gr_coord_transforms() from GrGLProgramBuilder

BUILDS! Changed num*includeProc() calls to num() calls

added gatherCoordTransforms(). added coordTransforms() for root proc only

Modified GrFragmentProcessor to append child proc transforms and textures to root proc's arrays.

BUG=skia:4182

Review URL: https://codereview.chromium.org/1275853005
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
5b4f05f709d2b45f1b61e613a4fd073c31fa66d8 10-Jul-2015 joshualitt <joshualitt@chromium.org> Move GrProcessorDataManager to GrMemoryPool

TBR=bsalomon@google.com
BUG=skia:

Review URL: https://codereview.chromium.org/1225363002
/external/skia/src/gpu/GrProcessor.cpp
71a6cbfc585959738dc0b375603696ca7f60605f 04-May-2015 reed <reed@google.com> remove redundant/deprecated TwoPointRadial gradiet -- use TwoPointConical

This CL derived from https://codereview.chromium.org/1114243005/

BUG=skia:

Review URL: https://codereview.chromium.org/1117423003
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
c9c3e62b4ef6ec288549a0ad1d252986d7f0889d 02-Apr-2015 bsalomon <bsalomon@google.com> Add constant color GrFP.

Committed: https://skia.googlesource.com/skia/+/dfbbec436cbcacc3270d4b28357c8393e67d6494

Review URL: https://codereview.chromium.org/978713002
/external/skia/src/gpu/GrProcessor.cpp
599ea40cec8da11da7ad7c37a2ff0ef4797dbd4a 02-Apr-2015 bsalomon <bsalomon@google.com> Revert of Add constant color GrFP. (patchset #10 id:180001 of https://codereview.chromium.org/978713002/)

Reason for revert:
Revert while investigating assertions.

Original issue's description:
> Add constant color GrFP.
>
> Committed: https://skia.googlesource.com/skia/+/dfbbec436cbcacc3270d4b28357c8393e67d6494

TBR=egdaniel@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1055023002
/external/skia/src/gpu/GrProcessor.cpp
dfbbec436cbcacc3270d4b28357c8393e67d6494 01-Apr-2015 bsalomon <bsalomon@google.com> Add constant color GrFP.

Review URL: https://codereview.chromium.org/978713002
/external/skia/src/gpu/GrProcessor.cpp
23ac62c83a49d675a38f1c20462b5537f3c8af01 30-Mar-2015 joshualitt <joshualitt@chromium.org> Small change to move GrProcessor and GrBatch pools over to SkSpinlock

BUG=skia:

Review URL: https://codereview.chromium.org/1040133002
/external/skia/src/gpu/GrProcessor.cpp
5baedd637806293e9da600af42ada8f75e7db580 09-Mar-2015 bsalomon <bsalomon@google.com> Use global GrMemoryPools protected by mutex for GrProcessor/GrBatch

BUG=chromium:464892

Review URL: https://codereview.chromium.org/991943002
/external/skia/src/gpu/GrProcessor.cpp
4d8da81562852e0ff7e18b66ee1cebd50ad81ee8 28-Jan-2015 joshualitt <joshualitt@chromium.org> GrBatchPrototype

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4

Committed: https://skia.googlesource.com/skia/+/d5a7db4a867c7e6ccf8451a053d987b470099198

Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/GrProcessor.cpp
c2893c5e3870d7e9a37ca146e7da88fba54977d5 28-Jan-2015 joshualitt <joshualitt@google.com> Revert of GrBatchPrototype (patchset #32 id:630001 of https://codereview.chromium.org/845103005/)

Reason for revert:
One last try to fix mac perf regression

Original issue's description:
> GrBatchPrototype
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4
>
> Committed: https://skia.googlesource.com/skia/+/d5a7db4a867c7e6ccf8451a053d987b470099198

TBR=bsalomon@google.com,kkinnunen@nvidia.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/877393002
/external/skia/src/gpu/GrProcessor.cpp
d5a7db4a867c7e6ccf8451a053d987b470099198 28-Jan-2015 joshualitt <joshualitt@chromium.org> GrBatchPrototype

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4

Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/GrProcessor.cpp
ca0a1799ffdceb4ccd998468566f39abac2f0b55 27-Jan-2015 joshualitt <joshualitt@google.com> Revert of GrBatchPrototype (patchset #30 id:570001 of https://codereview.chromium.org/845103005/)

Reason for revert:
creates large performance regression

Original issue's description:
> GrBatchPrototype
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4

TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/862823004
/external/skia/src/gpu/GrProcessor.cpp
d15e4e45374275c045572b304c229237c4a82be4 26-Jan-2015 joshualitt <joshualitt@chromium.org> GrBatchPrototype

BUG=skia:

Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/GrProcessor.cpp
54f0e9d784122cfd3f5968e0fea971d5b5a4805a 16-Jan-2015 egdaniel <egdaniel@google.com> Add Xfer Processor for GrCustomXfermodes

BUG=skia:

Review URL: https://codereview.chromium.org/852203003
/external/skia/src/gpu/GrProcessor.cpp
f351aa3bf1b1d7639be8e1e6a35137b89a8dd93d 14-Jan-2015 egdaniel <egdaniel@google.com> Add Arithmetic mode xp.

BUG=skia:

Review URL: https://codereview.chromium.org/837633005
/external/skia/src/gpu/GrProcessor.cpp
080e673b10ac607305f140ddb245e140ccde40c6 22-Dec-2014 egdaniel <egdaniel@google.com> Add XP to handle the cases where we disable color write.

BUG=skia:

Review URL: https://codereview.chromium.org/787233003
/external/skia/src/gpu/GrProcessor.cpp
290c09b8bbd8d221d363150e2ce87158f4668df0 19-Dec-2014 joshualitt <joshualitt@chromium.org> initial changes to add local matrix to primitive processor

BUG=skia:

Review URL: https://codereview.chromium.org/820523002
/external/skia/src/gpu/GrProcessor.cpp
8750924a1470e8215b2a344155259b93062d3fa2 17-Dec-2014 egdaniel <egdaniel@google.com> Add Coverage Drawing XP

BUG=skia:

Review URL: https://codereview.chromium.org/808813002
/external/skia/src/gpu/GrProcessor.cpp
9b98932adaceb7ad0a617ade16616923f6bffe84 15-Dec-2014 joshualitt <joshualitt@chromium.org> This change will ultimately pull uniform color, and to a much lesser degree uniform coverage, into GPs. There are still some loose ends because drawstate has the ability to override the GP, but fixing these cleanly will have to wait until we have deferred geometry in place and can make attribute / uniform decisions on the fly.

BUG=skia:

Review URL: https://codereview.chromium.org/746423007
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
c230414861558ce20b74281b1ca363e56c441832 11-Dec-2014 egdaniel <egdaniel@google.com> Create xfer processor backend.

This includes:
-Having an actual XP stage at the end of the gl pipeline.
-All Blending work is handled by XP until actually setting GL blend states
-GLPrograms test to test XP

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/4dffc940c430eec66d4707490eace19c9b3f7904

Review URL: https://codereview.chromium.org/764643004
/external/skia/src/gpu/GrProcessor.cpp
294738268d3a9317282224f03063eb38ad19717c 11-Dec-2014 joshualitt <joshualitt@google.com> Revert of Create xfer processor backend. (patchset #6 id:100001 of https://codereview.chromium.org/764643004/)

Reason for revert:
CL breaks windows DM

Original issue's description:
> Create xfer processor backend.
>
> This includes:
> -Having an actual XP stage at the end of the gl pipeline.
> -All Blending work is handled by XP until actually setting GL blend states
> -GLPrograms test to test XP
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4dffc940c430eec66d4707490eace19c9b3f7904

TBR=bsalomon@google.com,joshualitt@chromium.org,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/789343002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
4dffc940c430eec66d4707490eace19c9b3f7904 10-Dec-2014 egdaniel <egdaniel@google.com> Create xfer processor backend.

This includes:
-Having an actual XP stage at the end of the gl pipeline.
-All Blending work is handled by XP until actually setting GL blend states
-GLPrograms test to test XP

BUG=skia:

Review URL: https://codereview.chromium.org/764643004
/external/skia/src/gpu/GrProcessor.cpp
2e3b3e369d79e78f7635d4c20e83a47ab571bdf2 09-Dec-2014 joshualitt <joshualitt@chromium.org> This cl moves color and coverage off of drawstate. In an effort to keep this CL manageable, I have left the compute invariant input / output in a bit of a strange state(fixing this will be complicated).

In addition, NVPR makes this very complicated, and I haven't quite figured out a good way to handle it, so for now color and coverage DO live on optstate, but I will figure out some way to refactor that in future CLs.

BUG=skia:

Review URL: https://codereview.chromium.org/783763002
/external/skia/src/gpu/GrProcessor.cpp
915187b71d4068f21a15a1b8f4d919e1179d8b74 05-Dec-2014 egdaniel <egdaniel@google.com> Check XpFactory equality in DrawState

BUG=skia:

Review URL: https://codereview.chromium.org/767873006
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
378092f3d10b1dd62967f419c35cfefec7c10ee7 03-Dec-2014 egdaniel <egdaniel@google.com> Add XferProcessor factory in GrPaint and GrDrawState.

In this CL the XP should have zero effect on the actual rendering pipeline.

BUG=skia:

Review URL: https://codereview.chromium.org/751283002
/external/skia/src/gpu/GrProcessor.cpp
c07379d6b1a02ddbf1a5fd9518696b3737067532 20-Nov-2014 joshualitt <joshualitt@chromium.org> Adding GeometryData object

BUG=skia:

Review URL: https://codereview.chromium.org/745663002
/external/skia/src/gpu/GrProcessor.cpp
b6cbc38702dc4a1540a17181cbdab71060c92ce7 13-Nov-2014 egdaniel <egdaniel@google.com> Add GrProcOptInfo class to track various output information for color and coverage stages.

BUG=skia:

Review URL: https://codereview.chromium.org/719203002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
4973d9da4aeb7c4d8b8e67e167586c7cc9534eee 08-Nov-2014 joshualitt <joshualitt@chromium.org> Default geometry processor

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/ff343074b2a3fdaa5f120600e28717e366bceadd

Review URL: https://codereview.chromium.org/678953002
/external/skia/src/gpu/GrProcessor.cpp
ee0ea3f0dd878fb5c8354924c0c8627ab2a95c41 07-Nov-2014 joshualitt <joshualitt@google.com> Revert of Default geometry processor (patchset #9 id:160001 of https://codereview.chromium.org/678953002/)

Reason for revert:
breaks nexus 5

Original issue's description:
> Default geometry processor
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/ff343074b2a3fdaa5f120600e28717e366bceadd

TBR=bsalomon@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/691313003
/external/skia/src/gpu/GrProcessor.cpp
ff343074b2a3fdaa5f120600e28717e366bceadd 07-Nov-2014 joshualitt <joshualitt@chromium.org> Default geometry processor

BUG=skia:

Review URL: https://codereview.chromium.org/678953002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.cpp
de258cd6b402c4da78b66e88191ad02162d87916 16-Oct-2014 bsalomon <bsalomon@google.com> Make GrFragmentProcessor auto-compare coord xforms.

Review URL: https://codereview.chromium.org/657923002
/external/skia/src/gpu/GrProcessor.cpp
f2765410ba8adfe934b8d92e52ccc2a847934c61 16-Oct-2014 bsalomon <bsalomon@google.com> Last round of effect->processor (for now)

R=joshualitt@google.com

Review URL: https://codereview.chromium.org/659803005
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.cpp
a5305a110ab5201d5dadd40cbe711582d5ac4996 11-Oct-2014 joshualitt <joshualitt@chromium.org> Opt state takes a GP instead of a GeometryStage

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/71856d520461ae025a0332aa0ce9735a096d9baf

Review URL: https://codereview.chromium.org/637003003
/external/skia/src/gpu/GrProcessor.cpp
89c7a2ec3e9f7e1b9705214f450bb12288799da9 10-Oct-2014 joshualitt <joshualitt@chromium.org> Revert of Opt state takes a GP instead of a GeometryStage (patchset #18 id:1500001 of https://codereview.chromium.org/637003003/)

Reason for revert:
Revert because this may be causing a break in a chrome gpu test

Original issue's description:
> Opt state takes a GP instead of a GeometryStage
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/71856d520461ae025a0332aa0ce9735a096d9baf

TBR=bsalomon@google.com,egdaniel@google.com,danakj@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/647183002
/external/skia/src/gpu/GrProcessor.cpp
71856d520461ae025a0332aa0ce9735a096d9baf 10-Oct-2014 joshualitt <joshualitt@chromium.org> Opt state takes a GP instead of a GeometryStage

BUG=skia:

Review URL: https://codereview.chromium.org/637003003
/external/skia/src/gpu/GrProcessor.cpp
9e87fa7c0e52a79ad85bf32eeb71570938e357bd 09-Oct-2014 joshualitt <joshualitt@chromium.org> Force linking of static member variables for GLPrograms

BUG=skia:

Review URL: https://codereview.chromium.org/614163002
/external/skia/src/gpu/GrProcessor.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/GrProcessor.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/GrProcessor.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/GrProcessor.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/GrProcessor.cpp