History log of /external/skia/src/gpu/GrProcessor.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
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