History log of /external/skia/src/gpu/effects/GrDashingEffect.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
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/GrDashingEffect.cpp
2279325d539700ee3da29d6e874b3b3ce1dcf49c 30-Jan-2016 ethannicholas <ethannicholas@google.com> added support for PLS path rendering

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

Committed: https://skia.googlesource.com/skia/+/7df3f5e127f8016d17b637cc48a6a4718f1a6822

Review URL: https://codereview.chromium.org/1541903002
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
144c3c8b7ff3ebc389b41211f3388fb24a7ff0c2 30-Nov-2015 joshualitt <joshualitt@chromium.org> Make onPrepareDraws const

BUG=skia:

Review URL: https://codereview.chromium.org/1483103003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ff2103200bad7abcf8929ae22ac78a9f4f725142 24-Nov-2015 ethannicholas <ethannicholas@google.com> New API for computing optimization invariants.

Review URL: https://codereview.chromium.org/1467553002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
0eafe79f42e3c675f3c504aed4a41abf511df2b7 20-Nov-2015 egdaniel <egdaniel@google.com> Create GrGLSLVaryingHandler class for program building

BUG=skia:

Review URL: https://codereview.chromium.org/1462123003
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
e659a581f63fdccb64dce2dc8a478cf56831feea 13-Nov-2015 egdaniel <egdaniel@google.com> Move GrGLPrimitive/GeometryProc to GLSL

BUG=skia:

Review URL: https://codereview.chromium.org/1441683008
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
8dcdedc4a087ea46ce1e2458d335d60918e56310 11-Nov-2015 egdaniel <egdaniel@google.com> Make GrGLSLProgramBuilder base class for GrGLProgramBuilder.

This CL still keeps the weird diamond shape we have for all our ProgramBuilders.
However, the GrGLSL base class will allow us to pull multiple other parts
of our program setup away from GL which will eventually allow us to break up
the diamond.

As part of this all ShaderBuilder subclass have been made gl independent,
however I will move them to GLSL files/class names in a follow on CL.

BUG=skia:

Review URL: https://codereview.chromium.org/1416423003
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
1b55a963a2374a14bb82eb887bb99ee91680f0eb 18-Sep-2015 reed <reed@chromium.org> Reland of add a ClassID function to GrBatch (patchset #1 id:1 of https://codereview.chromium.org/1353043002/ )

Reason for revert:
DEPS roll seems to have landed w/o this revert, so revert it again

Original issue's description:
> Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )
>
> Reason for revert:
> Speculative revert to unblock DEPS roll
>
> Original issue's description:
> > add a ClassID function to GrBatch
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
> >
> > Committed: https://skia.googlesource.com/skia/+/eb44d53cf96a7eaf103a98d76079ce1f5495e343
>
> TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@google.com,joshualitt@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/48e3a45ade15c52c0c1a10cb00907dd444897745

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

Review URL: https://codereview.chromium.org/1344373005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
48e3a45ade15c52c0c1a10cb00907dd444897745 18-Sep-2015 reed <reed@chromium.org> Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )

Reason for revert:
Speculative revert to unblock DEPS roll

Original issue's description:
> add a ClassID function to GrBatch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9
>
> Committed: https://skia.googlesource.com/skia/+/eb44d53cf96a7eaf103a98d76079ce1f5495e343

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

Review URL: https://codereview.chromium.org/1353043002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
eb44d53cf96a7eaf103a98d76079ce1f5495e343 17-Sep-2015 joshualitt <joshualitt@chromium.org> add a ClassID function to GrBatch

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9

