diff options
author | fmalita <fmalita@chromium.org> | 2016-02-02 10:17:24 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-02 10:17:25 -0800 |
commit | eb543071725f465e93ddcc052d54fd5c03f043f9 (patch) | |
tree | 13f7ee346ce3fb0652e14d3dd976bfad4c859b6b /src/core/SkBitmapProcState_matrixProcs.cpp | |
parent | f46d8a6fffef5e09c2fe2e64068956b8e19278ac (diff) |
Sampler bias for all nofilter flavors
Instead of a piecemeal approach to sampler bias (http://crrev.com/1642273002,
http://crrev.com/1539083002), audit all nofilter procs (grep "fInvProc(")
and convert them to SkBitmapProcStateAutoMapper.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1656833002
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
R=reed@google.com
Review URL: https://codereview.chromium.org/1656833002
Diffstat (limited to 'src/core/SkBitmapProcState_matrixProcs.cpp')
-rw-r--r-- | src/core/SkBitmapProcState_matrixProcs.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/core/SkBitmapProcState_matrixProcs.cpp b/src/core/SkBitmapProcState_matrixProcs.cpp index 1c4b7b6bcf..d7457ec53b 100644 --- a/src/core/SkBitmapProcState_matrixProcs.cpp +++ b/src/core/SkBitmapProcState_matrixProcs.cpp @@ -328,14 +328,11 @@ static void fill_sequential(uint16_t xptr[], int start, int count) { static int nofilter_trans_preamble(const SkBitmapProcState& s, uint32_t** xy, int x, int y) { - SkPoint pt; - s.fInvProc(s.fInvMatrix, SkIntToScalar(x) + SK_ScalarHalf, - SkIntToScalar(y) + SK_ScalarHalf, &pt); - **xy = s.fIntTileProcY(SkScalarToFixed(pt.fY) >> 16, - s.fPixmap.height()); + const SkBitmapProcStateAutoMapper mapper(s, x, y); + **xy = s.fIntTileProcY(SkFractionalIntToInt(mapper.y()), s.fPixmap.height()); *xy += 1; // bump the ptr // return our starting X position - return SkScalarToFixed(pt.fX) >> 16; + return SkFractionalIntToInt(mapper.x()); } static void clampx_nofilter_trans(const SkBitmapProcState& s, |