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

Unified Diff: src/codec/SkSwizzler.h

Issue 1563393002: Use SkOpts routines in SkSwizzler (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 11 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
Index: src/codec/SkSwizzler.h
diff --git a/src/codec/SkSwizzler.h b/src/codec/SkSwizzler.h
index bdb10a14163da8b14f58569f23638fa5b3db3eea..153030377976549d7a56e21f0e4f80f25ca014e4 100644
--- a/src/codec/SkSwizzler.h
+++ b/src/codec/SkSwizzler.h
@@ -192,7 +192,11 @@ private:
int dstWidth, int bpp, int deltaSrc, int offset,
const SkPMColor ctable[]);
- const RowProc fRowProc;
+ RowProc fRowProc;
+ // We use this as a fallback. If the client requests sampling, we will
+ // most likely need to use this simpler row proc in place of a more highly
+ // optimized version.
+ const RowProc fScalarRowProc;
mtklein 2016/01/08 15:20:19 Scalar sounds like SkScalar to me. What about jus
msarett 2016/01/11 20:33:25 Done.
const SkPMColor* fColorTable; // Unowned pointer
// Subset Swizzles
@@ -279,8 +283,8 @@ private:
// fBPP is bitsPerPixel
const int fDstBPP; // Bytes per pixel for the destination color type
- SkSwizzler(RowProc proc, const SkPMColor* ctable, int srcOffset, int srcWidth, int dstOffset,
- int dstWidth, int srcBPP, int dstBPP);
+ SkSwizzler(RowProc optProc, RowProc scalarProc, const SkPMColor* ctable, int srcOffset,
+ int srcWidth, int dstOffset, int dstWidth, int srcBPP, int dstBPP);
int onSetSampleX(int) override;
« no previous file with comments | « gyp/codec.gyp ('k') | src/codec/SkSwizzler.cpp » ('j') | src/codec/SkSwizzler.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698