Add mixed samples support to XPs

Add mixed samples support to XPs

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/86ae0a9e465f157eaa263ef7515e10619946ff83

[Chris Dalton, 2015-06-04 01:19:17]

cdalton@nvidia.com changed reviewers:
+ bsalomon@google.com, egdaniel@google.com

[Chris Dalton, 2015-06-04 01:19:18]

https://codereview.chromium.org/1164973002/diff/20001/include/gpu/GrXferProce...
File include/gpu/GrXferProcessor.h (right):

https://codereview.chromium.org/1164973002/diff/20001/include/gpu/GrXferProce...
include/gpu/GrXferProcessor.h:370: GrXferProcessor* createXferProcessor(const
GrPipelineBuilder&,
I thought it might be nicer to just send in a PipelineBuilder rather than a
parameter like 'bool hasMixedSamples'.

(In the future we may also be able to hang the dst texture and/or POIs off the
PipelineBuilder?)

[Mark Kilgard, 2015-06-04 05:08:36]

markkilgard@gmail.com changed reviewers:
+ markkilgard@gmail.com

[Mark Kilgard, 2015-06-04 05:08:36]

lgtm

[Chris Dalton, 2015-06-04 08:22:31]

Patchset 3 is maybe a bit more exploratory. It changes GrGLXferProcessor to add
the coverage calculations instead of the subclass when we are performing a dst
read. I might have some more ideas tomorrow on how to do this better, but I
wanted to make sure the idea was on the table before you guys looked at it.

Feedback welcome.

[Chris Dalton, 2015-06-05 06:36:48]

Cleaned it all up. I split this patch into two logical changes. The first one is
here:

https://codereview.chromium.org/1170553002/

[egdaniel, 2015-06-08 21:29:46]

https://codereview.chromium.org/1164973002/diff/120001/src/gpu/effects/GrCove...
File src/gpu/effects/GrCoverageSetOpXP.cpp (right):

https://codereview.chromium.org/1164973002/diff/120001/src/gpu/effects/GrCove...
src/gpu/effects/GrCoverageSetOpXP.cpp:232: SkASSERT(!fInvertCoverage ||
!hasMixedSamples);
Lets return NULL here if we hit this case

[Chris Dalton, 2015-06-08 21:33:25]

The CQ bit was checked by cdalton@nvidia.com to run a CQ dry run

[Chris Dalton, 2015-06-08 21:33:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
markkilgard@gmail.com
Link to the patchset: https://codereview.chromium.org/1164973002/#ps140001
(title: " ")

[commit-bot: I haz the power, 2015-06-08 21:33:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1164973002/140001

[commit-bot: I haz the power, 2015-06-08 21:33:48]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-06-09 03:33 UTC

[egdaniel, 2015-06-08 21:34:12]

lgtm

[reed1, 2015-06-08 21:40:47]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2015-06-08 21:40:48]

api lgtm

[commit-bot: I haz the power, 2015-06-08 21:46:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-08 21:46:28]

Dry run: This issue passed the CQ dry run.

[Chris Dalton, 2015-06-08 22:09:49]

The CQ bit was checked by cdalton@nvidia.com

[commit-bot: I haz the power, 2015-06-08 22:10:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1164973002/140001

[commit-bot: I haz the power, 2015-06-08 22:11:09]

Committed patchset #8 (id:140001) as
https://skia.googlesource.com/skia/+/86ae0a9e465f157eaa263ef7515e10619946ff83