Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: include/core/SkShader.h

Issue 1810383004: allow more options for shader blitprocs (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: gcc hates my curlies Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/core/SkBlitter_PM4f.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/core/SkShader.h
diff --git a/include/core/SkShader.h b/include/core/SkShader.h
index 39ffd86f3a369d9c31c177cb640e1779e9c09104..c9c760d06d9ed01e897c41a4cb1e615f35f0a83d 100644
--- a/include/core/SkShader.h
+++ b/include/core/SkShader.h
@@ -142,18 +142,35 @@ public:
virtual void shadeSpan4f(int x, int y, SkPM4f[], int count);
+ struct BlitState;
+ typedef void (*BlitBW)(BlitState*,
+ int x, int y, const SkPixmap&, int count);
+ typedef void (*BlitAA)(BlitState*,
+ int x, int y, const SkPixmap&, int count, const SkAlpha[]);
+
struct BlitState {
+ // inputs
Context* fCtx;
SkXfermode* fXfer;
+
+ // outputs
enum { N = 2 };
void* fStorage[N];
+ BlitBW fBlitBW;
+ BlitAA fBlitAA;
};
- typedef void (*BlitProc)(BlitState*,
- int x, int y, const SkPixmap&, int count, const SkAlpha[]);
- BlitProc chooseBlitProc(const SkImageInfo& info, BlitState* state) {
- return this->onChooseBlitProc(info, state);
+
+ // Returns true if one or more of the blitprocs are set in the BlitState
+ bool chooseBlitProcs(const SkImageInfo& info, BlitState* state) {
+ state->fBlitBW = nullptr;
+ state->fBlitAA = nullptr;
+ if (this->onChooseBlitProcs(info, state)) {
+ SkASSERT(state->fBlitBW || state->fBlitAA);
+ return true;
+ }
+ return false;
}
-
+
/**
* The const void* ctx is only const because all the implementations are const.
* This can be changed to non-const if a new shade proc needs to change the ctx.
@@ -188,9 +205,7 @@ public:
MatrixClass getInverseClass() const { return (MatrixClass)fTotalInverseClass; }
const SkMatrix& getCTM() const { return fCTM; }
- virtual BlitProc onChooseBlitProc(const SkImageInfo&, BlitState*) {
- return nullptr;
- }
+ virtual bool onChooseBlitProcs(const SkImageInfo&, BlitState*) { return false; }
private:
SkMatrix fCTM;
« no previous file with comments | « no previous file | src/core/SkBlitter_PM4f.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698