Chromium Code Reviews| Index: include/gpu/GrXferProcessor.h |
| diff --git a/include/gpu/GrXferProcessor.h b/include/gpu/GrXferProcessor.h |
| index 812a9b32ab353ec8f6172bf701a77042e2cc02bc..c40c3547c72321aefaeae7ec74d92e50217765c6 100644 |
| --- a/include/gpu/GrXferProcessor.h |
| +++ b/include/gpu/GrXferProcessor.h |
| @@ -363,19 +363,24 @@ public: |
| */ |
| virtual bool supportsRGBCoverage(GrColor knownColor, uint32_t knownColorFlags) const = 0; |
| - struct InvariantOutput { |
| - bool fWillBlendWithDst; |
| - GrColor fBlendedColor; |
| - uint32_t fBlendedColorFlags; |
| + /** |
| + * Known color information after blending, but before accounting for any coverage. |
| + */ |
| + struct InvariantBlendedColor { |
| + bool fWillBlendWithDst; |
| + GrColor fKnownColor; |
| + GrColorComponentFlags fKnownColorFlags; |
| }; |
| /** |
| - * This function returns known information about the output of the xfer processor produced by |
| - * this xp factory. The invariant color information returned by this function refers to the |
| - * final color produced after all blending. |
| + * Returns information about the output color, produced by XPs from this factory, that will be |
| + * known after blending. Note that we can conflate coverage and color, so the actual values |
| + * written to pixels with partial coverage may not always seem consistent with the invariant |
| + * information returned by this function. |
| */ |
| - virtual void getInvariantOutput(const GrProcOptInfo& colorPOI, const GrProcOptInfo& coveragePOI, |
| - InvariantOutput*) const = 0; |
| + virtual void getInvariantBlendedColor(const GrProcOptInfo& colorPOI, |
| + const GrProcOptInfo& coveragePOI, |
|
egdaniel
2015/06/02 14:19:41
Do we need coveragePOI to be in here anymore?
Chris Dalton
2015/06/02 17:07:12
Done.
|
| + InvariantBlendedColor*) const = 0; |
| bool willNeedDstTexture(const GrCaps& caps, const GrProcOptInfo& colorPOI, |
| const GrProcOptInfo& coveragePOI) const; |