Review URL: https://codereview.chromium.org/1352813003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
a7008403dcd03302e88e2df546d8427afe9e0e80 17-Sep-2015 joshualitt <joshualitt@google.com> Revert of add a ClassID function to GrBatch (patchset #5 id:80001 of https://codereview.chromium.org/1352813003/ )

Reason for revert:
breaks mac bot

Original issue's description:
> add a ClassID function to GrBatch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4078d529e9e199eea13456db7bf3a63a104ab5b9

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

Review URL: https://codereview.chromium.org/1345393003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
4078d529e9e199eea13456db7bf3a63a104ab5b9 17-Sep-2015 joshualitt <joshualitt@chromium.org> add a ClassID function to GrBatch

BUG=skia:

Review URL: https://codereview.chromium.org/1352813003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
465283cdf98ed9ab5285ca7b9814e430fca1d452 11-Sep-2015 joshualitt <joshualitt@chromium.org> Remove batchtracker

BUG=skia:

Review URL: https://codereview.chromium.org/1332923003
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
96fcdcc219d2a0d3579719b84b28bede76efba64 27-Aug-2015 halcanary <halcanary@google.com> Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002

Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
e46f9feb44780a6269c6dcfe993f4215427fd98e 18-Aug-2015 bsalomon <bsalomon@google.com> Privatize GrBatch subclass overrides

Review URL: https://codereview.chromium.org/1301663002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
7539856c1b9cbb1886a6a498cc534b77fc83ddb2 17-Aug-2015 bsalomon <bsalomon@google.com> Make GrVertexBatch objects hold their own draws during GrDrawTarget flush

NO_MERGE_BUILDS

Review URL: https://codereview.chromium.org/1286043004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
16b991390bb988b194a868ab8de66db4c21c7c13 13-Aug-2015 bsalomon <bsalomon@google.com> Split GrDrawBatch and GrVertexBatch into their own files

TBR=joshualitt@google.com

Review URL: https://codereview.chromium.org/1296483002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
abd30f54b7ff1704a8930c4307ea242d09425d02 13-Aug-2015 bsalomon <bsalomon@google.com> Introduce GrBatch subclasses GrDrawBatch and GrVertexBatch to prepare for non-drawing batches

Review URL: https://codereview.chromium.org/1293583002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
cb02b38b2c48bfde333ce3c699dd0451e2d867fa 12-Aug-2015 bsalomon <bsalomon@google.com> Check for xfer barriers in GrBatch, auto-issue barriers in GrGpu

Review URL: https://codereview.chromium.org/1287973003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
91d844de47bdc1c3f807c85293e46943dcfc712d 10-Aug-2015 bsalomon <bsalomon@google.com> Rename GrPipelineInfo to GrPipelineOptimizations

TBR=joshualitt@google.com

Review URL: https://codereview.chromium.org/1274513005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
74417824cd988f7d6114858e09720ba00416ce11 07-Aug-2015 joshualitt <joshualitt@chromium.org> Make folder for batches

BUG=skia:

Review URL: https://codereview.chromium.org/1277233002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
fb1141ae1db2ffce479bc46413b008d31cf9d3cc 06-Aug-2015 bsalomon <bsalomon@google.com> Don't pass pipeline to GrBatch::generateGeometry()

R=joshualitt@google.com

Review URL: https://codereview.chromium.org/1275083002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
df0c55785033c191d2d509c22662861588e4acd8 03-Aug-2015 joshualitt <joshualitt@chromium.org> Use new API everywhere for GrDefaultGeoProcFactory

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/801823b3d825d96baadeb9077c5ed1a8ada05cf2

Review URL: https://codereview.chromium.org/1261083003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e494a58de341daf08ae23007694a4f6ed4d9cd06 03-Aug-2015 joshualitt <joshualitt@google.com> Revert of Use new API everywhere for GrDefaultGeoProcFactory (patchset #5 id:80001 of https://codereview.chromium.org/1261083003/)

Reason for revert:
breaking things

Original issue's description:
> Use new API everywhere for GrDefaultGeoProcFactory
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/801823b3d825d96baadeb9077c5ed1a8ada05cf2

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

Review URL: https://codereview.chromium.org/1264283004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
801823b3d825d96baadeb9077c5ed1a8ada05cf2 03-Aug-2015 joshualitt <joshualitt@chromium.org> Use new API everywhere for GrDefaultGeoProcFactory

BUG=skia:

Review URL: https://codereview.chromium.org/1261083003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
6af0c9394301b9f976813cc6dbdbbfe779030fe1 22-Jul-2015 wangyix <wangyix@google.com> Moved GrGLFragmentProcessor definition to its own file

BUG=skia:

Review URL: https://codereview.chromium.org/1246193002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
8cab9a7685e872427e6f0388f149575a9b6016ee 16-Jul-2015 joshualitt <joshualitt@chromium.org> All batches do their own pipeline compare

BUG=skia:

Review URL: https://codereview.chromium.org/1242033002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
1c735488cbec9abdb25c9a12e0c11af9c302a776 13-Jul-2015 joshualitt <joshualitt@chromium.org> Start to propagate constness of GrPipelineBuilder up the stack

BUG=skia:

Review URL: https://codereview.chromium.org/1229143007
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
7765a477ee3ca5e2c6ada1e16c31dfaae2079240 08-Jul-2015 bsalomon <bsalomon@google.com> Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.

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

Review URL: https://codereview.chromium.org/1213383005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d07a2793baea42956d27b3ad47ed562f08e4570a 08-Jul-2015 bsalomon <bsalomon@google.com> Revert of Rework GrPipelineInfo (patchset #7 id:120001 of https://codereview.chromium.org/1213383005/)

Reason for revert:
breaking stuff!

Original issue's description:
> Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.
>
> Committed: https://skia.googlesource.com/skia/+/f5179a4c490bc787190321bd8ffdb0e6a4efa9ac

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

Review URL: https://codereview.chromium.org/1213013003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
f5179a4c490bc787190321bd8ffdb0e6a4efa9ac 08-Jul-2015 bsalomon <bsalomon@google.com> Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.

Review URL: https://codereview.chromium.org/1213383005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
cba99b88fd5fb39def7a094dc32c0745c7a1cfea 24-Jun-2015 jvanverth <jvanverth@google.com> Move GLSL-specific routines/classes to separate glsl directory

The purpose is to begin separating our GLSL-specific code from
our GL-specific code, so it can be used with the GL45 platform

Review URL: https://codereview.chromium.org/1202293002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
dded69693dd3779f081326cde24c3954505b129d 12-Jun-2015 vbuzinov <vbuzinov@nvidia.com> Implement support for mixed sampled render targets

Adds a new FBO type kStencil_MSFBOType that is selected whenever
NV_framebuffer_mixed_samples extension is available. In this new
FBO type a non-msaa color buffer is created with a multisampled
stencil buffer attachment.

Replaces numSamples() with separate numColorSamples and numStencilSamples
methods in RenderTarget.

In mixed samples mode non-MSAA codepaths are used to draw simple shapes,
while NVPR-rendered paths and text are rendered with a multisampled
stencil.

BUG=skia:3177

Review URL: https://codereview.chromium.org/1001503002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
1422ec86f2e97323f876a267bc5dae3fbf1eac96 11-Jun-2015 senorblanco <senorblanco@chromium.org> Fix for dashing shader on Mali: use highp where necessary.

The varyings used to compute rect coverage need to be high precision,
since they're offset by the frag coord.

R=egdaniel

BUG=skia:3935

Review URL: https://codereview.chromium.org/1178123004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
eb1cb5c5b50febad115d859faca91d2d6af3fff2 22-May-2015 bsalomon <bsalomon@google.com> rename GrDrawTargetCaps.h to GrCaps.h and move to include

Review URL: https://codereview.chromium.org/1135113005
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
b8c241ad099f3f0c2cbf3e7c10f5f6207175d490 19-May-2015 joshualitt <joshualitt@chromium.org> Preliminary attempt to remove batch tracker

BUG=skia:

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

Review URL: https://codereview.chromium.org/1139723004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d3a560fa80bfb3e2d2e989f951bb3b1c52316654 19-May-2015 joshualitt <joshualitt@google.com> Revert of Preliminary attempt to remove batch tracker (patchset #3 id:40001 of https://codereview.chromium.org/1139723004/)

Reason for revert:
breaking bots

Original issue's description:
> Preliminary attempt to remove batch tracker
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84

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

Review URL: https://codereview.chromium.org/1132323003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84 19-May-2015 joshualitt <joshualitt@chromium.org> Preliminary attempt to remove batch tracker

BUG=skia:

Review URL: https://codereview.chromium.org/1139723004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d156d36af871c23ce471a18764f4597f09cfca95 19-May-2015 kkinnunen <kkinnunen@nvidia.com> Make GrStrokeInfo inherit from SkStrokeRec

Make the code more readable by inheriting GrStrokeInfo from SkStrokeRec.
This should avoid the long .getStrokeRec() and .getStrokeRecPtr(). These
were a bit cumbersome especially in cases where an alias variable was
created for these, and then the reader had to keep track to which
StrokeInfo member the StrokeRec alias was pointing.

Removes SkStrokeRec::SkStrokeRec(const SkStrokeRec&). It was memcpying.
Try to play it safe wrt compiler using the possible padding of
superclass for subclass members. Instead, let the compiler generate
the copy constructor. Assignment operator was already
compiler-generated, so at least in that way this is consistent.

Renames GrStrokeInfo::applyDash to applyDashToPath for consistency
with superclass applyToPath.

Review URL: https://codereview.chromium.org/1128113008
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e3ababe44315452cd33b96a18ce316ede09ff3c3 15-May-2015 joshualitt <joshualitt@chromium.org> remove localmatrix from GrGeometryProcessor base class

BUG=skia:

Review URL: https://codereview.chromium.org/1131513005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e578a95d3ab16544794b94da4e7ae13fc2ce6244 14-May-2015 joshualitt <joshualitt@chromium.org> Remove viewmatrix from GrGeometryProcessor base class

BUG=skia:

Review URL: https://codereview.chromium.org/1127953003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
88c23fc6e8ed0243159fd17d815b4f813805647b 13-May-2015 joshualitt <joshualitt@chromium.org> remove color from GrGeometryProcessor

BUG=skia:

Review URL: https://codereview.chromium.org/1140983002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
1ba8cc9193f13a812401e14b8037fdcf3f3acbac 13-May-2015 joshualitt <joshualitt@chromium.org> removing equality / compute invariant loops from GrGeometryProcessors

BUG=skia:

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

Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
80ea19ca4bdd68c1493666a5fe7e4ce9d43ded8b 12-May-2015 reed <reed@google.com> Revert of Revert of stop calling SkScalarDiv (patchset #1 id:1 of https://codereview.chromium.org/1138263002/)

Reason for revert:
android patched, blink has rolled

Original issue's description:
> Revert of stop calling SkScalarDiv (patchset #4 id:60001 of https://codereview.chromium.org/1135053002/)
>
> Reason for revert:
> need to wait for Blink roll (and patch android)
>
> Original issue's description:
> > stop calling SkScalarDiv
> >
> > BUG=skia:
> > TBR=
> >
> > Committed: https://skia.googlesource.com/skia/+/67d71c898249a7af3523b16c6a69895a63bfae0a
>
> TBR=
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/2629697933b5cc975e45d2a45c48f803fc6cbcec

TBR=
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1135693003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
2629697933b5cc975e45d2a45c48f803fc6cbcec 12-May-2015 reed <reed@google.com> Revert of stop calling SkScalarDiv (patchset #4 id:60001 of https://codereview.chromium.org/1135053002/)

Reason for revert:
need to wait for Blink roll (and patch android)

Original issue's description:
> stop calling SkScalarDiv
>
> BUG=skia:
> TBR=
>
> Committed: https://skia.googlesource.com/skia/+/67d71c898249a7af3523b16c6a69895a63bfae0a

TBR=
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1138263002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
67d71c898249a7af3523b16c6a69895a63bfae0a 12-May-2015 reed <reed@google.com> stop calling SkScalarDiv

BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/1135053002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e64eb570a5b9480bc24d0656ccabcff1ab13a229 07-May-2015 bsalomon <bsalomon@google.com> Iterate over instanced draws in GrGpu rather than above GrBatchTarget

Review URL: https://codereview.chromium.org/1127273007
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e40d39725908d99d255496025411e5c9ca32663e 07-May-2015 robertphillips <robertphillips@google.com> Isolate GrBufferAllocPools inside GrBatchTarget

This CL refactors the location of the GrBufferAllocPools so they reside entirely inside the GrBatchTarget. This is in preparation for making them use scratch resources.

Review URL: https://codereview.chromium.org/1131553002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
261694c98a4de4c67fb719e365c0345da0473a17 05-May-2015 kkinnunen <kkinnunen@nvidia.com> Avoid using SkPathEffect::DashInfo in GrStrokeInfo

Avoid using SkPathEffect::DashInfo for storing the phase
in GrStrokeInfo. Instead, just use normal instance variables.

Fixes the copy constructor pointing the DashInfo interval pointer
to the wrong interval data.

Also fixes GrStrokeInfo::setDashInfo(const SkPathEffect::DashInfo&)
by updating the fDashType correctly.

Makes it simpler to write code such as the operator== in the future.

Review URL: https://codereview.chromium.org/1110093002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
b5238a7571c243ba4a154a62575570c3078b3741 05-May-2015 bsalomon <bsalomon@google.com> Start on simplifying generateGeometry() overrides

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

Review URL: https://codereview.chromium.org/1122673002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
8415abe44cf205ac214b7793c076fd8c13272240 04-May-2015 bsalomon <bsalomon@google.com> Revert of Start on simplifying generateGeometry() overrides (patchset #10 id:160001 of https://codereview.chromium.org/1122673002/)

Reason for revert:
Breaking bots

Original issue's description:
> Start on simplifying generateGeometry() overrides
>
> Committed: https://skia.googlesource.com/skia/+/f28381c6866cad92af8ebe5b9d2db074613b1963

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

Review URL: https://codereview.chromium.org/1124633003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
f28381c6866cad92af8ebe5b9d2db074613b1963 04-May-2015 bsalomon <bsalomon@google.com> Start on simplifying generateGeometry() overrides

Review URL: https://codereview.chromium.org/1122673002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ed0bcad9c8147fd37c23bdda00ec27ec9ef8d66b 04-May-2015 bsalomon <bsalomon@google.com> Move instanced index buffer creation to flush time

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

Review URL: https://codereview.chromium.org/1116943004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d8ed1b64a6c85a8364c23e825164b7bebb43a3a0 04-May-2015 bsalomon <bsalomon@google.com> Revert of Move instanced index buffer creation to flush time (patchset #6 id:100001 of https://codereview.chromium.org/1116943004/)

Reason for revert:
messed up caching, recreating index buffers all the time.

Original issue's description:
> Move instanced index buffer creation to flush time
>
> Committed: https://skia.googlesource.com/skia/+/ab622c7b8cc8c39f0a594e4392b9e31b7e1ddb26

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

Review URL: https://codereview.chromium.org/1126613003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ab622c7b8cc8c39f0a594e4392b9e31b7e1ddb26 04-May-2015 bsalomon <bsalomon@google.com> Move instanced index buffer creation to flush time

Review URL: https://codereview.chromium.org/1116943004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
99c7c07e0f1f7b78980eb21d84bebda8b45a7178 01-May-2015 joshualitt <joshualitt@chromium.org> Move bounds to GrBatch

BUG=skia:

Review URL: https://codereview.chromium.org/1121463002
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
fa2008ff6ee5963fc2566b20c1af39367b77bdf7 29-Apr-2015 joshualitt <joshualitt@chromium.org> Add dash lines batch unit test

BUG=skia:

Review URL: https://codereview.chromium.org/1108053004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
4eaf9cef5a76098f78efac30beb966ac833d32c2 28-Apr-2015 joshualitt <joshualitt@chromium.org> create GrTestUtils.h, move some common functions into it

BUG=skia:

Review URL: https://codereview.chromium.org/1117443002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
50cb76b2bb0ffa607a1409f77be0ae7d64e31436 28-Apr-2015 joshualitt <joshualitt@google.com> Revert of removing equality / compute invariant loops from GrGeometryProcessors (patchset #2 id:20001 of https://codereview.chromium.org/1111603004/)

Reason for revert:
breaks gl programs

Original issue's description:
> removing equality / compute invariant loops from GrGeometryProcessors
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/ef292a0901205b9785a30daae2c036aa34a970ca

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

Review URL: https://codereview.chromium.org/1110993002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ef292a0901205b9785a30daae2c036aa34a970ca 28-Apr-2015 joshualitt <joshualitt@chromium.org> removing equality / compute invariant loops from GrGeometryProcessors

BUG=skia:

Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
72e3ae486c66871c2043eac4f08d85d419fbca2a 28-Apr-2015 bsalomon <bsalomon@google.com> Remove vertex/index buffer factilities from GrDrawTarget.

Review URL: https://codereview.chromium.org/1105263002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
1899651ffc459f5462aa989cd6d08507947b67e4 27-Apr-2015 kkinnunen <kkinnunen@nvidia.com> Extract gpu line dashing to GrDashLinePathRenderer

Move line dashing logic from GrContext::drawPath to
GrDashLinePathRenderer. This makes it possible to let path renderers render arbitrary dashed paths.

End goal is to implement dashing in GrStencilAndCoverPathRenderer.

Review URL: https://codereview.chromium.org/1100073003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
f3c2c46cf818fe408d0ec295338002fc01c64926 20-Apr-2015 senorblanco <senorblanco@chromium.org> Make the GPU dashing effect MSAA-friendly.

Review URL: https://codereview.chromium.org/1092793006
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
4b31de8328bbf3ee789157ae1dc6fe7cc74c796a 05-Mar-2015 joshualitt <joshualitt@chromium.org> handle null vertex or index buffers in batch

BUG=skia:

Review URL: https://codereview.chromium.org/979343002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d0f5457c5ec974b3a0b080125a5ba07214c6d9d2 02-Mar-2015 joshualitt <joshualitt@chromium.org> valgrind fix

TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/968873006
/external/skia/src/gpu/effects/GrDashingEffect.cpp
4f569beac777afde2f2dd9079a52535f8e7519b3 27-Feb-2015 joshualitt <joshualitt@chromium.org> Dash batch

BUG=skia:

Review URL: https://codereview.chromium.org/925673002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
44701df5ce572ac3cccec785cf52103d3d5d14a5 23-Feb-2015 joshualitt <joshualitt@chromium.org> Move clip off of draw target

BUG=skia:

Review URL: https://codereview.chromium.org/947443003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
dd2198701b0ec9da61ecf73418cad03642d715e7 12-Feb-2015 joshualitt <joshualitt@chromium.org> fix for uniform view matrix being uploaded but not used

As discussed, I'll follow this up by removing localmatrix/uniform view matrix from the base classe

BUG=skia:

Review URL: https://codereview.chromium.org/920933002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
5224ba7860ca8c734055895eac4b67888737047d 04-Feb-2015 joshualitt <joshualitt@chromium.org> Dash using vertex attributes for interval and stroke width

BUG=skia:

Review URL: https://codereview.chromium.org/892203003
/external/skia/src/gpu/effects/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.cpp
d15e4e45374275c045572b304c229237c4a82be4 26-Jan-2015 joshualitt <joshualitt@chromium.org> GrBatchPrototype

BUG=skia:

Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
8dd688b7569df569a672a8a67b2db86a9d376cfc 22-Jan-2015 egdaniel <egdaniel@google.com> Rename GrOptDrawState to GrPipeline and GrDrawState to GrPipelineBuilder

BUG=skia:

Review URL: https://codereview.chromium.org/858343002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
46d36f0e7b709a077c647841eee23bd3efdc4117 18-Jan-2015 robertphillips <robertphillips@google.com> Refactor position computation to enable device space "nudge"

To match raster's handling of BW geometry we want to be able to perform a device space "nudge" on all geometry. This CL sets us up to be able to do that in GrGLVertexBuilder::transformToNormalizedDeviceSpace.

BUG=423834
TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/854013002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
71c9260e6fa1798ad1e41b2c2ae9b3cce08bb610 14-Jan-2015 joshualitt <joshualitt@chromium.org> More changes to bring together path / geo procs

BUG=skia:

Review URL: https://codereview.chromium.org/820783005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
abb52a1a70a81915c6196e0fb3e9bcb05e8be14d 14-Jan-2015 joshualitt <joshualitt@chromium.org> Move most of the transform logic into the primitive processors

BUG=skia:

Review URL: https://codereview.chromium.org/822423004
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.cpp
ee2af95db72152dfa61c841875df0594ca93437d 30-Dec-2014 joshualitt <joshualitt@chromium.org> ViewMatrix uniform upload moved to GeometryProcessor

BUG=skia:

Review URL: https://codereview.chromium.org/827973002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
8059eb9f6e24ed609393fbda4ad71edea03ac258 30-Dec-2014 joshualitt <joshualitt@chromium.org> Move ViewMatrix off of drawstate

BUG=skia:

Review URL: https://codereview.chromium.org/815553003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
d27f73ef27ff65a6a0a5d00aa8e5b784b1a0b47e 29-Dec-2014 joshualitt <joshualitt@chromium.org> Remove coordchanges from drawstate

BUG=skia:

Review URL: https://codereview.chromium.org/818233002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
8fc6c2d82c1f30ff82274334c01f0799def6a609 23-Dec-2014 joshualitt <joshualitt@chromium.org> This CL cleans up the last remaining users of localCoordChange on paint

NOTREECHECKS=True
BUG=skia:

Review URL: https://codereview.chromium.org/817853002
/external/skia/src/gpu/effects/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
87f48d997ec29e5eeaa7567355775e93465dd60d 04-Dec-2014 joshualitt <joshualitt@chromium.org> create and thread batch tracker object

I remove the factory on the next CL

BUG=skia:

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

Review URL: https://codereview.chromium.org/772513002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
841a6b54c11137e210aa81e8b56e1763c3571cb8 04-Dec-2014 joshualitt <joshualitt@google.com> Revert of create and thread batch tracker object (patchset #9 id:160001 of https://codereview.chromium.org/772513002/)

Reason for revert:
Breaking linux layout test

Original issue's description:
> create and thread batch tracker object
>
> I remove the factory on the next CL
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/c3a6eb23483e5d28073b509a5f637f41660de294

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

Review URL: https://codereview.chromium.org/774133003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
c3a6eb23483e5d28073b509a5f637f41660de294 04-Dec-2014 joshualitt <joshualitt@chromium.org> create and thread batch tracker object

I remove the factory on the next CL

BUG=skia:

Review URL: https://codereview.chromium.org/772513002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
2dd1ae016d7f297b433c3ea3a771ef8e01657c1f 03-Dec-2014 joshualitt <joshualitt@chromium.org> First step to moving vertex attributes to the geometryProcessor

BUG=skia:

Review URL: https://codereview.chromium.org/761563002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
9853ccef19c200be93a6211f32589fa82a53067c 17-Nov-2014 joshualitt <joshualitt@chromium.org> Drawstate on stack

BUG=skia:

Review URL: https://codereview.chromium.org/732693002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
5478d427c6e67c986a3390162c8fec77c466058a 15-Nov-2014 joshualitt <joshualitt@chromium.org> Push creation of default GP to the caller

TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/715903002
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.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/effects/GrDashingEffect.cpp
38406c82b913350e55fa04af8c1941cd9b4aff52 31-Oct-2014 tfarina <tfarina@chromium.org> Cleanup: Go with SkDebugf instead of GrPrintf.

Since GrPrintf is just defined to SkDebugf, we can go with the later
directly.

BUG=None
TEST=None
R=bsalomon@google.com

Review URL: https://codereview.chromium.org/695663003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
74077b9941ed3f73d92ba978ef29bf3e6f630cbc 24-Oct-2014 joshualitt <joshualitt@chromium.org> Added varying struct

TBR=
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe

Review URL: https://codereview.chromium.org/671023002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
c6f3e2c17b3a7ccfd1ca473652ee9e34e89fad0a 24-Oct-2014 joshualitt <joshualitt@google.com> Revert of Added varying struct (patchset #9 id:160001 of https://codereview.chromium.org/671023002/)

Reason for revert:
may have caused gm change on arm

Original issue's description:
> Added varying struct
>
> TBR=
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe

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

Review URL: https://codereview.chromium.org/675193002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
852ae80b9c3c6fd53f993ac35133d80863993cbe 24-Oct-2014 joshualitt <joshualitt@chromium.org> Added varying struct

TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/671023002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
c369e7c9992a86151a3ea0516ce5308c211b196b 22-Oct-2014 joshualitt <joshualitt@chromium.org> GP Emit Code takes a struct

BUG=skia:

Review URL: https://codereview.chromium.org/666693012
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
47bb38283072dc87dc93220cd2f370ca109972ff 08-Oct-2014 joshualitt <joshualitt@chromium.org> Cleanup of shader building system

this is a huge refactor and cleanup of the gl shader building system in
Skia. The entire shader building pipeline is now part of
GrGLProgramCreator, which takes a gp, and some fps, and creates a
program. I added some subclasses of GrGLProgram to handle the
eccentricities of Nvpr/Nvpres. Outside of the builders folder
and GrGLPrograms, this change is basically just a rename

solo gp

BUG=skia:

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

Review URL: https://codereview.chromium.org/611653002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
db0d3ca07008ea2c1f24d0fd9ceecf10af6ae63b 07-Oct-2014 joshualitt <joshualitt@chromium.org> Revert of Cleanup of shader building system (patchset #25 id:470001 of https://codereview.chromium.org/611653002/)

Reason for revert:
Seems to have messed up windows 7 gms

Original issue's description:
> Cleanup of shader building system
>
> this is a huge refactor and cleanup of the gl shader building system in
> Skia. The entire shader building pipeline is now part of
> GrGLProgramCreator, which takes a gp, and some fps, and creates a
> program. I added some subclasses of GrGLProgram to handle the
> eccentricities of Nvpr/Nvpres. Outside of the builders folder
> and GrGLPrograms, this change is basically just a rename
>
>
> solo gp
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/fe1233c3f12f81bb675718516bbb32f72af726ec

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

Review URL: https://codereview.chromium.org/635533005
/external/skia/src/gpu/effects/GrDashingEffect.cpp
fe1233c3f12f81bb675718516bbb32f72af726ec 07-Oct-2014 joshualitt <joshualitt@chromium.org> Cleanup of shader building system

this is a huge refactor and cleanup of the gl shader building system in
Skia. The entire shader building pipeline is now part of
GrGLProgramCreator, which takes a gp, and some fps, and creates a
program. I added some subclasses of GrGLProgram to handle the
eccentricities of Nvpr/Nvpres. Outside of the builders folder
and GrGLPrograms, this change is basically just a rename

solo gp

BUG=skia:

Review URL: https://codereview.chromium.org/611653002
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
408d6125b32c86e1f81ce60465e3bf4491e755fc 17-Sep-2014 joshualitt <joshualitt@chromium.org> Breaking out full program and frag only

BUG=skia:
R=bsalomon@google.com

Author: joshualitt@chromium.org

Review URL: https://codereview.chromium.org/576543005
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
249af15fb82833d2274850c589812b6e69df0033 15-Sep-2014 joshualitt <joshualitt@chromium.org> BUG=skia:
R=bsalomon@google.com, egdaniel@google.com, jvanverth@google.com, robertphillips@google.com

Author: joshualitt@chromium.org

Review URL: https://codereview.chromium.org/543623004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
bd769d0f1c8cf6ccbb2738dfad1624a4c828e4eb 04-Sep-2014 joshualitt <joshualitt@chromium.org> Initial change to create GeometryProcessor

BUG=skia:
R=bsalomon@google.com, robertphillips@google.com, egdaniel@google.com, jvanverth@google.com

Author: joshualitt@chromium.org

Review URL: https://codereview.chromium.org/509153002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
7b3d5ee72c2238aa239bce4d5b3aea98a437ca7a 28-Aug-2014 egdaniel <egdaniel@google.com> Make setVertexAttribs in GrDrawState take a stride parameter.

BUG=skia:

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

R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/511593004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ea81ced1e573c98584e6830a92511f80593af0a4 27-Aug-2014 djsollen <djsollen@google.com> Revert of Make setVertexAttribs in GrDrawState take a stride parameter. (patchset #5 of https://codereview.chromium.org/511593004/)

Reason for revert:
GM failures on http://108.170.220.120:10115/builders/Test-Ubuntu12-ShuttleA-GTX660-x86-Release/builds/1867/

Original issue's description:
> Make setVertexAttribs in GrDrawState take a stride parameter.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/af013bf8afc4c77ab8ff230f536e2ade973427fa

R=bsalomon@google.com, egdaniel@google.com
TBR=bsalomon@google.com, egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Author: djsollen@google.com

Review URL: https://codereview.chromium.org/502533004
/external/skia/src/gpu/effects/GrDashingEffect.cpp
af013bf8afc4c77ab8ff230f536e2ade973427fa 27-Aug-2014 egdaniel <egdaniel@google.com> Make setVertexAttribs in GrDrawState take a stride parameter.

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/511593004
/external/skia/src/gpu/effects/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.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/GrDashingEffect.cpp
55fad7af61c21d502acb9891d631e8aa29e3628c 08-Jul-2014 bsalomon <bsalomon@google.com> Remove GrEffect::CreateEffectRef and GrEffect::AutoEffectRef.

R=robertphillips@google.com

Author: bsalomon@google.com

Review URL: https://codereview.chromium.org/371103003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
a88702642100466d521a68fc8c5f7019ea101e65 02-Jul-2014 egdaniel <egdaniel@google.com> Fix int to float bug in shader on gpu Dotted Line cl
NOTRY=True
NOTREECHECKS=True
R=bsalomon@google.com
TBR=bsalomon@google.com
BUG=skia:

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/361253002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
f767e792b929df9a2d9f35f9c68adaa9251f4da3 02-Jul-2014 egdaniel <egdaniel@google.com> Add the ability for gpu to render dotted lines (dashed line, 0 on interval, round caps)

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/345113003
/external/skia/src/gpu/effects/GrDashingEffect.cpp
e61c411c1258a323a010558c08de3d9f8d170dca 12-Jun-2014 egdaniel <egdaniel@google.com> Use vertex attributes for dash effect in gpu

This will allow us to batch dashed lines together when drawing. Also, this removes the need for
a coord transform matrix in the shader, thus we save the cost of uploading a new matrix uniform
everytime we do a simple transform to the dashed line we are drawing.

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/326103002
/external/skia/src/gpu/effects/GrDashingEffect.cpp
3b9e8be760ad0e576d5145e0aecf0dc73efcd640 20-May-2014 skia.committer@gmail.com <skia.committer@gmail.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@14793 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrDashingEffect.cpp
3595f88aff0a1b490aa6661f0ba2d3534ee86353 19-May-2014 commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> Pass in GrContext instead of SkGpuDevice for dashing and Sk2GrPaint conversion


BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/292773002

git-svn-id: http://skia.googlecode.com/svn/trunk@14790 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrDashingEffect.cpp
ad88340cf16162654ebddf4783ebb2ac56e1c3aa 19-May-2014 commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> Fix build error for windows for Dashing CL
NOTREECHECKS=True
NOTRY=True
R=bsalomon@google.com
TBR=bsalomon@google.com
BUG=skia:

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/291803004

git-svn-id: http://skia.googlecode.com/svn/trunk@14776 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrDashingEffect.cpp
628ed0b22045a9544b56e00e73137d6050ce900c 19-May-2014 commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> Add Dashing gpu effect for simple dashed lines

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/274673004

git-svn-id: http://skia.googlecode.com/svn/trunk@14775 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/effects/GrDashingEffect.cpp